<br><br><div class="gmail_quote">On Tue, Jun 24, 2008 at 11:56 PM, Roland Dreier <<a href="mailto:rdreier@cisco.com">rdreier@cisco.com</a>> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
To be honest this looks like a step backwards...<br>
<div class="Ih2E3d"><br>
 > +static ssize_t show_use_lro(struct device *d, struct device_attribute *attr,<br>
 > +                     char *buf)<br>
 > +{<br>
 > +    struct ipoib_dev_priv *priv = netdev_priv(to_net_dev(d));<br>
 > +<br>
 > +    if (priv->lro.use_lro)<br>
 > +            return sprintf(buf, "yes\n");<br>
 > +    else<br>
 > +            return sprintf(buf, "no\n");<br>
 > +}<br>
<br>
</div>This looks like a crazy interface ("yes"/"no" vs 1/0??) and in any case<br>
I think using the existing ethtool support to control LRO would be much<br>
simpler and better for users.<br>
<font color="#888888"><br>
 - R.<br>
</font><div><div></div><div class="Wj3C7c"><br>
</div></div></blockquote><div><br>I can change "yes" to "1" and "no" to "0".<br>I don't see that there is LRO support in the userspace ethtool.<br><br><br>Regards,<br>VLadimir<br>
</div></div><br>