[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.029 / U:7.048 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site