> > For DLD_PROP_SPEED, I thought we agreed that speed would be in "bps" not > > "Mbps". (linkprop.c:201) (Again, recall that 802.11b can have a speed of > > 5.5Mbps.) > > correct, that's a typo. on that line that I will fix. BTW, the kstat > itself is printed in dladm after being scaled down to Mbps for commands > like 'dladm show-phys', 'dladm show-dev' etc. so the loss of precision > is more wide-spread than just this one command (and I believe that > should be addressed as a separate bug, so I'm not going to pull that > into this fix).
Just to be sure I understand here: our plan is to have dladm display e.g. 5.5mb, right? (It'd also be nice if it displayed 1gb instead of 1000mb, but I suppose that's a separate matter.) -- meem
