[lkml]   [2011]   [May]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH RESEND 2/3 v2.6.39-rc7] block: make disk_block_events() properly wait for work cancellation
    On Tue, May 17, 2011 at 10:07 PM, Tejun Heo <> wrote:
    >> Just make the semaphore protect the count - and you're done.
    > Yeah, with that gone, we don't even need the open-coding inside
    > disk_check_events().  It can simply call syncing block and unblock.
    > But, do you want that in -rc7?  Unnecessarily complicated as the
    > current code may be, converting the lock to mutex is a larger change
    > than adding an outer mutex and I think it would be better to do that
    > during the next cycle.

    Quite frankly. right now I think I need to just release 2.6.39, and
    then for 2.6.40 merge the trivial

    if (!ev->block++)

    with a cc: stable for backporting.

    I'd _much_ prefer simple obvious code than have a outer mutex etc.
    Just make the rule be that "blocked" is protected by the new
    semaphore. I don't think it's used very often, and anybody who wants
    to block disk events needs to be in blockable context in order to wait
    for the delayed work cancel, right? So we can't be in some atomic
    context inside some other spinlock anyway, afaik. And there can be no
    lock order issues, since this would always be a new inner lock.


    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: 2011-05-18 11:49    [W:0.021 / U:45.880 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site