[lkml]   [2007]   [Jun]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH] sendfile removal

On Sat, 2 Jun 2007, Jens Axboe wrote:
> splice() WILL return EAGAIN, but in that case it should have triggered
> the read-ahead and thus started some IO.

That's not enough.

If the IO has already been started, splice needs to wait.

> For the from-file case, see __generic_file_splice_read(). splice does:
> if (!PageUptodate(page)) {
> /*
> * If in nonblock mode then dont block on
> * waiting
> * for an in-flight io page
> */
> if (flags & SPLICE_F_NONBLOCK) {
> if (TestSetPageLocked(page))
> break;
> } else
> lock_page(page);

Yeah, that's just wrong.

Your suggested:

> if ((flags & SPLICE_F_NONBLOCK) && spd.nr_pages) {
> if (TestSetPageLocked(page))
> break;
> } else
> lock_page(page);
> should do that - always block for the first page and potentially return
> a partial results for the remaining pages that read-ahead kicked into
> gear.

would work, but I suspect that for a server, returning EAGAIN once is
actually the best option - if it has a select() loop, and something else
is running, the "return EAGAIN once" actually makes tons of sense (it
would basically boil down to the kernel effectively saying "ok, try
anything else you might have pending in your queues first, if you get back
to me, I'll block then").

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

 \ /
  Last update: 2007-06-02 17:43    [W:0.180 / U:1.728 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site