Re: draft-ietf-mboned-ieee802-mcast-problems-06 review
Charlie Perkins <[email protected]> Fri, 19 Jul 2019 07:56:24 -0700
| Newsgroups | gmane.ietf.mboned |
|---|---|
| Message-ID | <[email protected]> |
Hello Jake, Thanks for the additional review. I will prepare a new revision of the document for submission on Monday. By the way, due to a last-minute change of plans, I will be unable to attend this IETF in person. I'll try to attend remotely and prepare some presentation materials if needed. Regards, Charlie P. On 7/18/2019 5:09 PM, Holland, Jake wrote: > Hi guys, > > I was working on the document shepherd write-up and noticed another > few minor issues in the doc that I think will need fixing: > > > New technical: > 1. > I noticed an addition in -06 in the AMT section that I recommend > changing: > > <OLD> > It is RECOMMENDED that multicast-enabled networks deploying AMT > relays for this purpose make the relays discoverable with the > following methods: > > o DNS-SD [RFC6763] > o the well-known IP addresses from Section 7 of [RFC7450], and > o DRIAD [I-D.ietf-mboned-driad-amt-discovery] > </OLD> > > I would remove the DRIAD line item, since this is advice to the > local Wi-Fi network operator: > <NEW> > It is RECOMMENDED that multicast-enabled networks deploying AMT > relays for this purpose make the relays locally discoverable with > the following methods, as described in [I-D.ietf-mboned-driad-amt-discovery]: > > o DNS-SD [RFC6763] > o the well-known IP addresses from Section 7 of [RFC7450] > </NEW> > > Sorry, I hadn't caught this suggestion when it was raised, but in > the context of this section, I don't think DRIAD belongs in the > list, because this is advice for the local (to the receiver) > network operator, but DRIAD discovery is something a remote sender > would deploy. > > This is because the new method DRIAD defines is for looking up an > AMT relay known to the sender, so it's only applicable to the > one sending traffic. (And if the Wi-Fi network operator is also > operating the source of traffic, it's either within the same > multicast-capable local or local-ish network, in which case the > DRIAD lookup isn't needed, or the sender is in a separate network, > in which case DRIAD could be useful, but it's not something > provided by the Wi-Fi portion of the network.) > > > New editorial (minor): > 2. > The first few references are separated by commas without exposition, > where usually they'd be without commas if they're intended as a > reference for a claim, but in fact I think 2 of these references are > outlining the well-known issues, and one is the description of the > 802.11 networks (new in -05, I think): > <OLD> > Well-known issues with multicast have prevented the deployment of > multicast in 802.11 [dot11], [mc-props], [mc-prob-stmt], and other > local-area wireless environments. Performance issues have been > </OLD> > > I think it might make more sense as: > <NEW> > Well-known issues with multicast have prevented the deployment of > multicast in 802.11 [dot11], and other local-area wireless > environments, as outlined in [mc-props] and [mc-prob-stmt]. > </NEW> > > (Or maybe "as outlined", might be better "as documented" or "as > described"? This bit of wordsmithing I don't care about, just > trying to fix the weird comma-separated reference list...) > > > References: > I've also checked the references a little more thoroughly, and > noticed a few issues I didn't see before: > > > 3. > The [arpsponge] reference link doesn't work: > https://ams-ix.net/downloads/arpsponge/3.12.2/arpsponge-3.12.2/arpsponge.txt > > I was thinking it might make sense to reference the original paper instead: > "Effects of IPv4 and IPv6 address resolution on AMS-IX and the ARP Sponge" > by Marco Wessel and Niels Sijm > http://citeseerx.ist.psu.edu/viewdoc/summary?doi=10.1.1.182.4692 > > Or perhaps the github repo: > https://github.com/AMS-IX/arpsponge > > 3.a. > (Editorial: in the reference to this, it might make sense to call it "one of the > most widely deployed arp sponges" instead of "the standard one".) > > 4. > [dot11] reference has a broken link: > https://standards.ieee.org/getieee802/download/802.11-2016.pdf > > I get a 404 on this. The RFC Style guide uses a different > formulation in one of the IEEE examples, which worked for me: > http://standards.ieee.org/findstds/standard/802.11-2016.html > > (from https://tools.ietf.org/html/rfc7322#section-4.8.6.6 ) > > 4.a. > Also: in the draft's html page the link is not clickable, and it > seems to be because the url contains extra text inside the > <> instead of outside of it (the "(includes 802.11v amendment)" bit) > > Medium Access Control (MAC) and Physical Layer (PHY) > Specification", March 2016, > <http://standards.ieee.org/getieee802/ > download/802.11-2016.pdf (includes 802.11v amendment)>. > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > > 5. > [dot11aa] same broken link style as dot11 (#4) > > > _______________________________________________ MBONED mailing list [email protected] https://www.ietf.org/mailman/listinfo/mboned