[MIB-DOCTORS] draft-ietf-ccamp-gmpls-alarm-spec-04.txt [was: PRELIMINARY Agenda and Package for August 31, 2006 Telechat ]

Nothing fatally flawed I think.

But just some comments.

>   o draft-ietf-ccamp-gmpls-alarm-spec-04.txt
>     GMPLS - Communication of Alarm Information (Proposed Standard)
>      - 2 of 8 
>     Token: Ross Callon

On page 7, I see:

     Severity: 8 bits

        Indicates the impact of the alarm indicated in the TLV.  See
        [RFC3877] and [M.3100] for more information on severity.  The
        following values are defined:

         Value       Definition
         -----       ----------
           0         Cleared
           1         Indeterminate
           2         Critical
           3         Major
           4         Minor
           5         Warning

And so I wonder if they mean bit values, or if they mean an 8-bit 
unsigned integer? I would assume (hope) the latter.
I also wonder why they do not use the same values as you do in RFC3877,
that is start at 1 and keep zero reserved (for now).

On page 9

     Error String: 32 bits minimum (variable)

        A string of characters in US-ASCII, representing the type of
        error/alarm.  This string is padded to the next largest 4 byte
        boundary using null characters.  Null padding is not required
        when the string is 32-bit aligned.  The contents of error
        string are implementation dependent.  See the condition types
        listed in Appendices of [GR833] for a list of example strings.
        Note length includes padding.

Strange they express this in BITs as opposed to OCTETS.
And I am also wondering of this string is ever meant for human
consumption. If so, then maybe it better be UTF-8 ??

page 14:

    .Pa "Acknowledgments"

   Valuable comments and input were received from a number of people,
   including Wes Doonan, Bert for the DISMAN reference, Tom Petch for
   getting the disman WG interactions started.  We also thank David
   Black for his valuable comments.

.Pa ??
nroff page eject maybe?

- some reference/citation problems:

!! Missing Reference for citation: [BCP26]
  P015 L022:    an IANA Considerations Section in RFCs" [BCP26].

Bert

_______________________________________________
MIB-DOCTORS mailing list
MIB-DOCTORS@ietf.org
https://www1.ietf.org/mailman/listinfo/mib-doctors