Re: [TICTOC] comments on draft-ietf-tictoc-1588overmpls-04.txt

"Shahram Davari" <davari@broadcom.com> Thu, 06 June 2013 10:25 UTC

Return-Path: <davari@broadcom.com>
X-Original-To: tictoc@ietfa.amsl.com
Delivered-To: tictoc@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 6CB0E21F992C for <tictoc@ietfa.amsl.com>; Thu, 6 Jun 2013 03:25:28 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.599
X-Spam-Level:
X-Spam-Status: No, score=-6.599 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, RCVD_IN_DNSWL_MED=-4]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id wHY0CJIAa-9j for <tictoc@ietfa.amsl.com>; Thu, 6 Jun 2013 03:25:24 -0700 (PDT)
Received: from mms3.broadcom.com (mms3.broadcom.com [216.31.210.19]) by ietfa.amsl.com (Postfix) with ESMTP id 378FB21F9926 for <tictoc@ietf.org>; Thu, 6 Jun 2013 03:25:24 -0700 (PDT)
Received: from [10.9.208.57] by mms3.broadcom.com with ESMTP (Broadcom SMTP Relay (Email Firewall v6.5)); Thu, 06 Jun 2013 03:16:19 -0700
X-Server-Uuid: B86B6450-0931-4310-942E-F00ED04CA7AF
Received: from SJEXCHCAS05.corp.ad.broadcom.com (10.16.203.12) by IRVEXCHCAS08.corp.ad.broadcom.com (10.9.208.57) with Microsoft SMTP Server (TLS) id 14.1.438.0; Thu, 6 Jun 2013 03:25:14 -0700
Received: from SJEXCHMB12.corp.ad.broadcom.com ( [fe80::bc15:c1e1:c29a:36f7]) by SJEXCHCAS05.corp.ad.broadcom.com ( [::1]) with mapi id 14.01.0438.000; Thu, 6 Jun 2013 03:25:14 -0700
From: Shahram Davari <davari@broadcom.com>
To: Lars Ellegaard <le@vitesse.com>
Thread-Topic: [TICTOC] comments on draft-ietf-tictoc-1588overmpls-04.txt
Thread-Index: AQHOHrvIG9Qp4DAEq0WS7lIAjM8M8JkmuDRwgAE3KqCAAAYJEIAA94OwgAAV4Qk=
Date: Thu, 06 Jun 2013 10:25:14 +0000
Message-ID: <7E018846-4BF7-42C2-9AE8-FA44C765E2F0@broadcom.com>
References: <20130223072233.32735.89371.idtracker@ietfa.amsl.com> <64683B4805156847B0ED373DAB3057CA015E930D@ottsrv0015.microsemi.net> <4A6CE49E6084B141B15C0713B8993F281BE28013@SJEXCHMB12.corp.ad.broadcom.com> <64683B4805156847B0ED373DAB3057CA01B19D5F@ottsrv0015.microsemi.net> <1B5CAFEB4D81154AA0F020783784C3150C775B@ESESSMB301.ericsson.se>, <785DA01679B0C44F951747BE9CF66E6B017994ABA1@mbox-cam4.vitesse.com>
In-Reply-To: <785DA01679B0C44F951747BE9CF66E6B017994ABA1@mbox-cam4.vitesse.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
MIME-Version: 1.0
X-WSS-ID: 7DAEBEF92L828724753-01-01
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
Cc: "tictoc@ietf.org" <tictoc@ietf.org>
Subject: Re: [TICTOC] comments on draft-ietf-tictoc-1588overmpls-04.txt
X-BeenThere: tictoc@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: Timing over IP Connection and Transfer of Clock BOF <tictoc.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/tictoc>, <mailto:tictoc-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/tictoc>
List-Post: <mailto:tictoc@ietf.org>
List-Help: <mailto:tictoc-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/tictoc>, <mailto:tictoc-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 06 Jun 2013 10:25:28 -0000

Hi Lars,

I have actually removed this paragraph completely for new version.

Regards,
Shahram


On Jun 6, 2013, at 2:23 AM, "Lars Ellegaard" <le@vitesse.com> wrote:

> Hi Shahram, Peter,
> 
> I noticed a point which I disagree on. As with Stefano's comment this refers to both text proposals.
> It regards the last sentence of the text block below:
> 
> "
> Also, LER/LSR TCs MUST be
>   synchronized to the same master clock (which is the service provider
>   Master clock), which implies that they need to choose one master to
>   which it will synchronize.
> "
> 
> In general a TC need not be synchronized to a master. It may be so (syntonization) for increased accuracy but it is not a requirement per IEEE1588-2008. The TC may also be synchronized because the LER/LSR implementation includes OC-slave or BC functionality in addition to TC.
> 
> I suggest changing to
> 
> Also, LER/LSR TCs MAY be
>   synchronized to the same master clock (which is the service provider
>   Master clock), which implies that they need to choose one master to
>   which it will synchronize.
> 
> 
> Best regards,
> Lars
> 
> 
> 
> 
> -----Original Message-----
> From: tictoc-bounces@ietf.org [mailto:tictoc-bounces@ietf.org] On Behalf Of Stefano Ruffini
> Sent: 05 June 2013 20:31
> To: Meyer, Peter; Shahram Davari; tictoc@ietf.org
> Subject: Re: [TICTOC] comments on draft-ietf-tictoc-1588overmpls-04.txt
> 
> Hi Shahram, Peter
> 
> There is one point I would disagree with (I am referring to Peter's latest proposal, but the same comment would apply to the previous text):
> "
> An MPLS domain may serve multiple customers.  In these cases the
>   MPLS domain (maintained by a service provider) may provide timing
>   services to multiple customers, each having their own Timing domain.
>   If so, then the LER BCs and TCs may interact with multiple grandmasters,
>   and consequently multiple time references.  Also, LER/LSR TCs MUST be
>   synchronized to the same master clock (which is the service provider
>   Master clock), which implies that they need to choose one master to
>   which it will synchronize.
> "
> 
> My understanding of this statement is that a LER would need to implement multiple BC instances (with multiple clocks); this looks too complex.
> 
> The case of multiple operators, seems to me only practically applicable to figure 3 (LER TC).
> 
> I would then propose to remove the reference to LER BC in this statement .
> 
> Best regards
> stefano
> 
> 
> -----Original Message-----
> From: tictoc-bounces@ietf.org [mailto:tictoc-bounces@ietf.org] On Behalf Of Meyer, Peter
> Sent: mercoledì 5 giugno 2013 20:01
> To: Shahram Davari; tictoc@ietf.org
> Subject: Re: [TICTOC] comments on draft-ietf-tictoc-1588overmpls-04.txt
> 
> Hi Shahram,
> 
> I hope in-line with my comments I have edited the text of draft -04.
> 
> I'm not sure how the formatting worked out (PC-based, line length, page length).
> 
> I modified sections 3, 4, 18, 19 & 20.
> 
> I include here both original and modified version so you can diff (most of the changes in 3 are due to line length / re-org, not actual text changes).
> 
> File attachment:
> "draft-ietf-tictoc-1588overmpls-04-org.txt" (52 KB) "draft-ietf-tictoc-1588overmpls-04-pm.txt" (53 KB)
> 
> 
> Regards,
> Peter
> 
> 
> -----Original Message-----
> From: Shahram Davari [mailto:davari@broadcom.com]
> Sent: June 4, 2013 7:40 PM
> To: Meyer, Peter; tictoc@ietf.org
> Subject: RE: [TICTOC] comments on draft-ietf-tictoc-1588overmpls-04.txt
> 
> Hi Peter,
> 
> Thanks for your god comments. Please see my responses inline. Please let me know whether you are satisfied with my responses since I am updating the draft in the next couple of days.
> 
> Thanks
> Shahram
> 
> -----Original Message-----
> From: tictoc-bounces@ietf.org [mailto:tictoc-bounces@ietf.org] On Behalf Of Meyer, Peter
> Sent: Monday, March 11, 2013 5:49 PM
> To: tictoc@ietf.org
> Subject: Re: [TICTOC] comments on draft-ietf-tictoc-1588overmpls-04.txt
> 
> Hi Shahram et al,
> 
> Some comments on draft -04.
> 
> 
> 1) Section 3.  "A generic method is defined in this document that does not require deep packet inspection at line rate, and can deterministically identify Timing messages.  The generic method is applicable to MPLS and MPLS-TP networks."
> 
> May want to add that this would apply only to one-step TC (I imagine that is the point of the correction at line rate).  A two-step TC would need to do deep packet inspection as it uses sourcePortIdentity & sequenceId fields for Follow_up or Delay_Resp.
> 
> SD> For 2-step, all that is needed is for an interface to detect the
> Timing message (based on top MPLS Label)  and record the Time along with some information from the Timing packet and send them to CPU while forwarding the Timing message as usual. Then the CPU based on the information received generates Follow-up message and injects it in to the LSP.  Therefore it is the job of CPU to look at sourcePortIdentity & sequenceId fields not the Hardware.
> 
> 
> 2) Section 4.  "An MPLS domain can serve multiple customers.  This means that the MPLS domain (maintained by a service provider) may provide timing services to multiple customers, each having their own Timing domain.  Therefore LER BCs need to interact with multiple grandmasters, and consequently multiple time references."
> 
> This should be re-phrased.  It switches from an optional situation ("may provide timing services") to mandatory situation ("LER BCs need to
> interact").   Some words such as "in such a deployment scenario, ...."
> and replace "can" with "may".   We have seen at ITU at least (with
> participation from operators BT, FT, DT, CMCC, AT&T, Sprint, etc.) that this multiple operator domain case was not useful enough to be included in standardization process for Telecom networks.
> 
> SD> I rephrased it as following:
> 
> " An MPLS domain can serve multiple customers.  This means that the
>   MPLS domain (maintained by a service provider) may provide timing
>   services to multiple customers, each having their own Timing domain.
>   Therefore in such deployment scenarios, LER BCs may interact with multiple grandmasters, and consequently multiple time references.  Also, LER/LSR TCs MUST be synchronized to the same master clock (which is the service provider Master clock), which implies that they need to choose one master to synchronize to."
> 
> 
> 3) Section 19.  "For transporting such peer delay measurement messages a single-hop LSP SHOULD to be created between the two adjacent LSRs engaged in peer delay measurement to carry peer delay measurement messages. Other methods such as PTP transport over Ethernet MAY be used for transporting peer delay measurement messages if the link between the two routers is Ethernet."
> 
> 
> This new statement to handle peer-delay (which in earlier drafts did not have a communication path listed), also allows a BC to be embedded in an LSR with a communication path between BC's as a single-hop LSP (mentioned March 22, 2012 in my feedback to draft -03, subject "[TICTOC] Updated 1588 over MPLS draf-03").
> 
> SD> When a single-hop LSP is created then I call the both ends of that
> LSP, LERs and not LSRs.
> Note that a router can act as LSR for some LSPs and LER for other LSPs.
> The point is that we can't
> Terminate a packet in the middle of an LSP (i.e., in an LSR). So BC functionality can't exist in the middle Of a PTP LSP. So I think it is a matter of terminology otherwise I agree that you could have BC in the core (not edge). 
> 
> Architecture diagrams Figure 1 and Figure should be updated to reflect
> the possibility of either BC or TC implementation.   Section 18.2 and
> section 18.3 and section 21 should also be updated to reflect the possibility of either BC or TC.  I understand the RFC is intended to be generic and not targeted only at TC.
> 
> 
> 
> 4) Section 20.  "When the MPLS network (provider network) serves multiple customers, it is important to maintain and process each customers clock and Timing messages separately from other customers to
> ensure there is no cross- customer effect.   For example if an LER BC is
> synchronized to a specific grandmaster, belonging to customer A, then the LER MUST use that BC clock only for customer A to ensure that customer A cannot attack other customers by manipulating its time."
> 
> This seems much more applicable to the TC LSR and should be stated.
> From section 4 we see the TC uses the primary synchronization domain (that of the service provide) to correct PTP messages.
> 
> SD> I don't see how in a TC LSR, one customer timing can affect other
> customer timings?
> Since in a TC, we just update the Timestamp of a packet based on the Service provider Time.
> May be I have not understood your point. Could you please clarify with an example.
> 
> 
> 
> 5) Section 20.  "Timing messages (as opposed to regular customer data) SHOULD not be encrypted or authenticated on an end-to-end basis."  I think there is a security draft in parallel being developed that may be relevant to that statement.
> 
> SD> I have updated the  text as following:
> 
> " Timing messages MAY be encrypted or authenticated, provided that the LERs/LSRs that are Timing capable/aware can authenticate/ decrypt the timing messages."
> 
> 
> 
> 
> Regards,
> Peter
> 
> 
> -----Original Message-----
> From: tictoc-bounces@ietf.org [mailto:tictoc-bounces@ietf.org] On Behalf Of internet-drafts@ietf.org
> Sent: February 23, 2013 2:23 AM
> To: i-d-announce@ietf.org
> Cc: tictoc@ietf.org
> Subject: [TICTOC] I-D Action: draft-ietf-tictoc-1588overmpls-04.txt
> 
> 
> A New Internet-Draft is available from the on-line Internet-Drafts directories.
> This draft is a work item of the Timing over IP Connection and Transfer of Clock Working Group of the IETF.
> 
>    Title           : Transporting Timing messages over MPLS
> Networks
>    Author(s)       : Shahram Davari
>                          Amit Oren
>                          Manav Bhatia
>                          Peter Roberts
>                          Laurent Montini
>    Filename        : draft-ietf-tictoc-1588overmpls-04.txt
>    Pages           : 36
>    Date            : 2013-02-22
> 
> Abstract:
>   This document defines the method for transporting Timing messages
>   such as PTP and NTP over an MPLS network.  The method allows for the
>   easy identification of these PDUs at the port level to allow for port
>   level processing of these PDUs in both LERs and LSRs.
> 
>   The basic idea is to transport Timing messages inside dedicated MPLS
>   LSPs.  These LSPs only carry timing messages and possibly Control and
>   Management packets, but they do not carry customer traffic.
> 
>   Two methods for transporting Timing messages over MPLS are defined.
>   The first method is to transport Timing messages directly over the
>   dedicated MPLS LSP via UDP/IP encapsulation, which is suitable for
>   MPLS networks.  The second method is to transport Timing messages
>   inside a PW via Ethernet encapsulation, which is suitable for both
>   MPLS and MPLS-TP networks.
> 
> 
> The IETF datatracker status page for this draft is:
> https://datatracker.ietf.org/doc/draft-ietf-tictoc-1588overmpls
> 
> There's also a htmlized version available at:
> http://tools.ietf.org/html/draft-ietf-tictoc-1588overmpls-04
> 
> A diff from the previous version is available at:
> http://www.ietf.org/rfcdiff?url2=draft-ietf-tictoc-1588overmpls-04
> 
> 
> Internet-Drafts are also available by anonymous FTP at:
> ftp://ftp.ietf.org/internet-drafts/
> 
> _______________________________________________
> TICTOC mailing list
> TICTOC@ietf.org
> https://www.ietf.org/mailman/listinfo/tictoc
> 
> 
> _______________________________________________
> TICTOC mailing list
> TICTOC@ietf.org
> https://www.ietf.org/mailman/listinfo/tictoc
> 
> 
> 
> _______________________________________________
> TICTOC mailing list
> TICTOC@ietf.org
> https://www.ietf.org/mailman/listinfo/tictoc
>