Attachment 279791 Details for Bug 414131 – backported version of 157:36d020268c6c

[patch] backported version of 157:36d020268c6c

xen-unstable-12208-36d020268c6c (text/plain), 1.76 KB, created by Ian Campbell on 2007-12-06 15:33:54 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Ian Campbell

Created: 2007-12-06 15:33:54 UTC

Size: 1.76 KB

patch

obsolete

># HG changeset patch
># User Ian Campbell <ian.campbell@xensource.com>
># Node ID 36d020268c6c099c7e70c0070ee76f07dedd547d
># Parent  f3fbe9549f5f146ace8b93dbe16043bf4d71938e
>PV-on-HVM: Conditionally disable checksum offload for receive on some
>older kernels.
>
>On some older kernels the CHECKSUM_UNNECESSARY flag on received
>skbuffs is not honoured by all subsystems, specifically netfilter
>under 2.6.9-42.EL
>
>To work around this we need to ensure that the correct checksum is
>calculated at some point. We can cause the backend to do this for us
>by advertising to the backend that we cannot support checksum offload.
>
>The correlation of this issue with the presence of NETIF_F_TSO is not
>100% but it is a good first approximation.
>
>Signed-off-by: Ian Campbell <ian.campbell@xensource.com>
>xen-unstable changeset: 12208:36d020268c6c099c7e70c0070ee76f07dedd547d
>xen-unstable date: Thu Nov 02 11:56:26 2006 +0000
>
>Index: linux-2.6.9-67.EL/drivers/xen/netfront/netfront.c
>===================================================================
>--- linux-2.6.9-67.EL.orig/drivers/xen/netfront/netfront.c	2007-12-06 15:14:24.000000000 +0000
>+++ linux-2.6.9-67.EL/drivers/xen/netfront/netfront.c	2007-12-06 15:15:02.000000000 +0000
>@@ -96,6 +96,7 @@
> 	dev->features |= NETIF_F_GSO_ROBUST;
> }
> #else
>+#define HAVE_NO_CSUM_OFFLOAD           1
> #define netif_needs_gso(dev, skb)	0
> #define dev_disable_gso_features(dev)	((void)0)
> #endif
>@@ -367,6 +368,14 @@
> 		goto abort_transaction;
> 	}
> 
>+#ifdef HAVE_NO_CSUM_OFFLOAD
>+	err = xenbus_printf(xbt, dev->nodename, "feature-no-csum-offload", "%d", 1);
>+	if (err) {
>+		message = "writing feature-no-csum-offload";
>+		goto abort_transaction;
>+	}
>+#endif
>+
> 	err = xenbus_printf(xbt, dev->nodename, "feature-sg", "%d", 1);
> 	if (err) {
> 		message = "writing feature-sg";

Actions: View | Diff

Attachments on bug 414131: 279791