[lkml]   [2011]   [Jan]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH 17/46] fs: Use rename lock and RCU for multi-step operations
    On Sat, Nov 27, 2010 at 1:44 AM, Nick Piggin <> wrote:
    > The remaining usages for dcache_lock is to allow atomic, multi-step read-side
    > operations over the directory tree by excluding modifications to the tree.
    > Also, to walk in the leaf->root direction in the tree where we don't have
    > a natural d_lock ordering.
    > This could be accomplished by taking every d_lock, but this would mean a
    > huge number of locks and actually gets very tricky.
    > Solve this instead by using the rename seqlock for multi-step read-side
    > operations, retry in case of a rename so we don't walk up the wrong parent.
    > Concurrent dentry insertions are not serialised against.  Concurrent deletes
    > are tricky when walking up the directory: our parent might have been deleted
    > when dropping locks so also need to check and retry for that.
    > We can also use the rename lock in cases where livelock is a worry (and it
    > is introduced in subsequent patch).
    > Signed-off-by: Nick Piggin <>
    > @@ -237,6 +238,7 @@ static struct dentry *d_kill(struct dentry *dentry, struct dentry *parent)
    >        __releases(dcache_inode_lock)
    >        __releases(dcache_lock)
    >  {
    > +       dentry->d_parent = NULL;
    >        list_del(&dentry->d_u.d_child);
    >        if (parent)
    >                spin_unlock(&parent->d_lock);

    There's an issue with ceph as it references the
    dentry->d_parent(->d_inode) at dentry_release(), so setting
    dentry->d_parent to NULL here doesn't work with ceph. Though there is
    some workaround for it, we would like to be sure that this one is
    really required so that we don't exacerbate the ugliness. The
    workaround is to keep a pointer to the parent inode in the private
    dentry structure, which will be referenced only at the .release()
    callback. This is clearly not ideal.

    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-01-18 23:35    [W:0.022 / U:12.988 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site