lkml.org 
[lkml]   [2007]   [Jun]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
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").

Linus
-
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/

\
 
 \ /
  Last update: 2007-06-02 17:43    [from the cache]
©2003-2014 Jasper Spaans. Advertise on this site