Re: [TICTOC] WGLC on draft-ietf-tictoc-ptp-mib-05.txt

Greg Dowd <gdowd@symmetricom.com> Mon, 10 June 2013 18:00 UTC

Return-Path: <btv1==87331abdaaf==gdowd@symmetricom.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 0A4C021F9968 for <tictoc@ietfa.amsl.com>; Mon, 10 Jun 2013 11:00:13 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.264
X-Spam-Level:
X-Spam-Status: No, score=-2.264 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, HTML_MESSAGE=0.001, IP_NOT_FRIENDLY=0.334]
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 JbVE1MHtR1Cn for <tictoc@ietfa.amsl.com>; Mon, 10 Jun 2013 11:00:08 -0700 (PDT)
Received: from mail5.symmetricom.com (mail5.symmetricom.com [69.25.98.10]) by ietfa.amsl.com (Postfix) with ESMTP id 747CF21F9925 for <tictoc@ietf.org>; Mon, 10 Jun 2013 11:00:07 -0700 (PDT)
X-ASG-Debug-ID: 1370887206-05bdbc0700581f0001-4wH9i1
Received: from SJC-HUB-01.symmetricom.com ([192.168.10.171]) by mail5.symmetricom.com with ESMTP id tweeqUJCY80twH6a (version=TLSv1 cipher=AES128-SHA bits=128 verify=NO); Mon, 10 Jun 2013 11:00:06 -0700 (PDT)
X-Barracuda-Envelope-From: gdowd@symmetricom.com
X-ASG-Whitelist: Client
Received: from SJC-MBX-01.symmetricom.com ([169.254.1.186]) by SJC-HUB-01.symmetricom.com ([::1]) with mapi id 14.01.0289.001; Mon, 10 Jun 2013 11:00:05 -0700
From: Greg Dowd <gdowd@symmetricom.com>
To: Kevin Gross <kevin.gross@avanw.com>, Tim Frost <tfrost@symmetricom.com>
Thread-Topic: [TICTOC] WGLC on draft-ietf-tictoc-ptp-mib-05.txt
X-ASG-Orig-Subj: RE: [TICTOC] WGLC on draft-ietf-tictoc-ptp-mib-05.txt
Thread-Index: AQHOUkve8l3OF4TVM0S7AgdJPsfOkpkIgNuAgAECRwCAJkhkAP//l7YQ
Date: Mon, 10 Jun 2013 18:00:05 +0000
Message-ID: <26FBD265CD8D234EAF41115A180F92BB6ECD10CB@SJC-MBX-01.symmetricom.com>
References: <5194FE2F.2060504@isoc.org> <CALw1_Q3DxKFc+oNFj_0iyDPp4m59hNS3--7R+nX_i1OzOaeT-g@mail.gmail.com> <4A4385BFBA1BC54C863E7394BD1AECB36EB1C82C@SJC-MBX-01.symmetricom.com> <CALw1_Q1D+iCamS4j1C=GEvsehJ1wYTbFz9y34m12nurW7RgfNQ@mail.gmail.com>
In-Reply-To: <CALw1_Q1D+iCamS4j1C=GEvsehJ1wYTbFz9y34m12nurW7RgfNQ@mail.gmail.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [192.168.3.9]
Content-Type: multipart/alternative; boundary="_000_26FBD265CD8D234EAF41115A180F92BB6ECD10CBSJCMBX01symmetr_"
MIME-Version: 1.0
X-Barracuda-Connect: UNKNOWN[192.168.10.171]
X-Barracuda-Start-Time: 1370887206
X-Barracuda-Encrypted: AES128-SHA
X-Barracuda-URL: http://192.168.10.96:80/cgi-mod/mark.cgi
X-Barracuda-BRTS-Status: 1
X-Virus-Scanned: by bsmtpd at symmetricom.com
Cc: "tictoc@ietf.org" <tictoc@ietf.org>, Karen ODonoghue <odonoghue@isoc.org>
Subject: Re: [TICTOC] WGLC on draft-ietf-tictoc-ptp-mib-05.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: Mon, 10 Jun 2013 18:00:47 -0000

I think the confusion was that your comment suggested using an OUI, not a profile OID (EUI-48).  The OUI would not have been sufficient to distinguish multiple profiles from an organization as Tim noted.  I'll review your comments with the team.  Thanks!

Greg Dowd | Symmetricom(r), Inc.
Staff Scientist
2300 Orchard Parkway, San Jose, CA 95131
Direct:  408.964.7643
gdowd@symmetricom.com<mailto:gdowd@symmetricom.com>  |  www.symmetricom.com<http://www.symmetricom.com/>
Symmetricom.  Leading the world in precise time solutions.

From: tictoc-bounces@ietf.org [mailto:tictoc-bounces@ietf.org] On Behalf Of Kevin Gross
Sent: Monday, June 10, 2013 10:10 AM
To: Tim Frost
Cc: tictoc@ietf.org; Karen ODonoghue
Subject: Re: [TICTOC] WGLC on draft-ietf-tictoc-ptp-mib-05.txt

Thanks for addressing my comments. It looks like there are two outstanding.

  1.  ClockInstanceType and
  2.  ClockProfileType
It sounds like you intend to address (1) in a future revision.

With regards to (2), I think you misunderstand how the profileIdentifier works. Please have a look at Section 19.3.3 of IEEE 1588-2008. This is an EUI-48, not merely an OUI. Organizations can change the bottom 24 bits to indicate new profiles or revisions of existing ones. Maintaining our own enumeration of profiles does not seem like a feasible undertaking.

Kevin Gross
+1-303-447-0517
Media Network Consultant
AVA Networks - www.AVAnw.com<http://www.avanw.com/>, www.X192.org<http://www.X192.org>

On Fri, May 17, 2013 at 2:33 AM, Tim Frost <tfrost@symmetricom.com<mailto:tfrost@symmetricom.com>> wrote:
Hi Kevin,

Thank you for taking the time to go through this MIB. We did consider your comments, and they were addressed in -05 (the latest version, and the subject of the last call).  I've gone through each of your comments below; it looks like we missed one, and decided against another, but the remainder have been done.

The change history from the -05 draft reads:

-05       Feb 13 Several changes in response to comments from Alun Luchuk
             and Kevin Gross:
             - Modified the use of wellKnownTransportTypes and
               wellKnownEncapsulationTypes
             - changed ptpbaseClockPortSyncOneStep to
               ptpbaseClockPortSyncTwoStep to match IEEE1588
               semantics
             - Re-ordered textual conventions to be alphabetic
             - Changed some types from Integer32 to use defined
               textual conventions
             - various minor descriptive text changes

In detail:
Abstract: Add a bracketed reference to IEEE 1588-2008 - DONE

Section 1: s/defined to monitor, measure the performance/defined to monitor and measure the performance - DONE

Section 1.1 What in this MIB is profile dependent? I did not identify anything in my review. It would be desirable. if possible,  to have a profile-independent MIB.

-    TF: Removed erroneous text about Telecom Profile - there is nothing in this MIB that is particular to that profile.

TEXTUAL-CONVENTION general issues - Consider alphabetizing - DONE

ClockInstanceType TEXTUAL-CONVENTION definition - Description requires elucidation or a reference.

-    TF: not done, looks like this one slipped through.

ClockProfileType TEXTUAL-CONVENTION definition
Consider using and OUI instead of an enumeration. IEEE 1588-1588 requires an OUI be associated with each profile.

-    TF: OUI would be specific to the organization, not the profile. If the ITU (for instance) developed more than one profile, then they would have the same OUI. Therefore it is best (in the authors' opinion) to stick to using an enumeration

OBJECT-TYPE general issues
Consider being consistent about repeating TEXTUAL-CONVENTION descriptions, explicitly referencing them or assuming readers will track down the reference without reminder.

-    TF: We've tried to be as consistent as we can. Of 17 TEXTUAL-CONVENTIONS, 13 are directly referenced to IEEE1588 sections. The remainder don't directly map onto IEEE1588 quantities, but onto associated concepts, and we have tried to explain these in the description. Certainly one could do with a better description, as you note above.

ptpDomainClockPortsTotal OBJECT-TYPE
I assume this is the number of ports for the managed PTP-capable system (i.e. a router or switch). Suggest adding "in the system" to the description to clarify scope of count. - DONE


ptpbaseClockTimePropertiesDSTable OBJECT-TYPE

s/clock Timeproperties Datasets for/clock time properties datasets for - DONE



PtpbaseClockTransDefaultDSEntry SEQUENCE

Use ClockDomainType textual convention instead of Integer32 for ptpbaseClockTransDefaultDSPrimaryDomain - DONE



PtpbaseClockPortEntry SEQUENCE

Change ptpbaseClockPortSyncOneStep to ptpbaseClockPortSyncTwoStep to match 1588 semantics for this Boolean - DONE



ptpbaseClockPortRunningEncapsulationType OBJECT-TYPE

Needs a textual convention or description of allowed values and their meanings

-    TF: Now uses an autonomous type instead of Integer32



ptpbaseClockPortTransDSlogMinPdelayReqInt OBJECT-TYPE

Change to ClockIntervalBase2 type DONE



Section 5: Copy-paste error: "creation and/or manipulation of tunnels"

-    TF: Section changed since -03, no longer has this phrase.



Section 6: To be added - DONE


Best regards,
Tim Frost

From: tictoc-bounces@ietf.org<mailto:tictoc-bounces@ietf.org> [mailto:tictoc-bounces@ietf.org<mailto:tictoc-bounces@ietf.org>] On Behalf Of Kevin Gross
Sent: 16 May 2013 18:09

To: Karen ODonoghue
Cc: tictoc@ietf.org<mailto:tictoc@ietf.org>
Subject: Re: [TICTOC] WGLC on draft-ietf-tictoc-ptp-mib-05.txt

I submitted comments on v03 11 January (http://www.ietf.org/mail-archive/web/tictoc/current/msg01331.html). Vinay indicated in a short message on 23 January the comments would be addressed in the next version (http://www.ietf.org/mail-archive/web/tictoc/current/msg01333.html). v04 was published 31 January. I did not see any notes from the authors associated with that revision. A quick check of the v03-v04 diff shows evidence that not all my comments were directly addressed. Can I get a summary of how my comments were addressed? I'm fine if some of these were rejected but I would like to verify that they were given consideration.

Kevin Gross
+1-303-447-0517<tel:%2B1-303-447-0517>
Media Network Consultant
AVA Networks - www.AVAnw.com<http://www.avanw.com/>, www.X192.org<http://www.X192.org>

On Thu, May 16, 2013 at 9:41 AM, Karen O'Donoghue <odonoghue@isoc.org<mailto:odonoghue@isoc.org>> wrote:
This message initiates a two week TICTOC Working Group Last Call on advancing:

Title : Precision Time Protocol Version 2 (PTPv2) Management Information Base
Author(s) : Vinay Shankarkumar, Laurent Montini, Tim Frost, Greg Dowd
Filename : draft-ietf-tictoc-ptp-mib-05.txt
Pages : 77
Date : 2013-02-25

http://datatracker.ietf.org/doc/draft-ietf-tictoc-ptp-mib/?include_text=1

as a Standards Track RFC. Substantive comments and statements of support for advancing this document should be directed to the mailing list. Editorial suggestions can be sent directly to the authors. This last call will end on 31 May 2013.

_______________________________________________
TICTOC mailing list
TICTOC@ietf.org<mailto:TICTOC@ietf.org>
https://www.ietf.org/mailman/listinfo/tictoc