diff options
author | Chris Mason <clm@fb.com> | 2016-01-29 11:19:37 -0500 |
---|---|---|
committer | Chris Mason <clm@fb.com> | 2016-01-29 11:19:37 -0500 |
commit | e410e34fad913dd568ec28d2a9949694324c14db (patch) | |
tree | c44bcb7e4a1844a17ad960fc7bfdcb73ff8485c5 /fs | |
parent | e1c0ebad3f77387c6684f8b7e86a4bbaca7577ac (diff) |
Revert "btrfs: synchronize incompat feature bits with sysfs files"
This reverts commit 14e46e04958df740c6c6a94849f176159a333f13.
This ends up doing sysfs operations from deep in balance (where we
should be GFP_NOFS) and under heavy balance load, we're making races
against sysfs internals.
Revert it for now while we figure things out.
Signed-off-by: Chris Mason <clm@fb.com>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/btrfs/free-space-tree.c | 7 | ||||
-rw-r--r-- | fs/btrfs/ioctl.c | 4 | ||||
-rw-r--r-- | fs/btrfs/super.c | 4 | ||||
-rw-r--r-- | fs/btrfs/volumes.c | 2 |
4 files changed, 0 insertions, 17 deletions
diff --git a/fs/btrfs/free-space-tree.c b/fs/btrfs/free-space-tree.c index 81a053d2966b..53dbeaf6ce94 100644 --- a/fs/btrfs/free-space-tree.c +++ b/fs/btrfs/free-space-tree.c | |||
@@ -23,7 +23,6 @@ | |||
23 | #include "locking.h" | 23 | #include "locking.h" |
24 | #include "free-space-tree.h" | 24 | #include "free-space-tree.h" |
25 | #include "transaction.h" | 25 | #include "transaction.h" |
26 | #include "sysfs.h" | ||
27 | 26 | ||
28 | static int __add_block_group_free_space(struct btrfs_trans_handle *trans, | 27 | static int __add_block_group_free_space(struct btrfs_trans_handle *trans, |
29 | struct btrfs_fs_info *fs_info, | 28 | struct btrfs_fs_info *fs_info, |
@@ -1184,9 +1183,6 @@ int btrfs_create_free_space_tree(struct btrfs_fs_info *fs_info) | |||
1184 | } | 1183 | } |
1185 | 1184 | ||
1186 | btrfs_set_fs_compat_ro(fs_info, FREE_SPACE_TREE); | 1185 | btrfs_set_fs_compat_ro(fs_info, FREE_SPACE_TREE); |
1187 | btrfs_sysfs_feature_update(fs_info, | ||
1188 | BTRFS_FEATURE_COMPAT_RO_FREE_SPACE_TREE, FEAT_COMPAT_RO); | ||
1189 | |||
1190 | fs_info->creating_free_space_tree = 0; | 1186 | fs_info->creating_free_space_tree = 0; |
1191 | 1187 | ||
1192 | ret = btrfs_commit_transaction(trans, tree_root); | 1188 | ret = btrfs_commit_transaction(trans, tree_root); |
@@ -1255,9 +1251,6 @@ int btrfs_clear_free_space_tree(struct btrfs_fs_info *fs_info) | |||
1255 | return PTR_ERR(trans); | 1251 | return PTR_ERR(trans); |
1256 | 1252 | ||
1257 | btrfs_clear_fs_compat_ro(fs_info, FREE_SPACE_TREE); | 1253 | btrfs_clear_fs_compat_ro(fs_info, FREE_SPACE_TREE); |
1258 | btrfs_sysfs_feature_update(fs_info, | ||
1259 | BTRFS_FEATURE_COMPAT_RO_FREE_SPACE_TREE, FEAT_COMPAT_RO); | ||
1260 | |||
1261 | fs_info->free_space_root = NULL; | 1254 | fs_info->free_space_root = NULL; |
1262 | 1255 | ||
1263 | ret = clear_free_space_tree(trans, free_space_root); | 1256 | ret = clear_free_space_tree(trans, free_space_root); |
diff --git a/fs/btrfs/ioctl.c b/fs/btrfs/ioctl.c index 1568f57a3376..83c9ad3f2621 100644 --- a/fs/btrfs/ioctl.c +++ b/fs/btrfs/ioctl.c | |||
@@ -1459,8 +1459,6 @@ int btrfs_defrag_file(struct inode *inode, struct file *file, | |||
1459 | 1459 | ||
1460 | if (range->compress_type == BTRFS_COMPRESS_LZO) { | 1460 | if (range->compress_type == BTRFS_COMPRESS_LZO) { |
1461 | btrfs_set_fs_incompat(root->fs_info, COMPRESS_LZO); | 1461 | btrfs_set_fs_incompat(root->fs_info, COMPRESS_LZO); |
1462 | btrfs_sysfs_feature_update(root->fs_info, | ||
1463 | BTRFS_FEATURE_INCOMPAT_COMPRESS_LZO, FEAT_INCOMPAT); | ||
1464 | } | 1462 | } |
1465 | 1463 | ||
1466 | ret = defrag_count; | 1464 | ret = defrag_count; |
@@ -4069,8 +4067,6 @@ static long btrfs_ioctl_default_subvol(struct file *file, void __user *argp) | |||
4069 | btrfs_free_path(path); | 4067 | btrfs_free_path(path); |
4070 | 4068 | ||
4071 | btrfs_set_fs_incompat(root->fs_info, DEFAULT_SUBVOL); | 4069 | btrfs_set_fs_incompat(root->fs_info, DEFAULT_SUBVOL); |
4072 | btrfs_sysfs_feature_update(root->fs_info, | ||
4073 | BTRFS_FEATURE_INCOMPAT_DEFAULT_SUBVOL, FEAT_INCOMPAT); | ||
4074 | btrfs_end_transaction(trans, root); | 4070 | btrfs_end_transaction(trans, root); |
4075 | out: | 4071 | out: |
4076 | mnt_drop_write_file(file); | 4072 | mnt_drop_write_file(file); |
diff --git a/fs/btrfs/super.c b/fs/btrfs/super.c index 49b424103c32..a8e049ae933d 100644 --- a/fs/btrfs/super.c +++ b/fs/btrfs/super.c | |||
@@ -58,7 +58,6 @@ | |||
58 | #include "dev-replace.h" | 58 | #include "dev-replace.h" |
59 | #include "free-space-cache.h" | 59 | #include "free-space-cache.h" |
60 | #include "backref.h" | 60 | #include "backref.h" |
61 | #include "sysfs.h" | ||
62 | #include "tests/btrfs-tests.h" | 61 | #include "tests/btrfs-tests.h" |
63 | 62 | ||
64 | #include "qgroup.h" | 63 | #include "qgroup.h" |
@@ -486,9 +485,6 @@ int btrfs_parse_options(struct btrfs_root *root, char *options) | |||
486 | btrfs_clear_opt(info->mount_opt, NODATACOW); | 485 | btrfs_clear_opt(info->mount_opt, NODATACOW); |
487 | btrfs_clear_opt(info->mount_opt, NODATASUM); | 486 | btrfs_clear_opt(info->mount_opt, NODATASUM); |
488 | btrfs_set_fs_incompat(info, COMPRESS_LZO); | 487 | btrfs_set_fs_incompat(info, COMPRESS_LZO); |
489 | btrfs_sysfs_feature_update(root->fs_info, | ||
490 | BTRFS_FEATURE_INCOMPAT_COMPRESS_LZO, | ||
491 | FEAT_INCOMPAT); | ||
492 | no_compress = 0; | 488 | no_compress = 0; |
493 | } else if (strncmp(args[0].from, "no", 2) == 0) { | 489 | } else if (strncmp(args[0].from, "no", 2) == 0) { |
494 | compress_type = "no"; | 490 | compress_type = "no"; |
diff --git a/fs/btrfs/volumes.c b/fs/btrfs/volumes.c index 5b505f61929e..366b335946fa 100644 --- a/fs/btrfs/volumes.c +++ b/fs/btrfs/volumes.c | |||
@@ -4469,8 +4469,6 @@ static void check_raid56_incompat_flag(struct btrfs_fs_info *info, u64 type) | |||
4469 | return; | 4469 | return; |
4470 | 4470 | ||
4471 | btrfs_set_fs_incompat(info, RAID56); | 4471 | btrfs_set_fs_incompat(info, RAID56); |
4472 | btrfs_sysfs_feature_update(info, BTRFS_FEATURE_INCOMPAT_RAID56, | ||
4473 | FEAT_INCOMPAT); | ||
4474 | } | 4472 | } |
4475 | 4473 | ||
4476 | #define BTRFS_MAX_DEVS(r) ((BTRFS_LEAF_DATA_SIZE(r) \ | 4474 | #define BTRFS_MAX_DEVS(r) ((BTRFS_LEAF_DATA_SIZE(r) \ |