[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <465DD97D.9010807@cosmosbay.com>
Date: Wed, 30 May 2007 22:07:25 +0200
From: Eric Dumazet <dada1@...mosbay.com>
To: Linus Torvalds <torvalds@...ux-foundation.org>
CC: Mark Lord <lkml@....ca>, Ingo Molnar <mingo@...e.hu>,
Jeff Garzik <jeff@...zik.org>,
Zach Brown <zach.brown@...cle.com>,
linux-kernel@...r.kernel.org,
Arjan van de Ven <arjan@...radead.org>,
Christoph Hellwig <hch@...radead.org>,
Andrew Morton <akpm@....com.au>,
Alan Cox <alan@...rguk.ukuu.org.uk>,
Ulrich Drepper <drepper@...hat.com>,
Evgeniy Polyakov <johnpol@....mipt.ru>,
"David S. Miller" <davem@...emloft.net>,
Suparna Bhattacharya <suparna@...ibm.com>,
Davide Libenzi <davidel@...ilserver.org>,
Jens Axboe <jens.axboe@...cle.com>,
Thomas Gleixner <tglx@...utronix.de>
Subject: Re: Syslets, Threadlets, generic AIO support, v6
Linus Torvalds a écrit :
>
> On Wed, 30 May 2007, Mark Lord wrote:
>> I wonder how useful it would be to reimplement sendfile()
>> using splice(), either in glibc or inside the kernel itself?
>
> I'd like that, if only because right now we have two separate paths that
> kind of do the same thing, and splice really is the only one that is
> generic.
>
> I thought Jens even had some experimental patches for it. It might be
> worth to "just do it" - there's some internal overhead, but on the other
> hand, it's also likely the best way to make sure any issues get sorted
> out.
>
Last time I played with splice(), I found a bug with readahead logic, most
probably because nobody but me tried it before.
(corrected by Fengguang Wu in commit 9ae9d68cbf3fe0ec17c17c9ecaa2188ffb854a66 )
So yes, reimplement sendfile() should help to find last splice() bugs, and as
a bonus it could add non blocking disk io, (O_NONBLOCK on input file -> socket)
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists