[lkml]   [2012]   [May]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH v3.4-rc5] block: iocontext->nr_tasks should be initialized to one
    On Tue, May 01, 2012 at 02:48:41PM -0400, Jeff Moyer wrote:
    > Vivek Goyal <> writes:
    > > On Tue, May 01, 2012 at 02:31:07PM -0400, Jeff Moyer wrote:
    > >
    > > [..]
    > >> > But as to actual users, I really don't know. I agree it's probably not
    > >> > that widely used. If google still had that code search, we could get a
    > >> > better idea :-)
    > >>
    > >> I know of one project: the venerable dump/restore utility uses CLONE_IO.
    > >
    > > I thought you wrote cooperating queue logic to fix dump as it was not
    > > using CLONE_IO and IO from multiple threads was going in separate
    > > queues.
    > That's correct. I believe I sent the patch for dump before the kernel
    > patch was accepted. Plus, it can't hurt, right?

    Ok, so now you have fixed dump to use CLONE_IO.

    So only other user of coop thing remaining potentially is qemu. I was
    doing some qemu testing where threads were doing IO to nearby area
    but no coop merging was taking place. So not sure in practice how well
    does it work. Thought, that's irrlevant for this discussion. Thought of
    mentioning this observation.


     \ /
      Last update: 2012-05-01 21:21    [from the cache]
    ©2003-2014 Jasper Spaans. hosted at Digital Ocean