From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jarek Poplawski Subject: Re: [PATCH v2] tcp: splice as many packets as possible at once Date: Tue, 20 Jan 2009 10:20:53 +0000 Message-ID: <20090120102053.GA17004@ff.dom.local> References: <20090114.012919.117682429.davem@davemloft.net> <20090115230331.GB1123@1wt.eu> <20090115231934.GA8328@gondor.apana.org.au> <20090115.152608.89323697.davem@davemloft.net> <20090120083726.GA13806@ff.dom.local> <20090120093352.GB13806@ff.dom.local> <20090120100043.GA9167@ioremap.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: David Miller , herbert@gondor.apana.org.au, w@1wt.eu, dada1@cosmosbay.com, ben@zeus.com, mingo@elte.hu, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, jens.axboe@oracle.com To: Evgeniy Polyakov Return-path: Received: from ug-out-1314.google.com ([66.249.92.170]:11471 "EHLO ug-out-1314.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753241AbZATKVA (ORCPT ); Tue, 20 Jan 2009 05:21:00 -0500 Content-Disposition: inline In-Reply-To: <20090120100043.GA9167@ioremap.net> Sender: netdev-owner@vger.kernel.org List-ID: On Tue, Jan 20, 2009 at 01:00:43PM +0300, Evgeniy Polyakov wrote: > Hi Jarek. Hi Evgeniy. ... > > diff --git a/include/net/sock.h b/include/net/sock.h > > index 5a3a151..4ded741 100644 > > --- a/include/net/sock.h > > +++ b/include/net/sock.h > > @@ -190,6 +190,8 @@ struct sock_common { > > * @sk_user_data: RPC layer private data > > * @sk_sndmsg_page: cached page for sendmsg > > * @sk_sndmsg_off: cached offset for sendmsg > > + * @sk_splice_page: cached page for splice > > + * @sk_splice_off: cached offset for splice > > Ugh, increase every socket by 16 bytes... Does TCP one still fit the > page? Good question! Alas I can't check this soon, but if it's really like this, of course this needs some better idea and rework. (BTW, I'd like to prevent here as much as possible some strange activities like 1 byte (payload) packets getting full pages without any accounting.) Thanks, Jarek P.