On Wed, 2009-06-17 at 15:27 -0700, Ted Kim wrote: > B. Solution > > We propose to add the following read-only Infiniband-specific > properties to dladm that could be displayed via 'dladm show-ib'. > These properties are well known to each ibd driver instance and it > would be a simple matter for the driver to provide them to users via > 'dladm show-ib'. After discussing things with the dladm team, we are > advocating a media specific subcommand (show-ib) because these > read-only items are very specific to InfiniBand mapping of > IPonIB. Note that while broadcast is possibly on or off, the status > cannot be changed from the host (only from the IB Subnet Manager). > > A sample 'dladm show-ib' output for an ibd instance would be like below: > > LINK PROPERTY VALUE POSSIBLE > > ibd0 partition ffff -- > ibd0 broadcast on on,off > ibd0 guid 2c9030001c86c --
I don't have a problem with the properties themselves, but the output format seems rather strange. Presumably, the properties will show up in show-linkprop in a format similar (but not quite the same) as that as shown above, so it doesn't seem valuable to have this type of one-property per line format. Instead, I'd expect these properties to show up as follows in the show-linkprop output: LINK PROPERTY PERM VALUE DEFAULT POSSIBLE ibd0 partition r- ffff -- -- ibd0 broadcast r- on on on,off ibd0 guid r- 2c9030001c86c -- -- ibd1 partition r- ... Then, to make things more legible for IB-specific output (analogous to the show-ether Ethernet-specific output), something like this in show-ib: LINK PARTITION BROADCAST GUID ibd0 ffff on 2c9030001c86c ibd1 abcd off 2134123412341 ibd2 ... That way, we don't have a subcommand that is almost totally without value beyond what show-linkprop shows, and one can get a nice IB overview without too much clutter with show-ib. -Seb
