]> git.baikalelectronics.ru Git - kernel.git/commitdiff
GFS2: Minor improvements to comments and documentation
authorBob Peterson <rpeterso@redhat.com>
Fri, 6 Apr 2018 20:07:45 +0000 (13:07 -0700)
committerBob Peterson <rpeterso@redhat.com>
Thu, 12 Apr 2018 17:07:51 +0000 (10:07 -0700)
This patch simply fixes some comments and the gfs2-glocks.txt file:
Places where i_rwsem was called i_mutex, and adding i_rw_mutex.

Signed-off-by: Bob Peterson <rpeterso@redhat.com>
Documentation/filesystems/gfs2-glocks.txt
fs/gfs2/bmap.c
fs/gfs2/ops_fstype.c

index 1fb12f9dfe4877fb5a37744b1298d4b464aa74c6..7059623635b2d32fdaac02856447d6ab851eb04e 100644 (file)
@@ -100,14 +100,15 @@ indicates that it is caching uptodate data.
 
 Glock locking order within GFS2:
 
- 1. i_mutex (if required)
+ 1. i_rwsem (if required)
  2. Rename glock (for rename only)
  3. Inode glock(s)
     (Parents before children, inodes at "same level" with same parent in
      lock number order)
  4. Rgrp glock(s) (for (de)allocation operations)
  5. Transaction glock (via gfs2_trans_begin) for non-read operations
- 6. Page lock  (always last, very important!)
+ 6. i_rw_mutex (if required)
+ 7. Page lock  (always last, very important!)
 
 There are two glocks per inode. One deals with access to the inode
 itself (locking order as above), and the other, known as the iopen
index 685c305cbeb6f7bcc6ad097ce20acfd1859ebdf3..278ed0869c3c11b9a3a512de9bb043c0b35bfee8 100644 (file)
@@ -1744,7 +1744,7 @@ do_grow_qunlock:
  * @newsize: the size to make the file
  *
  * The file size can grow, shrink, or stay the same size. This
- * is called holding i_mutex and an exclusive glock on the inode
+ * is called holding i_rwsem and an exclusive glock on the inode
  * in question.
  *
  * Returns: errno
index e6a0a8a89ea7e5696ea262b2d1d047b0e9962c00..3ba3f167641ccb9d7935532377abf2565fc2d2ea 100644 (file)
@@ -825,7 +825,7 @@ static int init_inodes(struct gfs2_sbd *sdp, int undo)
                goto fail_rindex;
        }
        /*
-        * i_mutex on quota files is special. Since this inode is hidden system
+        * i_rwsem on quota files is special. Since this inode is hidden system
         * file, we are safe to define locking ourselves.
         */
        lockdep_set_class(&sdp->sd_quota_inode->i_rwsem,