[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4947FA1C.2090509@vlnb.net>
Date: Tue, 16 Dec 2008 21:57:32 +0300
From: Vladislav Bolkhovitin <vst@...b.net>
To: Christoph Hellwig <hch@...radead.org>
CC: James Bottomley <James.Bottomley@...senPartnership.com>,
Evgeniy Polyakov <zbr@...emap.net>, linux-scsi@...r.kernel.org,
Andrew Morton <akpm@...ux-foundation.org>,
FUJITA Tomonori <fujita.tomonori@....ntt.co.jp>,
Mike Christie <michaelc@...wisc.edu>,
Jeff Garzik <jeff@...zik.org>,
Boaz Harrosh <bharrosh@...asas.com>,
Linus Torvalds <torvalds@...ux-foundation.org>,
linux-kernel@...r.kernel.org, scst-devel@...ts.sourceforge.net,
Bart Van Assche <bart.vanassche@...il.com>,
"Nicholas A. Bellinger" <nab@...ux-iscsi.org>,
netdev@...r.kernel.org
Subject: Re: [PATCH][RFC 23/23]: Support for zero-copy TCP transmit of user
space data
Christoph Hellwig wrote:
> If you don't believe that increasing struct page size for a fringe
> feature is a no-go submit the patch to the VM people and wait..
I guessed, it can be no-go, this is why I wrote that this feature isn't
required for iSCSI-SCST functionality. But, since the implementation is
*so* simple and doesn't do any layering violation, I have a hope that
once it disabled by default it will be harmless and, hence, could be
accepted. Only few people need this feature. Otherwise there will be an
alternative for them between enable that feature, then recompile, vs
patch the kernel, enable that feature, then recompile.
When I was developing it my main goal was to do it as simple as
possible. I believe, I succeeded in it. If it's rejected, it will simply
live out of tree until me or somebody else finds time to reimplement it
in an acceptable, although at least 10 times more complicated manner.
Thanks, I'll follow your advise.
Vlad
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists