From patchwork Fri Jul 22 08:33:25 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Anand Avati X-Patchwork-Id: 998412 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by demeter1.kernel.org (8.14.4/8.14.4) with ESMTP id p6M8i03A015329 for ; Fri, 22 Jul 2011 08:44:00 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753226Ab1GVIn6 (ORCPT ); Fri, 22 Jul 2011 04:43:58 -0400 Received: from hub024-nj-1.exch024.serverdata.net ([206.225.165.116]:24402 "EHLO HUB024-nj-1.exch024.serverdata.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752247Ab1GVIn5 (ORCPT ); Fri, 22 Jul 2011 04:43:57 -0400 X-Greylist: IP, sender and recipient auto-whitelisted, not delayed by milter-greylist-4.2.6 (demeter1.kernel.org [140.211.167.41]); Fri, 22 Jul 2011 08:44:00 +0000 (UTC) X-Greylist: delayed 602 seconds by postgrey-1.27 at vger.kernel.org; Fri, 22 Jul 2011 04:43:57 EDT Received: from localhost.localdomain (184.107.76.11) by east.exch024.serverdata.net (10.240.10.31) with Microsoft SMTP Server (TLS) id 14.1.289.1; Fri, 22 Jul 2011 01:33:54 -0700 From: Anand Avati To: CC: , , , Anand Avati Subject: [PATCH 1/2] vfs: pass 'struct file *' as parameter to ->check_flags() methods Date: Fri, 22 Jul 2011 01:33:25 -0700 Message-ID: <1311323606-15040-1-git-send-email-avati@gluster.com> X-Mailer: git-send-email 1.7.4.1 In-Reply-To: References: MIME-Version: 1.0 Sender: linux-nfs-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-nfs@vger.kernel.org Along with corresponding changes in - - Documentation/ - nfs - bad_inodes.c - fcntl.c --- Documentation/filesystems/Locking | 2 +- Documentation/filesystems/vfs.txt | 2 +- fs/bad_inode.c | 2 +- fs/fcntl.c | 2 +- fs/nfs/file.c | 6 +++--- include/linux/fs.h | 2 +- 6 files changed, 8 insertions(+), 8 deletions(-) diff --git a/Documentation/filesystems/Locking b/Documentation/filesystems/Locking index 57d827d..9619841 100644 --- a/Documentation/filesystems/Locking +++ b/Documentation/filesystems/Locking @@ -426,7 +426,7 @@ prototypes: loff_t *, int); unsigned long (*get_unmapped_area)(struct file *, unsigned long, unsigned long, unsigned long, unsigned long); - int (*check_flags)(int); + int (*check_flags)(struct file *, int); int (*flock) (struct file *, int, struct file_lock *); ssize_t (*splice_write)(struct pipe_inode_info *, struct file *, loff_t *, size_t, unsigned int); diff --git a/Documentation/filesystems/vfs.txt b/Documentation/filesystems/vfs.txt index 88b9f55..442aefb 100644 --- a/Documentation/filesystems/vfs.txt +++ b/Documentation/filesystems/vfs.txt @@ -764,7 +764,7 @@ struct file_operations { ssize_t (*sendfile) (struct file *, loff_t *, size_t, read_actor_t, void *); ssize_t (*sendpage) (struct file *, struct page *, int, size_t, loff_t *, int); unsigned long (*get_unmapped_area)(struct file *, unsigned long, unsigned long, unsigned long, unsigned long); - int (*check_flags)(int); + int (*check_flags)(struct file *, int); int (*flock) (struct file *, int, struct file_lock *); ssize_t (*splice_write)(struct pipe_inode_info *, struct file *, size_t, unsigned int); ssize_t (*splice_read)(struct file *, struct pipe_inode_info *, size_t, unsigned int); diff --git a/fs/bad_inode.c b/fs/bad_inode.c index bfcb18f..c7eef18 100644 --- a/fs/bad_inode.c +++ b/fs/bad_inode.c @@ -120,7 +120,7 @@ static unsigned long bad_file_get_unmapped_area(struct file *file, return -EIO; } -static int bad_file_check_flags(int flags) +static int bad_file_check_flags(struct file *filp, int flags) { return -EIO; } diff --git a/fs/fcntl.c b/fs/fcntl.c index 22764c7..1a2a6d3 100644 --- a/fs/fcntl.c +++ b/fs/fcntl.c @@ -174,7 +174,7 @@ static int setfl(int fd, struct file * filp, unsigned long arg) } if (filp->f_op && filp->f_op->check_flags) - error = filp->f_op->check_flags(arg); + error = filp->f_op->check_flags(filp, arg); if (error) return error; diff --git a/fs/nfs/file.c b/fs/nfs/file.c index 2f093ed..9f96a8b 100644 --- a/fs/nfs/file.c +++ b/fs/nfs/file.c @@ -56,7 +56,7 @@ static ssize_t nfs_file_write(struct kiocb *, const struct iovec *iov, unsigned long nr_segs, loff_t pos); static int nfs_file_flush(struct file *, fl_owner_t id); static int nfs_file_fsync(struct file *, int datasync); -static int nfs_check_flags(int flags); +static int nfs_check_flags(struct file *, int flags); static int nfs_lock(struct file *filp, int cmd, struct file_lock *fl); static int nfs_flock(struct file *filp, int cmd, struct file_lock *fl); static int nfs_setlease(struct file *file, long arg, struct file_lock **fl); @@ -105,7 +105,7 @@ const struct inode_operations nfs3_file_inode_operations = { # define IS_SWAPFILE(inode) (0) #endif -static int nfs_check_flags(int flags) +static int nfs_check_flags(struct file *filp, int flags) { if ((flags & (O_APPEND | O_DIRECT)) == (O_APPEND | O_DIRECT)) return -EINVAL; @@ -126,7 +126,7 @@ nfs_file_open(struct inode *inode, struct file *filp) filp->f_path.dentry->d_name.name); nfs_inc_stats(inode, NFSIOS_VFSOPEN); - res = nfs_check_flags(filp->f_flags); + res = nfs_check_flags(filp, filp->f_flags); if (res) return res; diff --git a/include/linux/fs.h b/include/linux/fs.h index b5b9792..98ce7c7 100644 --- a/include/linux/fs.h +++ b/include/linux/fs.h @@ -1564,7 +1564,7 @@ struct file_operations { int (*lock) (struct file *, int, struct file_lock *); ssize_t (*sendpage) (struct file *, struct page *, int, size_t, loff_t *, int); unsigned long (*get_unmapped_area)(struct file *, unsigned long, unsigned long, unsigned long, unsigned long); - int (*check_flags)(int); + int (*check_flags)(struct file *, int); int (*flock) (struct file *, int, struct file_lock *); ssize_t (*splice_write)(struct pipe_inode_info *, struct file *, loff_t *, size_t, unsigned int); ssize_t (*splice_read)(struct file *, loff_t *, struct pipe_inode_info *, size_t, unsigned int);