Commit 2b0ce2c2 authored by Mitch Harder's avatar Mitch Harder Committed by Chris Mason

Btrfs: Check INCOMPAT flags on remount and add helper function

In support of the recently added capability to remount with lzo
compression, provide a helper function to check the compression
INCOMPAT flags when remounting with lzo compression, and set
the flags if necessary.

Also, implement the new helper function when defragmenting with
explicit lzo compression and when setting the default subvolume.
Signed-off-by: default avatarMitch Harder <mitch.harder@sabayonlinux.org>
Signed-off-by: default avatarChris Mason <chris.mason@fusionio.com>
parent b478b2ba
...@@ -3288,6 +3288,23 @@ void __btrfs_abort_transaction(struct btrfs_trans_handle *trans, ...@@ -3288,6 +3288,23 @@ void __btrfs_abort_transaction(struct btrfs_trans_handle *trans,
struct btrfs_root *root, const char *function, struct btrfs_root *root, const char *function,
unsigned int line, int errno); unsigned int line, int errno);
#define btrfs_set_fs_incompat(__fs_info, opt) \
__btrfs_set_fs_incompat((__fs_info), BTRFS_FEATURE_INCOMPAT_##opt)
static inline void __btrfs_set_fs_incompat(struct btrfs_fs_info *fs_info,
u64 flag)
{
struct btrfs_super_block *disk_super;
u64 features;
disk_super = fs_info->super_copy;
features = btrfs_super_incompat_flags(disk_super);
if (!(features & flag)) {
features |= flag;
btrfs_set_super_incompat_flags(disk_super, features);
}
}
#define btrfs_abort_transaction(trans, root, errno) \ #define btrfs_abort_transaction(trans, root, errno) \
do { \ do { \
__btrfs_abort_transaction(trans, root, __func__, \ __btrfs_abort_transaction(trans, root, __func__, \
......
...@@ -1053,11 +1053,9 @@ int btrfs_defrag_file(struct inode *inode, struct file *file, ...@@ -1053,11 +1053,9 @@ int btrfs_defrag_file(struct inode *inode, struct file *file,
u64 newer_than, unsigned long max_to_defrag) u64 newer_than, unsigned long max_to_defrag)
{ {
struct btrfs_root *root = BTRFS_I(inode)->root; struct btrfs_root *root = BTRFS_I(inode)->root;
struct btrfs_super_block *disk_super;
struct file_ra_state *ra = NULL; struct file_ra_state *ra = NULL;
unsigned long last_index; unsigned long last_index;
u64 isize = i_size_read(inode); u64 isize = i_size_read(inode);
u64 features;
u64 last_len = 0; u64 last_len = 0;
u64 skip = 0; u64 skip = 0;
u64 defrag_end = 0; u64 defrag_end = 0;
...@@ -1244,11 +1242,8 @@ int btrfs_defrag_file(struct inode *inode, struct file *file, ...@@ -1244,11 +1242,8 @@ int btrfs_defrag_file(struct inode *inode, struct file *file,
mutex_unlock(&inode->i_mutex); mutex_unlock(&inode->i_mutex);
} }
disk_super = root->fs_info->super_copy;
features = btrfs_super_incompat_flags(disk_super);
if (range->compress_type == BTRFS_COMPRESS_LZO) { if (range->compress_type == BTRFS_COMPRESS_LZO) {
features |= BTRFS_FEATURE_INCOMPAT_COMPRESS_LZO; btrfs_set_fs_incompat(root->fs_info, COMPRESS_LZO);
btrfs_set_super_incompat_flags(disk_super, features);
} }
ret = defrag_count; ret = defrag_count;
...@@ -2784,8 +2779,6 @@ static long btrfs_ioctl_default_subvol(struct file *file, void __user *argp) ...@@ -2784,8 +2779,6 @@ static long btrfs_ioctl_default_subvol(struct file *file, void __user *argp)
struct btrfs_path *path; struct btrfs_path *path;
struct btrfs_key location; struct btrfs_key location;
struct btrfs_disk_key disk_key; struct btrfs_disk_key disk_key;
struct btrfs_super_block *disk_super;
u64 features;
u64 objectid = 0; u64 objectid = 0;
u64 dir_id; u64 dir_id;
...@@ -2836,12 +2829,7 @@ static long btrfs_ioctl_default_subvol(struct file *file, void __user *argp) ...@@ -2836,12 +2829,7 @@ static long btrfs_ioctl_default_subvol(struct file *file, void __user *argp)
btrfs_mark_buffer_dirty(path->nodes[0]); btrfs_mark_buffer_dirty(path->nodes[0]);
btrfs_free_path(path); btrfs_free_path(path);
disk_super = root->fs_info->super_copy; btrfs_set_fs_incompat(root->fs_info, DEFAULT_SUBVOL);
features = btrfs_super_incompat_flags(disk_super);
if (!(features & BTRFS_FEATURE_INCOMPAT_DEFAULT_SUBVOL)) {
features |= BTRFS_FEATURE_INCOMPAT_DEFAULT_SUBVOL;
btrfs_set_super_incompat_flags(disk_super, features);
}
btrfs_end_transaction(trans, root); btrfs_end_transaction(trans, root);
return 0; return 0;
......
...@@ -401,6 +401,7 @@ int btrfs_parse_options(struct btrfs_root *root, char *options) ...@@ -401,6 +401,7 @@ int btrfs_parse_options(struct btrfs_root *root, char *options)
compress_type = "lzo"; compress_type = "lzo";
info->compress_type = BTRFS_COMPRESS_LZO; info->compress_type = BTRFS_COMPRESS_LZO;
btrfs_set_opt(info->mount_opt, COMPRESS); btrfs_set_opt(info->mount_opt, COMPRESS);
btrfs_set_fs_incompat(info, COMPRESS_LZO);
} else if (strncmp(args[0].from, "no", 2) == 0) { } else if (strncmp(args[0].from, "no", 2) == 0) {
compress_type = "no"; compress_type = "no";
info->compress_type = BTRFS_COMPRESS_NONE; info->compress_type = BTRFS_COMPRESS_NONE;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment