Re: [PATCH net] hyperv: Fix the NETIF_F_SG flag setting in netvsc
From: KY Srinivasan Date: Wed, 17 Jul 2013 11:40:41 + > > >> -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 >> Reviewed-by: K. Y. Srinivasan > > We would need to apply this patch to 3.9 and 3.10 as well that are currently > broken. I have already queued this patch up for -stable submission. -- 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/
RE: [PATCH net] hyperv: Fix the NETIF_F_SG flag setting in netvsc
> -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 > Reviewed-by: K. Y. Srinivasan 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 = _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, _ops); > SET_NETDEV_DEV(net, >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/
Re: [PATCH net] hyperv: Fix the NETIF_F_SG flag setting in netvsc
From: Haiyang Zhang Date: Tue, 16 Jul 2013 23:01:20 -0700 > 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 > Reviewed-by: K. Y. Srinivasan Applied and queued up for -stable, thanks. -- 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/
Re: [PATCH net] hyperv: Fix the NETIF_F_SG flag setting in netvsc
From: Haiyang Zhang haiya...@microsoft.com Date: Tue, 16 Jul 2013 23:01:20 -0700 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 Applied and queued up for -stable, thanks. -- 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/
RE: [PATCH net] hyperv: Fix the NETIF_F_SG flag setting in netvsc
-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, ethtool_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/
Re: [PATCH net] hyperv: Fix the NETIF_F_SG flag setting in netvsc
From: KY Srinivasan k...@microsoft.com Date: Wed, 17 Jul 2013 11:40:41 + -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. I have already queued this patch up for -stable submission. -- 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/