diff options
| author | Josef Bacik <jbacik@fusionio.com> | 2013-04-17 18:16:59 +0200 |
|---|---|---|
| committer | Josef Bacik <jbacik@fusionio.com> | 2013-05-06 21:55:00 +0200 |
| commit | 62dbd7176e196cd042c5542696981b268264fe92 (patch) | |
| tree | 90992e6775549b70fa3aa106a40c25d964dc78c2 | |
| parent | Btrfs: add all ioctl checks before user change for quota operations (diff) | |
| download | linux-62dbd7176e196cd042c5542696981b268264fe92.tar.xz linux-62dbd7176e196cd042c5542696981b268264fe92.zip | |
Btrfs: fix lockdep warning
The locking order for stuff is
__sb_start_write
ordered_mutex
but with sync() we don't do __sb_start_write for some strange reason, which
means that our iput in wait_ordered_extents could start a transaction which does
the __sb_start_write while we're holding the ordered_mutex. Fix this by using
delayed iput in sync. Thanks,
Reported-by: David Sterba <dsterba@suse.cz>
Signed-off-by: Josef Bacik <jbacik@fusionio.com>
| -rw-r--r-- | fs/btrfs/super.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/btrfs/super.c b/fs/btrfs/super.c index 5dc54ae5ac81..a4807ced23cc 100644 --- a/fs/btrfs/super.c +++ b/fs/btrfs/super.c @@ -866,7 +866,7 @@ int btrfs_sync_fs(struct super_block *sb, int wait) return 0; } - btrfs_wait_ordered_extents(root, 0); + btrfs_wait_ordered_extents(root, 1); trans = btrfs_attach_transaction_barrier(root); if (IS_ERR(trans)) { |