> -----Original Message----- > From: Haiyang Zhang [mailto:haiya...@microsoft.com] > Sent: Wednesday, July 17, 2013 2:01 AM > To: da...@davemloft.net; net...@vger.kernel.org > Cc: Haiyang Zhang; KY Srinivasan; o...@aepfle.de; jasow...@redhat.com; linux- > ker...@vger.kernel.org; de...@linuxdriverproject.org > Subject: [PATCH net] hyperv: Fix the NETIF_F_SG flag setting in netvsc > > SG mode is not currently supported by netvsc, so remove this flag for now. > Otherwise, it will be unconditionally enabled by commit ec5f0615642 > "Kill link between CSUM and SG features" > Previously, the SG feature is disabled because CSUM is not set here. > > Signed-off-by: Haiyang Zhang <haiya...@microsoft.com> > Reviewed-by: K. Y. Srinivasan <k...@microsoft.com>
We would need to apply this patch to 3.9 and 3.10 as well that are currently broken. K. Y > --- > drivers/net/hyperv/netvsc_drv.c | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/net/hyperv/netvsc_drv.c b/drivers/net/hyperv/netvsc_drv.c > index 4dccead..23a0fff 100644 > --- a/drivers/net/hyperv/netvsc_drv.c > +++ b/drivers/net/hyperv/netvsc_drv.c > @@ -431,8 +431,8 @@ static int netvsc_probe(struct hv_device *dev, > net->netdev_ops = &device_ops; > > /* TODO: Add GSO and Checksum offload */ > - net->hw_features = NETIF_F_SG; > - net->features = NETIF_F_SG | NETIF_F_HW_VLAN_CTAG_TX; > + net->hw_features = 0; > + net->features = NETIF_F_HW_VLAN_CTAG_TX; > > SET_ETHTOOL_OPS(net, ðtool_ops); > SET_NETDEV_DEV(net, &dev->device); > -- > 1.7.4.1 > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
