[lkml]   [2011]   [Dec]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] VFS: br_write_lock locks on possible CPUs other than online CPUs
    On Tue, Dec 20, 2011 at 10:56:59AM +1100, Dave Chinner wrote:

    > > That got one hell of a deadlock potential ;-/ So far I'm more or less
    > > in favor of doing get_online_cpus() explicitly in fs/namespace.c, outside
    > > of namespace_sem. But I still have not convinced myself that it's
    > > really safe ;-/
    > Agreed, it looks like a lot simpler solution to this problem than a
    > notifier. But I don't think I know enough about the usage context to
    > determine if it is safe, either, so i can't really help you there. :/

    That's really nasty; mntput_no_expire() (and thus mntput()) wants
    br_write_lock()/br_write_unlock(). Right now we *know* that mntput()
    is non-blocking in situations when we are holding more than one reference.
    With that kind of change that isn't true anymore - one needs to have
    long-term refs to make it safe. And that's not going to be fun to audit...

    Can we get some kind of non-blocking exclusion against CPU hotplug? Note
    that we care about it only for writers, i.e. when we are going to cause
    cacheline bouncing from hell, no matter what.

    I *really* hate making br_write_lock() blocking and explicit get_online_cpus()
    around it isn't really any better. Too much PITA verifying correctness after
    the locking change.

    At that point in the cycle the original patch (loop over all CPUs, online or
    not) may turn out to be the only sane variant, as much as its going to
    hurt us.

     \ /
      Last update: 2011-12-20 05:09    [from the cache]
    ©2003-2014 Jasper Spaans. hosted at Digital Ocean