lkml.org 
[lkml]   [2008]   [Oct]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH -v2 02/16] fsnotify: pass a file instead of an inode to open, read, and write
    Date
    fanotify, the upcoming notification system actually needs a f_path so it can
    do opens in the context of listeners, and it needs a file so it can get f_flags
    from the original process. Close already was passing a file.

    Signed-off-by: Eric Paris <eparis@redhat.com>
    ---

    fs/compat.c | 5 ++---
    fs/nfsd/vfs.c | 4 ++--
    fs/open.c | 2 +-
    fs/read_write.c | 8 ++++----
    include/linux/fsnotify.h | 9 ++++++---
    5 files changed, 15 insertions(+), 13 deletions(-)

    diff --git a/fs/compat.c b/fs/compat.c
    index 075d050..f59c796 100644
    --- a/fs/compat.c
    +++ b/fs/compat.c
    @@ -1125,11 +1125,10 @@ out:
    if (iov != iovstack)
    kfree(iov);
    if ((ret + (type == READ)) > 0) {
    - struct dentry *dentry = file->f_path.dentry;
    if (type == READ)
    - fsnotify_access(dentry);
    + fsnotify_access(file);
    else
    - fsnotify_modify(dentry);
    + fsnotify_modify(file);
    }
    return ret;
    }
    diff --git a/fs/nfsd/vfs.c b/fs/nfsd/vfs.c
    index 18060be..b576e38 100644
    --- a/fs/nfsd/vfs.c
    +++ b/fs/nfsd/vfs.c
    @@ -937,7 +937,7 @@ nfsd_vfs_read(struct svc_rqst *rqstp, struct svc_fh *fhp, struct file *file,
    nfsdstats.io_read += host_err;
    *count = host_err;
    err = 0;
    - fsnotify_access(file->f_path.dentry);
    + fsnotify_access(file);
    } else
    err = nfserrno(host_err);
    out:
    @@ -1004,7 +1004,7 @@ nfsd_vfs_write(struct svc_rqst *rqstp, struct svc_fh *fhp, struct file *file,
    set_fs(oldfs);
    if (host_err >= 0) {
    nfsdstats.io_write += cnt;
    - fsnotify_modify(file->f_path.dentry);
    + fsnotify_modify(file);
    }

    /* clear setuid/setgid flag after write */
    diff --git a/fs/open.c b/fs/open.c
    index 07da935..1057c34 100644
    --- a/fs/open.c
    +++ b/fs/open.c
    @@ -1020,7 +1020,7 @@ long do_sys_open(int dfd, const char __user *filename, int flags, int mode)
    put_unused_fd(fd);
    fd = PTR_ERR(f);
    } else {
    - fsnotify_open(f->f_path.dentry);
    + fsnotify_open(f);
    fd_install(fd, f);
    }
    }
    diff --git a/fs/read_write.c b/fs/read_write.c
    index 9ba495d..39f1fce 100644
    --- a/fs/read_write.c
    +++ b/fs/read_write.c
    @@ -258,7 +258,7 @@ ssize_t vfs_read(struct file *file, char __user *buf, size_t count, loff_t *pos)
    else
    ret = do_sync_read(file, buf, count, pos);
    if (ret > 0) {
    - fsnotify_access(file->f_path.dentry);
    + fsnotify_access(file);
    add_rchar(current, ret);
    }
    inc_syscr(current);
    @@ -313,7 +313,7 @@ ssize_t vfs_write(struct file *file, const char __user *buf, size_t count, loff_
    else
    ret = do_sync_write(file, buf, count, pos);
    if (ret > 0) {
    - fsnotify_modify(file->f_path.dentry);
    + fsnotify_modify(file);
    add_wchar(current, ret);
    }
    inc_syscw(current);
    @@ -604,9 +604,9 @@ out:
    kfree(iov);
    if ((ret + (type == READ)) > 0) {
    if (type == READ)
    - fsnotify_access(file->f_path.dentry);
    + fsnotify_access(file);
    else
    - fsnotify_modify(file->f_path.dentry);
    + fsnotify_modify(file);
    }
    return ret;
    }
    diff --git a/include/linux/fsnotify.h b/include/linux/fsnotify.h
    index a895131..dd76948 100644
    --- a/include/linux/fsnotify.h
    +++ b/include/linux/fsnotify.h
    @@ -136,8 +136,9 @@ static inline void fsnotify_mkdir(struct inode *inode, struct dentry *dentry)
    /*
    * fsnotify_access - file was read
    */
    -static inline void fsnotify_access(struct dentry *dentry)
    +static inline void fsnotify_access(struct file *file)
    {
    + struct dentry *dentry = file->f_path.dentry;
    struct inode *inode = dentry->d_inode;
    u32 mask = IN_ACCESS;

    @@ -152,8 +153,9 @@ static inline void fsnotify_access(struct dentry *dentry)
    /*
    * fsnotify_modify - file was modified
    */
    -static inline void fsnotify_modify(struct dentry *dentry)
    +static inline void fsnotify_modify(struct file *file)
    {
    + struct dentry *dentry = file->f_path.dentry;
    struct inode *inode = dentry->d_inode;
    u32 mask = IN_MODIFY;

    @@ -168,8 +170,9 @@ static inline void fsnotify_modify(struct dentry *dentry)
    /*
    * fsnotify_open - file was opened
    */
    -static inline void fsnotify_open(struct dentry *dentry)
    +static inline void fsnotify_open(struct file *file)
    {
    + struct dentry *dentry = file->f_path.dentry;
    struct inode *inode = dentry->d_inode;
    u32 mask = IN_OPEN;



    \
     
     \ /
      Last update: 2008-10-14 22:55    [W:3.260 / U:0.076 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site