diff options
author | Jeff Mahoney <jeffm@suse.com> | 2009-03-30 14:02:35 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-03-30 15:16:38 -0400 |
commit | d984561b326cd0fe0d1183d11b9b4fa1d011d21d (patch) | |
tree | e0487588581bccaa2b875529ed84ec5a4a254ab9 /fs/reiserfs/namei.c | |
parent | 6c17675e1e02ebde220ef639a3fb1333928ec2f4 (diff) |
reiserfs: eliminate per-super xattr lock
With the switch to using inode->i_mutex locking during lookups/creation
in the xattr root, the per-super xattr lock is no longer needed.
This patch removes it.
Signed-off-by: Jeff Mahoney <jeffm@suse.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/reiserfs/namei.c')
-rw-r--r-- | fs/reiserfs/namei.c | 29 |
1 files changed, 0 insertions, 29 deletions
diff --git a/fs/reiserfs/namei.c b/fs/reiserfs/namei.c index c8430f1c824f..ddf1bcd41c87 100644 --- a/fs/reiserfs/namei.c +++ b/fs/reiserfs/namei.c | |||
@@ -609,9 +609,6 @@ static int reiserfs_create(struct inode *dir, struct dentry *dentry, int mode, | |||
609 | 609 | ||
610 | reiserfs_write_lock(dir->i_sb); | 610 | reiserfs_write_lock(dir->i_sb); |
611 | 611 | ||
612 | if (locked) | ||
613 | reiserfs_write_lock_xattrs(dir->i_sb); | ||
614 | |||
615 | retval = journal_begin(&th, dir->i_sb, jbegin_count); | 612 | retval = journal_begin(&th, dir->i_sb, jbegin_count); |
616 | if (retval) { | 613 | if (retval) { |
617 | drop_new_inode(inode); | 614 | drop_new_inode(inode); |
@@ -624,11 +621,6 @@ static int reiserfs_create(struct inode *dir, struct dentry *dentry, int mode, | |||
624 | if (retval) | 621 | if (retval) |
625 | goto out_failed; | 622 | goto out_failed; |
626 | 623 | ||
627 | if (locked) { | ||
628 | reiserfs_write_unlock_xattrs(dir->i_sb); | ||
629 | locked = 0; | ||
630 | } | ||
631 | |||
632 | inode->i_op = &reiserfs_file_inode_operations; | 624 | inode->i_op = &reiserfs_file_inode_operations; |
633 | inode->i_fop = &reiserfs_file_operations; | 625 | inode->i_fop = &reiserfs_file_operations; |
634 | inode->i_mapping->a_ops = &reiserfs_address_space_operations; | 626 | inode->i_mapping->a_ops = &reiserfs_address_space_operations; |
@@ -655,8 +647,6 @@ static int reiserfs_create(struct inode *dir, struct dentry *dentry, int mode, | |||
655 | retval = journal_end(&th, dir->i_sb, jbegin_count); | 647 | retval = journal_end(&th, dir->i_sb, jbegin_count); |
656 | 648 | ||
657 | out_failed: | 649 | out_failed: |
658 | if (locked) | ||
659 | reiserfs_write_unlock_xattrs(dir->i_sb); | ||
660 | reiserfs_write_unlock(dir->i_sb); | 650 | reiserfs_write_unlock(dir->i_sb); |
661 | return retval; | 651 | return retval; |
662 | } | 652 | } |
@@ -686,9 +676,6 @@ static int reiserfs_mknod(struct inode *dir, struct dentry *dentry, int mode, | |||
686 | 676 | ||
687 | reiserfs_write_lock(dir->i_sb); | 677 | reiserfs_write_lock(dir->i_sb); |
688 | 678 | ||
689 | if (locked) | ||
690 | reiserfs_write_lock_xattrs(dir->i_sb); | ||
691 | |||
692 | retval = journal_begin(&th, dir->i_sb, jbegin_count); | 679 | retval = journal_begin(&th, dir->i_sb, jbegin_count); |
693 | if (retval) { | 680 | if (retval) { |
694 | drop_new_inode(inode); | 681 | drop_new_inode(inode); |
@@ -702,11 +689,6 @@ static int reiserfs_mknod(struct inode *dir, struct dentry *dentry, int mode, | |||
702 | goto out_failed; | 689 | goto out_failed; |
703 | } | 690 | } |
704 | 691 | ||
705 | if (locked) { | ||
706 | reiserfs_write_unlock_xattrs(dir->i_sb); | ||
707 | locked = 0; | ||
708 | } | ||
709 | |||
710 | inode->i_op = &reiserfs_special_inode_operations; | 692 | inode->i_op = &reiserfs_special_inode_operations; |
711 | init_special_inode(inode, inode->i_mode, rdev); | 693 | init_special_inode(inode, inode->i_mode, rdev); |
712 | 694 | ||
@@ -736,8 +718,6 @@ static int reiserfs_mknod(struct inode *dir, struct dentry *dentry, int mode, | |||
736 | retval = journal_end(&th, dir->i_sb, jbegin_count); | 718 | retval = journal_end(&th, dir->i_sb, jbegin_count); |
737 | 719 | ||
738 | out_failed: | 720 | out_failed: |
739 | if (locked) | ||
740 | reiserfs_write_unlock_xattrs(dir->i_sb); | ||
741 | reiserfs_write_unlock(dir->i_sb); | 721 | reiserfs_write_unlock(dir->i_sb); |
742 | return retval; | 722 | return retval; |
743 | } | 723 | } |
@@ -767,8 +747,6 @@ static int reiserfs_mkdir(struct inode *dir, struct dentry *dentry, int mode) | |||
767 | locked = reiserfs_cache_default_acl(dir); | 747 | locked = reiserfs_cache_default_acl(dir); |
768 | 748 | ||
769 | reiserfs_write_lock(dir->i_sb); | 749 | reiserfs_write_lock(dir->i_sb); |
770 | if (locked) | ||
771 | reiserfs_write_lock_xattrs(dir->i_sb); | ||
772 | 750 | ||
773 | retval = journal_begin(&th, dir->i_sb, jbegin_count); | 751 | retval = journal_begin(&th, dir->i_sb, jbegin_count); |
774 | if (retval) { | 752 | if (retval) { |
@@ -790,11 +768,6 @@ static int reiserfs_mkdir(struct inode *dir, struct dentry *dentry, int mode) | |||
790 | goto out_failed; | 768 | goto out_failed; |
791 | } | 769 | } |
792 | 770 | ||
793 | if (locked) { | ||
794 | reiserfs_write_unlock_xattrs(dir->i_sb); | ||
795 | locked = 0; | ||
796 | } | ||
797 | |||
798 | reiserfs_update_inode_transaction(inode); | 771 | reiserfs_update_inode_transaction(inode); |
799 | reiserfs_update_inode_transaction(dir); | 772 | reiserfs_update_inode_transaction(dir); |
800 | 773 | ||
@@ -824,8 +797,6 @@ static int reiserfs_mkdir(struct inode *dir, struct dentry *dentry, int mode) | |||
824 | unlock_new_inode(inode); | 797 | unlock_new_inode(inode); |
825 | retval = journal_end(&th, dir->i_sb, jbegin_count); | 798 | retval = journal_end(&th, dir->i_sb, jbegin_count); |
826 | out_failed: | 799 | out_failed: |
827 | if (locked) | ||
828 | reiserfs_write_unlock_xattrs(dir->i_sb); | ||
829 | reiserfs_write_unlock(dir->i_sb); | 800 | reiserfs_write_unlock(dir->i_sb); |
830 | return retval; | 801 | return retval; |
831 | } | 802 | } |