thanks for the comments. > A level-0 question first: what's changed? > > It's usually helpful to provide some sort of overview of what you've > changed since the last ARC review. Two common ways of doing that are > to take the issues file and interleave the questions with the answers,
I've provided an issues.txt with clarifications, describing the sections that have been updated. > > Remaining comments: > > - How are adv_*_cap Volatile? What will change here? (Does this > actually mean Obsolete or something else?) We chose the default option of making all Brussels interfaces Volatile until we got feedback on how user-friendly they actually were. In the case of the adv*cap options, the name of the parameter itself is somewhat clumsy, being chosen only because of its existing usage in ndd. The objective is that if we find that users prefer other syntax, we could mark this Obsolete in future. > - Loopback ioctls? not sure I follow the question. Were you asking what steps would be taken to address the loopback ioctls issue raised at inception? We looked further into this issue, and found that SunVTS ioctls are not really "properties" that control the behavior of the interface, like other examples considered, but are part of loopback testing interfaces that are documented in netlb.h as "may be supported". We've filed an RFE: 6613193 loopback ioctls should be implemented as Brussels properties. to track this (currently under brussels:software/driver but will be converted to solaris with putback) > - Will use of ndd for drivers eventually go away? Is that part of > the plan or will we carry compatibility forever? yes! Absolutely! this will be covered by the ndd compatibility component of Brussels. See Section 5.2 of http://cr.opensolaris.org/~sowmini/commitment.materials/brussels.pdf > - Nit: strncmp on page 20 of brussles.pdf is quite odd. I don't > think this does what you're expecting it does. (Notably, strcmp > would be just as safe, because "_drain_max" is a fixed-length > string with a terminating NUL byte.) Accept, fixed. --Sowmini
