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 | |
| 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')
| -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)) { |
