lkml.org 
[lkml]   [2007]   [Jul]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Subject[PATCH 22/23] elevate mnt writers for vfs_unlink() callers
    From
    Date


    Signed-off-by: Dave Hansen <haveblue@us.ibm.com>
    ---

    lxc-dave/fs/namei.c | 4 ++++
    lxc-dave/ipc/mqueue.c | 5 ++++-
    2 files changed, 8 insertions(+), 1 deletion(-)

    diff -puN fs/namei.c~elevate-mnt-writers-for-vfs-unlink-callers fs/namei.c
    --- lxc/fs/namei.c~elevate-mnt-writers-for-vfs-unlink-callers 2007-07-10 12:46:16.000000000 -0700
    +++ lxc-dave/fs/namei.c 2007-07-10 12:46:16.000000000 -0700
    @@ -2195,7 +2195,11 @@ static long do_unlinkat(int dfd, const c
    inode = dentry->d_inode;
    if (inode)
    atomic_inc(&inode->i_count);
    + error = mnt_want_write(nd.mnt);
    + if (error)
    + goto exit2;
    error = vfs_unlink(nd.dentry->d_inode, dentry);
    + mnt_drop_write(nd.mnt);
    exit2:
    dput(dentry);
    }
    diff -puN ipc/mqueue.c~elevate-mnt-writers-for-vfs-unlink-callers ipc/mqueue.c
    --- lxc/ipc/mqueue.c~elevate-mnt-writers-for-vfs-unlink-callers 2007-07-10 12:46:16.000000000 -0700
    +++ lxc-dave/ipc/mqueue.c 2007-07-10 12:46:16.000000000 -0700
    @@ -750,8 +750,11 @@ asmlinkage long sys_mq_unlink(const char
    inode = dentry->d_inode;
    if (inode)
    atomic_inc(&inode->i_count);
    -
    + err = mnt_want_write(mqueue_mnt);
    + if (err)
    + goto out_err;
    err = vfs_unlink(dentry->d_parent->d_inode, dentry);
    + mnt_drop_write(mqueue_mnt);
    out_err:
    dput(dentry);

    _
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/
    \
     
     \ /
      Last update: 2007-07-12 02:29    [W:2.564 / U:0.020 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site