fs/ocfs2: fix comments mentioning i_mutex
authorhongnanli <hongnan.li@linux.alibaba.com>
Tue, 22 Mar 2022 21:38:45 +0000 (14:38 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Tue, 22 Mar 2022 22:57:00 +0000 (15:57 -0700)
inode->i_mutex has been replaced with inode->i_rwsem long ago.  Fix
comments still mentioning i_mutex.

Link: https://lkml.kernel.org/r/20220214031314.100094-1-hongnan.li@linux.alibaba.com
Signed-off-by: hongnanli <hongnan.li@linux.alibaba.com>
Acked-by: Joseph Qi <joseph.qi@linux.alibaba.com>
Cc: Mark Fasheh <mark@fasheh.com>
Cc: Joel Becker <jlbec@evilplan.org>
Cc: Junxiao Bi <junxiao.bi@oracle.com>
Cc: Changwei Ge <gechangwei@live.cn>
Cc: Gang He <ghe@suse.com>
Cc: Jun Piao <piaojun@huawei.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
fs/ocfs2/alloc.c
fs/ocfs2/aops.c
fs/ocfs2/cluster/nodemanager.c
fs/ocfs2/dir.c
fs/ocfs2/file.c
fs/ocfs2/inode.c
fs/ocfs2/localalloc.c
fs/ocfs2/namei.c
fs/ocfs2/ocfs2.h
fs/ocfs2/quota_global.c
fs/ocfs2/xattr.c

index bf9357123bc58781156b003a58d080a74fce5933..49f41074baadd194d989debdb291b2333f6512e2 100644 (file)
@@ -5981,7 +5981,7 @@ bail:
        return status;
 }
 
-/* Expects you to already be holding tl_inode->i_mutex */
+/* Expects you to already be holding tl_inode->i_rwsem */
 int __ocfs2_flush_truncate_log(struct ocfs2_super *osb)
 {
        int status;
index 498da317580a75e2e6bbb03e2f66f1a930d44216..c23e1c243cc619da1fa6e4145c6c2a5d4b5c93ce 100644 (file)
@@ -2311,7 +2311,7 @@ static int ocfs2_dio_end_io_write(struct inode *inode,
 
        down_write(&oi->ip_alloc_sem);
 
-       /* Delete orphan before acquire i_mutex. */
+       /* Delete orphan before acquire i_rwsem. */
        if (dwc->dw_orphaned) {
                BUG_ON(dwc->dw_writer_pid != task_pid_nr(current));
 
index 625c9252141698d2cba513d9a67aeb7f8190560f..27fee68f860a682f66d080635025dfb259d985e9 100644 (file)
@@ -689,7 +689,7 @@ static struct config_group *o2nm_cluster_group_make_group(struct config_group *g
        struct o2nm_node_group *ns = NULL;
        struct config_group *o2hb_group = NULL, *ret = NULL;
 
-       /* this runs under the parent dir's i_mutex; there can be only
+       /* this runs under the parent dir's i_rwsem; there can be only
         * one caller in here at a time */
        if (o2nm_single_cluster)
                return ERR_PTR(-ENOSPC);
index f2cc1ff29e6de0671b2e0eefa130a7508a5d51be..81c3d65d68fec7cd4eb7061f1a5cefd81e65d60d 100644 (file)
@@ -1957,7 +1957,7 @@ bail_nolock:
 }
 
 /*
- * NOTE: this should always be called with parent dir i_mutex taken.
+ * NOTE: this should always be called with parent dir i_rwsem taken.
  */
 int ocfs2_find_files_on_disk(const char *name,
                             int namelen,
@@ -2003,7 +2003,7 @@ int ocfs2_lookup_ino_from_name(struct inode *dir, const char *name,
  * Return 0 if the name does not exist
  * Return -EEXIST if the directory contains the name
  *
- * Callers should have i_mutex + a cluster lock on dir
+ * Callers should have i_rwsem + a cluster lock on dir
  */
 int ocfs2_check_dir_for_entry(struct inode *dir,
                              const char *name,
index 24321c44cd42e07d0d78b510c2f66f02ae406eef..01b7407a8893f0af1e7bb0d596035c66e8043cd3 100644 (file)
@@ -270,7 +270,7 @@ int ocfs2_update_inode_atime(struct inode *inode,
 
        /*
         * Don't use ocfs2_mark_inode_dirty() here as we don't always
-        * have i_mutex to guard against concurrent changes to other
+        * have i_rwsem to guard against concurrent changes to other
         * inode fields.
         */
        inode->i_atime = current_time(inode);
@@ -1065,7 +1065,7 @@ static int ocfs2_extend_file(struct inode *inode,
        /*
         * The alloc sem blocks people in read/write from reading our
         * allocation until we're done changing it. We depend on
-        * i_mutex to block other extend/truncate calls while we're
+        * i_rwsem to block other extend/truncate calls while we're
         * here.  We even have to hold it for sparse files because there
         * might be some tail zeroing.
         */
index 6c2411c2afcf1c8ecb9ce4fb6d4839a30a2d9ae6..5739dc301569865ec4070e3ffaee579adea17757 100644 (file)
@@ -713,7 +713,7 @@ bail:
 /*
  * Serialize with orphan dir recovery. If the process doing
  * recovery on this orphan dir does an iget() with the dir
- * i_mutex held, we'll deadlock here. Instead we detect this
+ * i_rwsem held, we'll deadlock here. Instead we detect this
  * and exit early - recovery will wipe this inode for us.
  */
 static int ocfs2_check_orphan_recovery_state(struct ocfs2_super *osb,
index 5f6bacbeef6b8e00976f253b560d37b2ab928a6e..c4426d12a2adb1920a153d71515d7f1faefd5ce1 100644 (file)
@@ -606,7 +606,7 @@ out:
 
 /*
  * make sure we've got at least bits_wanted contiguous bits in the
- * local alloc. You lose them when you drop i_mutex.
+ * local alloc. You lose them when you drop i_rwsem.
  *
  * We will add ourselves to the transaction passed in, but may start
  * our own in order to shift windows.
@@ -636,7 +636,7 @@ int ocfs2_reserve_local_alloc_bits(struct ocfs2_super *osb,
 
        /*
         * We must double check state and allocator bits because
-        * another process may have changed them while holding i_mutex.
+        * another process may have changed them while holding i_rwsem.
         */
        spin_lock(&osb->osb_lock);
        if (!ocfs2_la_state_enabled(osb) ||
@@ -1029,7 +1029,7 @@ enum ocfs2_la_event {
 /*
  * Given an event, calculate the size of our next local alloc window.
  *
- * This should always be called under i_mutex of the local alloc inode
+ * This should always be called under i_rwsem of the local alloc inode
  * so that local alloc disabling doesn't race with processes trying to
  * use the allocator.
  *
index 2c46ff6ba4ea23492367d370073b9d17c32cbdd9..c75fd54b91854ba9a95b0538a4723e2235983ad6 100644 (file)
@@ -476,7 +476,7 @@ leave:
                ocfs2_free_alloc_context(meta_ac);
 
        /*
-        * We should call iput after the i_mutex of the bitmap been
+        * We should call iput after the i_rwsem of the bitmap been
         * unlocked in ocfs2_free_alloc_context, or the
         * ocfs2_delete_inode will mutex_lock again.
         */
index bb62cc2e0211b3a1f036758d25c619f3ca1c497d..3375275714612663b759fd676270ab6067071b32 100644 (file)
@@ -355,7 +355,7 @@ struct ocfs2_super
        struct delayed_work             la_enable_wq;
 
        /*
-        * Must hold local alloc i_mutex and osb->osb_lock to change
+        * Must hold local alloc i_rwsem and osb->osb_lock to change
         * local_alloc_bits. Reads can be done under either lock.
         */
        unsigned int local_alloc_bits;
@@ -430,7 +430,7 @@ struct ocfs2_super
        atomic_t                        osb_tl_disable;
        /*
         * How many clusters in our truncate log.
-        * It must be protected by osb_tl_inode->i_mutex.
+        * It must be protected by osb_tl_inode->i_rwsem.
         */
        unsigned int truncated_clusters;
 
index f033de733adb3c7314d47fc5400ad8bf2934abc5..273f65e0aabac127be3d4175ee6b205b94a0aeef 100644 (file)
@@ -36,7 +36,7 @@
  * should be obeyed by all the functions:
  * - any write of quota structure (either to local or global file) is protected
  *   by dqio_sem or dquot->dq_lock.
- * - any modification of global quota file holds inode cluster lock, i_mutex,
+ * - any modification of global quota file holds inode cluster lock, i_rwsem,
  *   and ip_alloc_sem of the global quota file (achieved by
  *   ocfs2_lock_global_qf). It also has to hold qinfo_lock.
  * - an allocation of new blocks for local quota file is protected by
index dd784eb0cd7c42ea1ac26d75eb6f8cdeb9c457b8..95d0611c5fc7d12464d53b727d477e1bb883ca7b 100644 (file)
@@ -7205,7 +7205,7 @@ out:
  * Used for reflink a non-preserve-security file.
  *
  * It uses common api like ocfs2_xattr_set, so the caller
- * must not hold any lock expect i_mutex.
+ * must not hold any lock expect i_rwsem.
  */
 int ocfs2_init_security_and_acl(struct inode *dir,
                                struct inode *inode,