don't open-code file_inode()

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
index 3161d77bf29989f5a6edb6d817973630e2b996b4..3c28589722171b3991a43e1391b713fdcb929be5 100644 (file)
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
@@ -2501,7 +2501,7 @@ static void amdgpu_debugfs_remove_files(struct amdgpu_device *adev)
 static ssize_t amdgpu_debugfs_regs_read(struct file *f, char __user *buf,
                                        size_t size, loff_t *pos)
 {
-       struct amdgpu_device *adev = f->f_inode->i_private;
+       struct amdgpu_device *adev = file_inode(f)->i_private;
        ssize_t result = 0;
        int r;
        bool pm_pg_lock, use_bank;
@@ -2570,7 +2570,7 @@ end:
 static ssize_t amdgpu_debugfs_regs_write(struct file *f, const char __user *buf,
                                         size_t size, loff_t *pos)
 {
-       struct amdgpu_device *adev = f->f_inode->i_private;
+       struct amdgpu_device *adev = file_inode(f)->i_private;
        ssize_t result = 0;
        int r;
 
@@ -2601,7 +2601,7 @@ static ssize_t amdgpu_debugfs_regs_write(struct file *f, const char __user *buf,
 static ssize_t amdgpu_debugfs_regs_pcie_read(struct file *f, char __user *buf,
                                        size_t size, loff_t *pos)
 {
-       struct amdgpu_device *adev = f->f_inode->i_private;
+       struct amdgpu_device *adev = file_inode(f)->i_private;
        ssize_t result = 0;
        int r;
 
@@ -2628,7 +2628,7 @@ static ssize_t amdgpu_debugfs_regs_pcie_read(struct file *f, char __user *buf,
 static ssize_t amdgpu_debugfs_regs_pcie_write(struct file *f, const char __user *buf,
                                         size_t size, loff_t *pos)
 {
-       struct amdgpu_device *adev = f->f_inode->i_private;
+       struct amdgpu_device *adev = file_inode(f)->i_private;
        ssize_t result = 0;
        int r;
 
@@ -2656,7 +2656,7 @@ static ssize_t amdgpu_debugfs_regs_pcie_write(struct file *f, const char __user
 static ssize_t amdgpu_debugfs_regs_didt_read(struct file *f, char __user *buf,
                                        size_t size, loff_t *pos)
 {
-       struct amdgpu_device *adev = f->f_inode->i_private;
+       struct amdgpu_device *adev = file_inode(f)->i_private;
        ssize_t result = 0;
        int r;
 
@@ -2683,7 +2683,7 @@ static ssize_t amdgpu_debugfs_regs_didt_read(struct file *f, char __user *buf,
 static ssize_t amdgpu_debugfs_regs_didt_write(struct file *f, const char __user *buf,
                                         size_t size, loff_t *pos)
 {
-       struct amdgpu_device *adev = f->f_inode->i_private;
+       struct amdgpu_device *adev = file_inode(f)->i_private;
        ssize_t result = 0;
        int r;
 
@@ -2711,7 +2711,7 @@ static ssize_t amdgpu_debugfs_regs_didt_write(struct file *f, const char __user
 static ssize_t amdgpu_debugfs_regs_smc_read(struct file *f, char __user *buf,
                                        size_t size, loff_t *pos)
 {
-       struct amdgpu_device *adev = f->f_inode->i_private;
+       struct amdgpu_device *adev = file_inode(f)->i_private;
        ssize_t result = 0;
        int r;
 
@@ -2738,7 +2738,7 @@ static ssize_t amdgpu_debugfs_regs_smc_read(struct file *f, char __user *buf,
 static ssize_t amdgpu_debugfs_regs_smc_write(struct file *f, const char __user *buf,
                                         size_t size, loff_t *pos)
 {
-       struct amdgpu_device *adev = f->f_inode->i_private;
+       struct amdgpu_device *adev = file_inode(f)->i_private;
        ssize_t result = 0;
        int r;
 
@@ -2766,7 +2766,7 @@ static ssize_t amdgpu_debugfs_regs_smc_write(struct file *f, const char __user *
 static ssize_t amdgpu_debugfs_gca_config_read(struct file *f, char __user *buf,
                                        size_t size, loff_t *pos)
 {
-       struct amdgpu_device *adev = f->f_inode->i_private;
+       struct amdgpu_device *adev = file_inode(f)->i_private;
        ssize_t result = 0;
        int r;
        uint32_t *config, no_regs = 0;
@@ -2836,7 +2836,7 @@ static ssize_t amdgpu_debugfs_gca_config_read(struct file *f, char __user *buf,
 static ssize_t amdgpu_debugfs_sensor_read(struct file *f, char __user *buf,
                                        size_t size, loff_t *pos)
 {
-       struct amdgpu_device *adev = f->f_inode->i_private;
+       struct amdgpu_device *adev = file_inode(f)->i_private;
        int idx, r;
        int32_t value;
 

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ring.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ring.c
index 3cb5e903cd62896529d2ea18d5ea7f49c2feae73..2636f619569ecbdcba307dc6103fdeb18ea6b42d 100644 (file)
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ring.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ring.c
@@ -283,7 +283,7 @@ void amdgpu_ring_fini(struct amdgpu_ring *ring)
 static ssize_t amdgpu_debugfs_ring_read(struct file *f, char __user *buf,
                                        size_t size, loff_t *pos)
 {
-       struct amdgpu_ring *ring = (struct amdgpu_ring*)f->f_inode->i_private;
+       struct amdgpu_ring *ring = file_inode(f)->i_private;
        int r, i;
        uint32_t value, result, early[3];
 

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
index dcaf691f56b5577352d2238bdfd27e2f1aca2386..a5d00ef2b421a19ef60ac78a1447e3a80b8f092b 100644 (file)
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
@@ -1412,7 +1412,7 @@ static const struct drm_info_list amdgpu_ttm_debugfs_list[] = {
 static ssize_t amdgpu_ttm_vram_read(struct file *f, char __user *buf,
                                    size_t size, loff_t *pos)
 {
-       struct amdgpu_device *adev = f->f_inode->i_private;
+       struct amdgpu_device *adev = file_inode(f)->i_private;
        ssize_t result = 0;
        int r;
 
@@ -1456,7 +1456,7 @@ static const struct file_operations amdgpu_ttm_vram_fops = {
 static ssize_t amdgpu_ttm_gtt_read(struct file *f, char __user *buf,
                                   size_t size, loff_t *pos)
 {
-       struct amdgpu_device *adev = f->f_inode->i_private;
+       struct amdgpu_device *adev = file_inode(f)->i_private;
        ssize_t result = 0;
        int r;
 

diff --git a/drivers/staging/greybus/camera.c b/drivers/staging/greybus/camera.c
index 491bdd720c0c24a09879300b63599d978428ae39..cebb76e7d55c0c1540c712e94bd6ab12877fff26 100644 (file)
--- a/drivers/staging/greybus/camera.c
+++ b/drivers/staging/greybus/camera.c
@@ -1091,7 +1091,7 @@ static ssize_t gb_camera_debugfs_read(struct file *file, char __user *buf,
                                      size_t len, loff_t *offset)
 {
        const struct gb_camera_debugfs_entry *op = file->private_data;
-       struct gb_camera *gcam = file->f_inode->i_private;
+       struct gb_camera *gcam = file_inode(file)->i_private;
        struct gb_camera_debugfs_buffer *buffer;
        ssize_t ret;
 
@@ -1113,7 +1113,7 @@ static ssize_t gb_camera_debugfs_write(struct file *file,
                                       loff_t *offset)
 {
        const struct gb_camera_debugfs_entry *op = file->private_data;
-       struct gb_camera *gcam = file->f_inode->i_private;
+       struct gb_camera *gcam = file_inode(file)->i_private;
        ssize_t ret;
        char *kbuf;
 

diff --git a/drivers/staging/greybus/es2.c b/drivers/staging/greybus/es2.c
index baab460eeaa3828f7a3f84490f4dda566b647c44..d367cdffaa1040d0c2604faa28f320ad94b43254 100644 (file)
--- a/drivers/staging/greybus/es2.c
+++ b/drivers/staging/greybus/es2.c
@@ -1250,7 +1250,7 @@ static int apb_log_poll(void *data)
 static ssize_t apb_log_read(struct file *f, char __user *buf,
                                size_t count, loff_t *ppos)
 {
-       struct es2_ap_dev *es2 = f->f_inode->i_private;
+       struct es2_ap_dev *es2 = file_inode(f)->i_private;
        ssize_t ret;
        size_t copied;
        char *tmp_buf;
@@ -1304,7 +1304,7 @@ static void usb_log_disable(struct es2_ap_dev *es2)
 static ssize_t apb_log_enable_read(struct file *f, char __user *buf,
                                size_t count, loff_t *ppos)
 {
-       struct es2_ap_dev *es2 = f->f_inode->i_private;
+       struct es2_ap_dev *es2 = file_inode(f)->i_private;
        int enable = !IS_ERR_OR_NULL(es2->apb_log_task);
        char tmp_buf[3];
 
@@ -1317,7 +1317,7 @@ static ssize_t apb_log_enable_write(struct file *f, const char __user *buf,
 {
        int enable;
        ssize_t retval;
-       struct es2_ap_dev *es2 = f->f_inode->i_private;
+       struct es2_ap_dev *es2 = file_inode(f)->i_private;
 
        retval = kstrtoint_from_user(buf, count, 10, &enable);
        if (retval)

diff --git a/drivers/staging/greybus/svc.c b/drivers/staging/greybus/svc.c
index 550055ec27a5ce620f42b2c92662719ec0c32a65..8779270cadc1d4c86c21b8ac8ba08c96356fc564 100644 (file)
--- a/drivers/staging/greybus/svc.c
+++ b/drivers/staging/greybus/svc.c
@@ -757,7 +757,7 @@ static int gb_svc_version_request(struct gb_operation *op)
 static ssize_t pwr_debugfs_voltage_read(struct file *file, char __user *buf,
                                        size_t len, loff_t *offset)
 {
-       struct svc_debugfs_pwrmon_rail *pwrmon_rails = file->f_inode->i_private;
+       struct svc_debugfs_pwrmon_rail *pwrmon_rails = file_inode(file)->i_private;
        struct gb_svc *svc = pwrmon_rails->svc;
        int ret, desc;
        u32 value;
@@ -780,7 +780,7 @@ static ssize_t pwr_debugfs_voltage_read(struct file *file, char __user *buf,
 static ssize_t pwr_debugfs_current_read(struct file *file, char __user *buf,
                                        size_t len, loff_t *offset)
 {
-       struct svc_debugfs_pwrmon_rail *pwrmon_rails = file->f_inode->i_private;
+       struct svc_debugfs_pwrmon_rail *pwrmon_rails = file_inode(file)->i_private;
        struct gb_svc *svc = pwrmon_rails->svc;
        int ret, desc;
        u32 value;
@@ -803,7 +803,7 @@ static ssize_t pwr_debugfs_current_read(struct file *file, char __user *buf,
 static ssize_t pwr_debugfs_power_read(struct file *file, char __user *buf,
                                      size_t len, loff_t *offset)
 {
-       struct svc_debugfs_pwrmon_rail *pwrmon_rails = file->f_inode->i_private;
+       struct svc_debugfs_pwrmon_rail *pwrmon_rails = file_inode(file)->i_private;
        struct gb_svc *svc = pwrmon_rails->svc;
        int ret, desc;
        u32 value;

diff --git a/drivers/staging/greybus/timesync.c b/drivers/staging/greybus/timesync.c
index 2e68af7dea6d51ad8c3ae5657e0af9debaf4a285..09ce00484b3ff650f0d1197e58d147649799ba92 100644 (file)
--- a/drivers/staging/greybus/timesync.c
+++ b/drivers/staging/greybus/timesync.c
@@ -921,7 +921,7 @@ EXPORT_SYMBOL_GPL(gb_timesync_schedule_asynchronous);
 static ssize_t gb_timesync_ping_read(struct file *file, char __user *ubuf,
                                     size_t len, loff_t *offset, bool ktime)
 {
-       struct gb_timesync_svc *timesync_svc = file->f_inode->i_private;
+       struct gb_timesync_svc *timesync_svc = file_inode(file)->i_private;
        char *buf;
        ssize_t ret = 0;
 

diff --git a/drivers/target/target_core_configfs.c b/drivers/target/target_core_configfs.c
index 2001005bef45845dc45fdbf1e4bebf8c82d84fc5..a35a347ec357ad48626cc4083eca16588c29eb8b 100644 (file)
--- a/drivers/target/target_core_configfs.c
+++ b/drivers/target/target_core_configfs.c
@@ -143,7 +143,7 @@ static ssize_t target_core_item_dbroot_store(struct config_item *item,
                pr_err("db_root: cannot open: %s\n", db_root_stage);
                return -EINVAL;
        }
-       if (!S_ISDIR(fp->f_inode->i_mode)) {
+       if (!S_ISDIR(file_inode(fp)->i_mode)) {
                filp_close(fp, 0);
                mutex_unlock(&g_tf_lock);
                pr_err("db_root: not a directory: %s\n", db_root_stage);

diff --git a/fs/aio.c b/fs/aio.c
index 428484f2f8413dc6972da9857b3392264b9e6421..8edf253484af981c0e0f6a9a931a71f85d721918 100644 (file)
--- a/fs/aio.c
+++ b/fs/aio.c
@@ -277,10 +277,10 @@ static void put_aio_ring_file(struct kioctx *ctx)
        struct address_space *i_mapping;
 
        if (aio_ring_file) {
-               truncate_setsize(aio_ring_file->f_inode, 0);
+               truncate_setsize(file_inode(aio_ring_file), 0);
 
                /* Prevent further access to the kioctx from migratepages */
-               i_mapping = aio_ring_file->f_inode->i_mapping;
+               i_mapping = aio_ring_file->f_mapping;
                spin_lock(&i_mapping->private_lock);
                i_mapping->private_data = NULL;
                ctx->aio_ring_file = NULL;
@@ -483,7 +483,7 @@ static int aio_setup_ring(struct kioctx *ctx)
 
        for (i = 0; i < nr_pages; i++) {
                struct page *page;
-               page = find_or_create_page(file->f_inode->i_mapping,
+               page = find_or_create_page(file->f_mapping,
                                           i, GFP_HIGHUSER | __GFP_ZERO);
                if (!page)
                        break;

diff --git a/fs/autofs4/inode.c b/fs/autofs4/inode.c
index 438b5bf675b6a2a65651b4b06a397865d9d94af1..09e7d68dff02db1dd054c0a6e0fd97aa6bef8aaf 100644 (file)
--- a/fs/autofs4/inode.c
+++ b/fs/autofs4/inode.c
@@ -94,7 +94,7 @@ static int autofs4_show_options(struct seq_file *m, struct dentry *root)
                seq_printf(m, ",indirect");
 #ifdef CONFIG_CHECKPOINT_RESTORE
        if (sbi->pipe)
-               seq_printf(m, ",pipe_ino=%ld", sbi->pipe->f_inode->i_ino);
+               seq_printf(m, ",pipe_ino=%ld", file_inode(sbi->pipe)->i_ino);
        else
                seq_printf(m, ",pipe_ino=-1");
 #endif

diff --git a/fs/fcntl.c b/fs/fcntl.c
index 350a2c8cfd28f3f1339613b31564da1c2422e917..6e2771c210f63a177591bb0dd11e61bdc95bb6a2 100644 (file)
--- a/fs/fcntl.c
+++ b/fs/fcntl.c
@@ -52,7 +52,7 @@ static int setfl(int fd, struct file * filp, unsigned long arg)
                   arg |= O_NONBLOCK;
 
        /* Pipe packetized mode is controlled by O_DIRECT flag */
-       if (!S_ISFIFO(filp->f_inode->i_mode) && (arg & O_DIRECT)) {
+       if (!S_ISFIFO(inode->i_mode) && (arg & O_DIRECT)) {
                if (!filp->f_mapping || !filp->f_mapping->a_ops ||
                        !filp->f_mapping->a_ops->direct_IO)
                                return -EINVAL;

diff --git a/fs/orangefs/file.c b/fs/orangefs/file.c
index 02cc6139ec90798900f6c626934c9f6d2baee266..e6bbc8083d77235919ee56e093605283bb2d8dbb 100644 (file)
--- a/fs/orangefs/file.c
+++ b/fs/orangefs/file.c
@@ -724,7 +724,7 @@ static int orangefs_lock(struct file *filp, int cmd, struct file_lock *fl)
 {
        int rc = -EINVAL;
 
-       if (ORANGEFS_SB(filp->f_inode->i_sb)->flags & ORANGEFS_OPT_LOCAL_LOCK) {
+       if (ORANGEFS_SB(file_inode(filp)->i_sb)->flags & ORANGEFS_OPT_LOCAL_LOCK) {
                if (cmd == F_GETLK) {
                        rc = 0;
                        posix_test_lock(filp, fl);

diff --git a/fs/orangefs/orangefs-debugfs.c b/fs/orangefs/orangefs-debugfs.c
index 38887cc5577fa901acd907a0af2fb016a3441b2c..f1e824979aada8ea2c3c31e66a91a1f3313a5ce6 100644 (file)
--- a/fs/orangefs/orangefs-debugfs.c
+++ b/fs/orangefs/orangefs-debugfs.c
@@ -434,6 +434,7 @@ static ssize_t orangefs_debug_write(struct file *file,
        char *debug_string;
        struct orangefs_kernel_op_s *new_op = NULL;
        struct client_debug_mask c_mask = { NULL, 0, 0 };
+       char *s;
 
        gossip_debug(GOSSIP_DEBUGFS_DEBUG,
                "orangefs_debug_write: %pD\n",
@@ -521,8 +522,9 @@ static ssize_t orangefs_debug_write(struct file *file,
        }
 
        mutex_lock(&orangefs_debug_lock);
-       memset(file->f_inode->i_private, 0, ORANGEFS_MAX_DEBUG_STRING_LEN);
-       sprintf((char *)file->f_inode->i_private, "%s\n", debug_string);
+       s = file_inode(file)->i_private;
+       memset(s, 0, ORANGEFS_MAX_DEBUG_STRING_LEN);
+       sprintf(s, "%s\n", debug_string);
        mutex_unlock(&orangefs_debug_lock);
 
        *ppos += count;

diff --git a/fs/overlayfs/copy_up.c b/fs/overlayfs/copy_up.c
index 36795eed40b09ee1bbb6766e65019b8d1c6695f0..2838bddb1f91324da782bff2915cce1b60a1ddc0 100644 (file)
--- a/fs/overlayfs/copy_up.c
+++ b/fs/overlayfs/copy_up.c
@@ -33,7 +33,7 @@ static int ovl_check_fd(const void *data, struct file *f, unsigned int fd)
 {
        const struct dentry *dentry = data;
 
-       if (f->f_inode == d_inode(dentry))
+       if (file_inode(f) == d_inode(dentry))
                pr_warn_ratelimited("overlayfs: Warning: Copying up %pD, but open R/O on fd %u which will cease to be coherent [pid=%d %s]\n",
                                    f, fd, current->pid, current->comm);
        return 0;

diff --git a/kernel/audit_watch.c b/kernel/audit_watch.c
index 0d302a87f21b58ac711aaf5720cca12640bfc0ef..99401efef7acd4d20b7909b9cf49ebc5e2a192be 100644 (file)
--- a/kernel/audit_watch.c
+++ b/kernel/audit_watch.c
@@ -548,8 +548,8 @@ int audit_exe_compare(struct task_struct *tsk, struct audit_fsnotify_mark *mark)
        exe_file = get_task_exe_file(tsk);
        if (!exe_file)
                return 0;
-       ino = exe_file->f_inode->i_ino;
-       dev = exe_file->f_inode->i_sb->s_dev;
+       ino = file_inode(exe_file)->i_ino;
+       dev = file_inode(exe_file)->i_sb->s_dev;
        fput(exe_file);
        return audit_mark_compare(mark, ino, dev);
 }

diff --git a/kernel/events/core.c b/kernel/events/core.c
index 6ee1febdf6ff838b21db38277e0760ae0909e91f..5134a1c17186f0621da100ca1ed6dd5e2e60813b 100644 (file)
--- a/kernel/events/core.c
+++ b/kernel/events/core.c
@@ -6701,7 +6701,7 @@ static bool perf_addr_filter_match(struct perf_addr_filter *filter,
                                     struct file *file, unsigned long offset,
                                     unsigned long size)
 {
-       if (filter->inode != file->f_inode)
+       if (filter->inode != file_inode(file))
                return false;
 
        if (filter->offset > offset + size)

diff --git a/kernel/locking/qspinlock_stat.h b/kernel/locking/qspinlock_stat.h
index eb0a599fcf58f3d5fa9a72f7dc429bdaa1bfe4a5..e852be4851fc91e0b4beb6dbb8c66b45206364a5 100644 (file)
--- a/kernel/locking/qspinlock_stat.h
+++ b/kernel/locking/qspinlock_stat.h
@@ -108,11 +108,7 @@ static ssize_t qstat_read(struct file *file, char __user *user_buf,
        /*
         * Get the counter ID stored in file->f_inode->i_private
         */
-       if (!file->f_inode) {
-               WARN_ON_ONCE(1);
-               return -EBADF;
-       }
-       counter = (long)(file->f_inode->i_private);
+       counter = (long)file_inode(file)->i_private;
 
        if (counter >= qstat_num)
                return -EBADF;
@@ -177,11 +173,7 @@ static ssize_t qstat_write(struct file *file, const char __user *user_buf,
        /*
         * Get the counter ID stored in file->f_inode->i_private
         */
-       if (!file->f_inode) {
-               WARN_ON_ONCE(1);
-               return -EBADF;
-       }
-       if ((long)(file->f_inode->i_private) != qstat_reset_cnts)
+       if ((long)file_inode(file)->i_private != qstat_reset_cnts)
                return count;
 
        for_each_possible_cpu(cpu) {

diff --git a/security/smack/smack_lsm.c b/security/smack/smack_lsm.c
index 1cb060293505d14046f084d5c7c5c3e80b480594..ad170310a8562709d02909e99eb1e97ef2cbade2 100644 (file)
--- a/security/smack/smack_lsm.c
+++ b/security/smack/smack_lsm.c
@@ -225,7 +225,7 @@ static int smk_bu_credfile(const struct cred *cred, struct file *file,
 {
        struct task_smack *tsp = cred->security;
        struct smack_known *sskp = tsp->smk_task;
-       struct inode *inode = file->f_inode;
+       struct inode *inode = file_inode(file);
        struct inode_smack *isp = inode->i_security;
        char acc[SMK_NUM_ACCESS_TYPE + 1];