Meem, Thanks for the comments.
On Jul 31, 2007, at 11:09 PM, Peter Memishian wrote: > * I notice that Crossbow plans to extend show-dev. Post-UV, > show-dev > still deals with device names, whereas the other show-<foo> > commands > deal with link names. Accordingly, we've added a show-phys > subcommand that describes "physical links" by their linkname, and > obsoleted show-dev (see section 4.1.2 and section 4.1.3. of > [1] for > more details). Would it be reasonable for this new > functionality to > be folded into show-phys? If the option is specific to devices I agree. If there's a chance that this information might be applicable to other types of links, then we might want to expose them through show-link. For example, we might down the road expose expose MAC address slots and factory MAC address from aggregations. In this case, should we have an option that shows that information as part of show-phys and show-aggr, or should we show that information via show-link since it might apply to different types of links. It's best if we can find an approach which is consistent with Clearview UV. > > * Similarly, as we've discussed in the past, a number of the new > dladm > "-d <dev>" options need to become "-l <link>" post-UV. Agreed. We've kept the man page in sync with ONNV since that's what we're based on today. As soon as UV and Crossbow merge, then we'll update the option names accordingly to be consistent across the board. > * In general, field names in dladm show-<foo> output should be a > single > word. (Among other things, this sidesteps having to quote > whitespace > when using the -o <field> option that we'd like all the show- > <foo> > commands to have.) Accordingly, the "MAC ADDRESS" field > needs to be > recast. OK, I don't see a problem with that. Thanks, Nicolas. -- Nicolas Droux - Solaris Networking - Sun Microsystems, Inc. droux at sun.com - http://blogs.sun.com/droux
