diff options
author | Josef Bacik <jbacik@fusionio.com> | 2013-04-17 12:16:59 -0400 |
---|---|---|
committer | Josef Bacik <jbacik@fusionio.com> | 2013-05-06 15:55:00 -0400 |
commit | 62dbd7176e196cd042c5542696981b268264fe92 (patch) | |
tree | 90992e6775549b70fa3aa106a40c25d964dc78c2 /fs/btrfs/super.c | |
parent | 534e6623b7bc03ddcf6c98b9398ff512d5135fe5 (diff) | |
download | op-kernel-dev-62dbd7176e196cd042c5542696981b268264fe92.zip op-kernel-dev-62dbd7176e196cd042c5542696981b268264fe92.tar.gz |
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>
Diffstat (limited to 'fs/btrfs/super.c')
-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 5dc54ae..a4807ce 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)) { |