Re: [Gen-art] Gen-ART Review of draft-ietf-forces-interfelfb-04

[Jamal Hadi Salim <[e-mail]>]

Hi Russ,
You review is appreciated. Comments/responses below.

On Tue, May 31, 2016 at 2:54 PM, Russ Housley <[e-mail]> wrote:

>
> Major Concerns:
>
> In Section 5.2, the document says:
>
>    o  The Ethernet type is used to identify the frame as inter-FE LFB
>       type.  Ethertype 0xFEFE is to be used (XXX: Note to editor, likely
>       we wont get that value - update when available).
>
> There is a process for getting an ethertype assigned.  It is handled by
> the IESG.  Please see: http://www.ietf.org/iesg/statement/ethertypes.html
>
>
>

Indeed we are following that process. New (unpublished) text refers
to the ethertype as TBA1.


> Minor Concerns:
>
> In Section 1.2: s/document reiterates the terminology defined/
>                  /document depends on the terminology defined/
>
>
Fixed.


> In Section 2: s/The current ForCES model/The ForCES model/
>
>
Fixed.


>
> Nits:
>
> Many places: s/IPV4/IPv4/
>
>
Fixed.


> In Section 2: s/architecture calls out for/architecture calls for/
>
>
Fixed.


> In Section 3.1, why are the paragraphs in bullets?

They see to work
> just fine as normal paragraphs.
>
>
The assumptions are intended to stand out. I am struggling with
how to achieve that without enumerating them as in the doc.
Perhaps getting rid of that section and making it flow into the main
introductory text would be better? i.e something like:
"We make two fundamental assumptions ...:"
Then the two bullets follow.

In Section 5.2: s/header illustrated in Figure 6)/
>                  /header (illustrated in Figure 6)
>

Fixed.

Thanks again Russ.

cheers,
jamal