aboutsummaryrefslogtreecommitdiffstats
path: root/fs
diff options
context:
space:
mode:
authorChris Mason <clm@fb.com>2016-01-29 11:19:37 -0500
committerChris Mason <clm@fb.com>2016-01-29 11:19:37 -0500
commite410e34fad913dd568ec28d2a9949694324c14db (patch)
treec44bcb7e4a1844a17ad960fc7bfdcb73ff8485c5 /fs
parente1c0ebad3f77387c6684f8b7e86a4bbaca7577ac (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.c7
-rw-r--r--fs/btrfs/ioctl.c4
-rw-r--r--fs/btrfs/super.c4
-rw-r--r--fs/btrfs/volumes.c2
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
28static int __add_block_group_free_space(struct btrfs_trans_handle *trans, 27static 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);
4075out: 4071out:
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) \