[lkml]   [2007]   [Oct]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] Fix bad data from non-direct-io read after direct-io write
    Linus Torvalds wrote:
    > On Fri, 26 Oct 2007, Zach Brown wrote:
    >> I can throw together a patch if you haven't already committed one by the
    >> time you read this ;).
    > I'm not touching that code except to send out possible patches for others
    > to test and comment on. I have no test-cases, nor any real interest in it.
    > So yeah, please send me a tested and thought-through patch.

    Sure thing. It looks like Karl just sent out the patch I had in mind,
    so I'll run it through the tests on Monday. I assume everyone can
    handle waiting that long.

    > It sounded like Karl actually had a test-case to trigger this, but maybe
    > I'm confused (and it sounds a bit unlikely, since it should be hard to
    > trigger..

    It's disappointingly easy to trigger with ext3 and ordered extending
    writes. The invalidation can race with jbd pinning the bhs while it
    commits the transaction. It happens that ext3 returns failure from
    ->releasepage if jbd is committing the transaction.;a=blob;f=c/aio-dio-invalidate-failure.c

    > Although maybe you can trigger it by doing a direct_IO write
    > with the *source* being an mmap() of the file you're writing to, and
    > depending on the write itself re-populating the destination in the page
    > cache?)

    dio will definitely bring the page in with get_user_pages() but I'd like
    to think that that pinned page could be safely invalidated out of the
    page cache while dio holds a page reference. I've never tried it, though.

    - z
    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-10-27 01:41    [W:0.021 / U:5.524 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site