diff options
author | J. Bruce Fields <bfields@redhat.com> | 2012-04-18 15:21:34 -0400 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2013-11-09 00:16:40 -0500 |
commit | 40bd22c9f8617ddd5da06044c81f72a2cf700791 (patch) | |
tree | 7969f179ac6e23c8fd1a9ea6bd4e9cc35ee1bfa6 | |
parent | 275555163e3ac09c016c964a10e0f389cf152c4c (diff) |
vfs: rename I_MUTEX_QUOTA now that it's not used for quotas
I_MUTEX_QUOTA is now just being used whenever we want to lock two
non-directories. So the name isn't right. I_MUTEX_NONDIR2 isn't
especially elegant but it's the best I could think of.
Also fix some outdated documentation.
Acked-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
-rw-r--r-- | fs/inode.c | 4 | ||||
-rw-r--r-- | include/linux/fs.h | 9 |
2 files changed, 8 insertions, 5 deletions
diff --git a/fs/inode.c b/fs/inode.c index 51f329b98aa4..ce48c359ce9e 100644 --- a/fs/inode.c +++ b/fs/inode.c | |||
@@ -957,10 +957,10 @@ void lock_two_nondirectories(struct inode *inode1, struct inode *inode2) | |||
957 | WARN_ON_ONCE(S_ISDIR(inode2->i_mode)); | 957 | WARN_ON_ONCE(S_ISDIR(inode2->i_mode)); |
958 | if (inode1 < inode2) { | 958 | if (inode1 < inode2) { |
959 | mutex_lock(&inode1->i_mutex); | 959 | mutex_lock(&inode1->i_mutex); |
960 | mutex_lock_nested(&inode2->i_mutex, I_MUTEX_QUOTA); | 960 | mutex_lock_nested(&inode2->i_mutex, I_MUTEX_NONDIR2); |
961 | } else { | 961 | } else { |
962 | mutex_lock(&inode2->i_mutex); | 962 | mutex_lock(&inode2->i_mutex); |
963 | mutex_lock_nested(&inode1->i_mutex, I_MUTEX_QUOTA); | 963 | mutex_lock_nested(&inode1->i_mutex, I_MUTEX_NONDIR2); |
964 | } | 964 | } |
965 | } | 965 | } |
966 | EXPORT_SYMBOL(lock_two_nondirectories); | 966 | EXPORT_SYMBOL(lock_two_nondirectories); |
diff --git a/include/linux/fs.h b/include/linux/fs.h index 4e1a0b41f966..ed7f94af1ab2 100644 --- a/include/linux/fs.h +++ b/include/linux/fs.h | |||
@@ -623,10 +623,13 @@ static inline int inode_unhashed(struct inode *inode) | |||
623 | * 0: the object of the current VFS operation | 623 | * 0: the object of the current VFS operation |
624 | * 1: parent | 624 | * 1: parent |
625 | * 2: child/target | 625 | * 2: child/target |
626 | * 3: quota file | 626 | * 3: xattr |
627 | * 4: second non-directory | ||
628 | * The last is for certain operations (such as rename) which lock two | ||
629 | * non-directories at once. | ||
627 | * | 630 | * |
628 | * The locking order between these classes is | 631 | * The locking order between these classes is |
629 | * parent -> child -> normal -> xattr -> quota | 632 | * parent -> child -> normal -> xattr -> second non-directory |
630 | */ | 633 | */ |
631 | enum inode_i_mutex_lock_class | 634 | enum inode_i_mutex_lock_class |
632 | { | 635 | { |
@@ -634,7 +637,7 @@ enum inode_i_mutex_lock_class | |||
634 | I_MUTEX_PARENT, | 637 | I_MUTEX_PARENT, |
635 | I_MUTEX_CHILD, | 638 | I_MUTEX_CHILD, |
636 | I_MUTEX_XATTR, | 639 | I_MUTEX_XATTR, |
637 | I_MUTEX_QUOTA | 640 | I_MUTEX_NONDIR2 |
638 | }; | 641 | }; |
639 | 642 | ||
640 | void lock_two_nondirectories(struct inode *, struct inode*); | 643 | void lock_two_nondirectories(struct inode *, struct inode*); |