From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: [PATCH 5/6] xen/netback: try to pull a minimum of 72 bytes into the skb data area Date: Tue, 23 Feb 2010 16:47:10 +0000 Message-ID: <1266943630-17002-5-git-send-email-ian.campbell@citrix.com> References: <1266943615.11737.6467.camel@zakaz.uk.xensource.com> Return-path: In-Reply-To: <1266943615.11737.6467.camel@zakaz.uk.xensource.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: xen-devel@lists.xensource.com Cc: Jeremy Fitzhardinge , Steven Smith List-Id: xen-devel@lists.xenproject.org From: Steven Smith --- drivers/xen/netback/netback.c | 9 +++------ 1 files changed, 3 insertions(+), 6 deletions(-) diff --git a/drivers/xen/netback/netback.c b/drivers/xen/netback/netback.c index e668704..0bc6398 100644 --- a/drivers/xen/netback/netback.c +++ b/drivers/xen/netback/netback.c @@ -116,13 +116,10 @@ static inline int netif_page_index(struct page *pg) /* * This is the amount of packet we copy rather than map, so that the * guest can't fiddle with the contents of the headers while we do - * packet processing on them (netfilter, routing, etc). This could - * probably do with being larger, since 1) 64-bytes isn't necessarily - * long enough to cover a full christmas-tree ip+tcp header, let alone - * packet contents, and 2) the data is probably in cache anyway - * (though perhaps some other cpu's cache). + * packet processing on them (netfilter, routing, etc). 72 is enough + * to cover TCP+IP headers including options. */ -#define PKT_PROT_LEN 64 +#define PKT_PROT_LEN 72 static struct pending_tx_info { struct xen_netif_tx_request req; -- 1.5.6.5