diff options
author | David Sterba <dsterba@suse.cz> | 2014-03-28 12:38:48 -0400 |
---|---|---|
committer | David Sterba <dsterba@suse.cz> | 2014-11-12 10:53:13 -0500 |
commit | 6b5fe46dfa52441f49c7432b1c1b1cb767834708 (patch) | |
tree | 331e7651b2dbad5bcfe6dca181d8992c7a365f2e /fs/btrfs/super.c | |
parent | 572d9ab7845ea0e043ec34cd733a75228130ad03 (diff) |
btrfs: do commit in sync_fs if there are pending changes
If a pending change is requested, it's not processed unless there is a
transaction commit about to happen, not even after sync or SYNC_FS
ioctl. For example a remount that toggles the inode_cache option will
not take effect after sync on a quiescent filesystem.
Signed-off-by: David Sterba <dsterba@suse.cz>
Diffstat (limited to 'fs/btrfs/super.c')
-rw-r--r-- | fs/btrfs/super.c | 14 |
1 files changed, 11 insertions, 3 deletions
diff --git a/fs/btrfs/super.c b/fs/btrfs/super.c index 54bd91ece35b..1da16d59e115 100644 --- a/fs/btrfs/super.c +++ b/fs/btrfs/super.c | |||
@@ -993,9 +993,17 @@ int btrfs_sync_fs(struct super_block *sb, int wait) | |||
993 | trans = btrfs_attach_transaction_barrier(root); | 993 | trans = btrfs_attach_transaction_barrier(root); |
994 | if (IS_ERR(trans)) { | 994 | if (IS_ERR(trans)) { |
995 | /* no transaction, don't bother */ | 995 | /* no transaction, don't bother */ |
996 | if (PTR_ERR(trans) == -ENOENT) | 996 | if (PTR_ERR(trans) == -ENOENT) { |
997 | return 0; | 997 | /* |
998 | return PTR_ERR(trans); | 998 | * Exit unless we have some pending changes |
999 | * that need to go through commit | ||
1000 | */ | ||
1001 | if (fs_info->pending_changes == 0) | ||
1002 | return 0; | ||
1003 | trans = btrfs_start_transaction(root, 0); | ||
1004 | } else { | ||
1005 | return PTR_ERR(trans); | ||
1006 | } | ||
999 | } | 1007 | } |
1000 | return btrfs_commit_transaction(trans, root); | 1008 | return btrfs_commit_transaction(trans, root); |
1001 | } | 1009 | } |