[ippm] Suresh Krishnan's Discuss on draft-ietf-ippm-6man-pdm-option-09: (with DISCUSS and COMMENT)

[Suresh Krishnan <[e-mail]>]

Suresh Krishnan has entered the following ballot position for
draft-ietf-ippm-6man-pdm-option-09: Discuss

When responding, please keep the subject line intact and reply to all
email addresses included in the To and CC lines. (Feel free to cut this
introductory paragraph, however.)


Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html
for more information about IESG DISCUSS and COMMENT positions.


The document, along with other ballot positions, can be found here:
https://datatracker.ietf.org/doc/draft-ietf-ippm-6man-pdm-option/



----------------------------------------------------------------------
DISCUSS:
----------------------------------------------------------------------

* Section 3.2.1.
  The option length seems to be wrong here. This will make the parser
parse incorrectly onto a following option or worse. I think this MUST be
set to 10 instead of 16 (Or some field is missing from the description of
the option)

   8-bit unsigned integer. Length of the option, in octets, excluding
   the Option Type and Option Length fields. This field MUST be set to
   16.

* Section 3.2.1.
  The option does not seem to state an alignment requirement, but I think
one is required to properly align the multi-byte PSN and Delta fields.
Can you please specify one.

* Section 5

The IANA considerations section needs to be more specific as you are
requesting a specific type of option.

e.g. This draft requests an Destination Option Type assignment with
   the act bits set to 00 and the chg bit set to 0 from the ...


----------------------------------------------------------------------
COMMENT:
----------------------------------------------------------------------

* Section 1.6

I am not sure where this inference comes from. Can you please clarify?

It is likely that an IPv6 packet containing PDM will be dropped if using
IPv6 transition technologies.

* Section 3.1

This text is not correct as the PDM option is not an implementation of
the Destination options *header*.

The IPv6 Performance and Diagnostic Metrics Destination Option (PDM) is
an implementation of the Destination Options Header.

Suggest rewording to 

The IPv6 Performance and Diagnostic Metrics Destination Option (PDM) is
implemented as an IPv6 Option carried in the Destination Options Header.