Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756314AbZANMHW (ORCPT ); Wed, 14 Jan 2009 07:07:22 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752336AbZANMG5 (ORCPT ); Wed, 14 Jan 2009 07:06:57 -0500 Received: from nf-out-0910.google.com ([64.233.182.184]:53535 "EHLO nf-out-0910.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752071AbZANMG4 (ORCPT ); Wed, 14 Jan 2009 07:06:56 -0500 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=date:from:to:cc:subject:message-id:references:mime-version :content-type:content-disposition:in-reply-to:user-agent; b=suQi0kyc7W9rEBQJYf4aCVNAQkzYycF4wBZJDosLSu7Gt58lO75uVdt8E971HP7jQG TF+6HBVE7+LHuvnIiXp4gOB/nD8ZrZfbZaldd0y1GaNVXAsGDHgWkgasQBE+C4KNk/ox Kk3AOInYjkVULH+tt2/VijegMPflJwYGt+GcQ= Date: Wed, 14 Jan 2009 12:06:47 +0000 From: Jarek Poplawski To: David Miller Cc: herbert@gondor.apana.org.au, zbr@ioremap.net, dada1@cosmosbay.com, w@1wt.eu, ben@zeus.com, mingo@elte.hu, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, jens.axboe@oracle.com, Changli Gao Subject: Re: [PATCH] tcp: splice as many packets as possible at once Message-ID: <20090114120647.GA5954@ff.dom.local> References: <20090113.232710.55011568.davem@davemloft.net> <20090114082630.GB16692@gondor.apana.org.au> <20090114085308.GB4234@ff.dom.local> <20090114.012919.117682429.davem@davemloft.net> <20090114095454.GD4234@ff.dom.local> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20090114095454.GD4234@ff.dom.local> User-Agent: Mutt/1.5.18 (2008-05-17) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3551 Lines: 101 On Wed, Jan 14, 2009 at 09:54:54AM +0000, Jarek Poplawski wrote: > On Wed, Jan 14, 2009 at 01:29:19AM -0800, David Miller wrote: ... > > net: Fix data corruption when splicing from sockets. > > > > From: Jarek Poplawski ... > > > > Signed-off-by: David S. Miller > > You are very brave! I'd prefer to wait for at least minimal testing > by Willy... On the other hand, I can't waste your trust like this (plus I'm not suicidal), so a little update: Based on a review by Changli Gao : http://lkml.org/lkml/2008/2/26/210 Foreseen-by: Changli Gao Diagnosed-by: Willy Tarreau Reported-by: Willy Tarreau Signed-off-by: Jarek Poplawski > > Thanks, > Jarek P. > > BTW, an skb parameter could be removed from spd_fill_page() to make it > even faster... > > ... > > static inline int spd_fill_page(struct splice_pipe_desc *spd, struct page *page, > > unsigned int len, unsigned int offset, > > - struct sk_buff *skb) > > + struct sk_buff *skb, int linear) > > { > > if (unlikely(spd->nr_pages == PIPE_BUFFERS)) > > return 1; > > > > + if (linear) { > > + page = linear_to_page(page, len, offset); > > + if (!page) > > + return 1; > > + } > > + > > spd->pages[spd->nr_pages] = page; > > spd->partial[spd->nr_pages].len = len; > > spd->partial[spd->nr_pages].offset = offset; > > - spd->partial[spd->nr_pages].private = (unsigned long) skb_get(skb); > > spd->nr_pages++; > > + get_page(page); > > + > > return 0; > > } > > > > @@ -1369,7 +1382,7 @@ static inline void __segment_seek(struct page **page, unsigned int *poff, > > static inline int __splice_segment(struct page *page, unsigned int poff, > > unsigned int plen, unsigned int *off, > > unsigned int *len, struct sk_buff *skb, > > - struct splice_pipe_desc *spd) > > + struct splice_pipe_desc *spd, int linear) > > { > > if (!*len) > > return 1; > > @@ -1392,7 +1405,7 @@ static inline int __splice_segment(struct page *page, unsigned int poff, > > /* the linear region may spread across several pages */ > > flen = min_t(unsigned int, flen, PAGE_SIZE - poff); > > > > - if (spd_fill_page(spd, page, flen, poff, skb)) > > + if (spd_fill_page(spd, page, flen, poff, skb, linear)) > > return 1; > > > > __segment_seek(&page, &poff, &plen, flen); > > @@ -1419,7 +1432,7 @@ static int __skb_splice_bits(struct sk_buff *skb, unsigned int *offset, > > if (__splice_segment(virt_to_page(skb->data), > > (unsigned long) skb->data & (PAGE_SIZE - 1), > > skb_headlen(skb), > > - offset, len, skb, spd)) > > + offset, len, skb, spd, 1)) > > return 1; > > > > /* > > @@ -1429,7 +1442,7 @@ static int __skb_splice_bits(struct sk_buff *skb, unsigned int *offset, > > const skb_frag_t *f = &skb_shinfo(skb)->frags[seg]; > > > > if (__splice_segment(f->page, f->page_offset, f->size, > > - offset, len, skb, spd)) > > + offset, len, skb, spd, 0)) > > return 1; > > } > > > > -- > > To unsubscribe from this list: send the line "unsubscribe netdev" in > > the body of a message to majordomo@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/