<div dir="ltr"><div class="gmail_extra">On Tue, Nov 12, 2013 at 4:06 PM, Jan Engelhardt <span dir="ltr"><<a href="mailto:jengelh@inai.de" target="_blank">jengelh@inai.de</a>></span> wrote:<br><div class="gmail_quote">

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":1z6" style="overflow:hidden">In any case, even if the ethtool structures awkwardly specified the use<br>
of "bytes", the user-visible part should be in "bits", and this patch<br>
was to give an impetus.</div></blockquote></div><br>I won't commit the patch unless there's clarity that setting the value in megabits works properly. I don't believe in breaking things to motivate progress. :-)</div>

<div class="gmail_extra"><br></div><div class="gmail_extra">I'm fine with us providing the setting as Mbps, but it's unclear whether that will get passed to the kernel as MBps.</div></div>