[ofa-general] Re: [PATCH v2 for-2.6.27] IPOIB: add LRO support.

Roland Dreier rdreier at cisco.com
Tue Jun 24 13:56:10 PDT 2008


To be honest this looks like a step backwards...

 > +static ssize_t show_use_lro(struct device *d, struct device_attribute *attr,
 > +			 char *buf)
 > +{
 > +	struct ipoib_dev_priv *priv = netdev_priv(to_net_dev(d));
 > +
 > +	if (priv->lro.use_lro)
 > +		return sprintf(buf, "yes\n");
 > +	else
 > +		return sprintf(buf, "no\n");
 > +}

This looks like a crazy interface ("yes"/"no" vs 1/0??) and in any case
I think using the existing ethtool support to control LRO would be much
simpler and better for users.

 - R.





More information about the general mailing list