Re: [pim] draft-ietf-pim-igmp-mld-extension WGLC

Hi,

> On Dec 16, 2020, at 11:21, Jeffrey (Zhaohui) Zhang <zzhang=40juniper.net@dmarc.ietf.org> wrote:
> 
> Support progressing this work.
>  
> I have a couple of curiosity questions on the following:
>  
>    Note that there may be additional data following this extension.  The
>    Total Extension Length field would indicate where this extension
>    ends, and the additional data starts.
>  
> How would you have additional data following this extension? The TLV mechanism seems to be universal/generic to handle all possible extensions, so why would you have another extension blob after this, and how would you indicate so? By another bit in the reserved field?
> Just wondering if this additional data needs to be mentioned at all.

I think this is a good question.
IMO, the text, "Note that there may be additional data following this extension.", should be replaced to "Note that additional TLV or some specific fields may be nested (i.e., enclosed) in the Extension Value field in the TLV field defined in this document."

>   Also, there is a possibility
>    that an implementation uses the Additional Data part of IGMP/MLD
>    messages, but not according to this extension scheme.
>  
> Do you mean a non-standard existing implementation? Now that you mention that, are we supposed to define the interop procedures? Are they supposed to interop with the implementations following this draft at all?

I also think this text is strange.
Can we simply remove this text?

Regards,

Hitoshi

> Also, for the following:
>  
>    The extension will be part of additional data as mentioned in
>    [RFC3810] Section 5.1.12 (resp.  [RFC3376] Section 4.1.10) for query
>    messages and [RFC3810] Section 5.2.12 (resp.  [RFC3376]
>    Section 4.2.11) for report messages.
>  
> 5.2.12 should be 5.2.11.
>  
> Jeffrey
>  
> From: pim <pim-bounces@ietf.org> On Behalf Of Michael McBride
> Sent: Monday, December 7, 2020 7:24 PM
> To: pim@ietf.org
> Subject: [pim] draft-ietf-pim-igmp-mld-extension WGLC
>  
> [External Email. Be cautious of content]
>  
> Hello all,
>  
> Today begins a two week wglc for https://tools.ietf.org/html/draft-ietf-pim-igmp-mld-extension-02
>  
> Please review (it’s a quick read) and let us know your opinion on it’s readiness to progress to the iesg.
>  
> Thanks,
> mike
> 
> Juniper Business Use Only
> 
> _______________________________________________
> pim mailing list
> pim@ietf.org
> https://www.ietf.org/mailman/listinfo/pim