Re: [PATCH v2] Btrfs: clear compress-force when remounting with compress option
On Mon, Jun 30, 2014 at 10:51:25AM +0800, Wang Shilong wrote: Steps to reproduce: # mkfs.btrfs -f /dev/sdb # mount /dev/sdb /mnt -o compress-force=lzo # mount /dev/sdb /mnt -o remount,compress=zlib # cat /proc/mounts Remounting from compress-force to compress could not clear compress-force option. The problem is there is no way for users to clear compress-force option separately. Fix this problem by clearing @FORCE_COMPRESS flag when remounting to compress=xxx. Suggested-by: Tsutomu Itoh t-i...@jp.fujitsu.com Signed-off-by: Wang Shilong wangsl.f...@cn.fujitsu.com I'm sorry for lack of answer in the V1 thread, V2 is the minimal bugfix and I'm fine with it right away. Reviewed-by: David Sterba dste...@suse.cz -- To unsubscribe from this list: send the line unsubscribe linux-btrfs in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2] Btrfs: clear compress-force when remounting with compress option
(2014/07/02 1:04), David Sterba wrote: On Mon, Jun 30, 2014 at 10:51:25AM +0800, Wang Shilong wrote: Steps to reproduce: # mkfs.btrfs -f /dev/sdb # mount /dev/sdb /mnt -o compress-force=lzo # mount /dev/sdb /mnt -o remount,compress=zlib # cat /proc/mounts Remounting from compress-force to compress could not clear compress-force option. The problem is there is no way for users to clear compress-force option separately. Fix this problem by clearing @FORCE_COMPRESS flag when remounting to compress=xxx. Suggested-by: Tsutomu Itoh t-i...@jp.fujitsu.com Signed-off-by: Wang Shilong wangsl.f...@cn.fujitsu.com I'm sorry for lack of answer in the V1 thread, V2 is the minimal bugfix and I'm fine with it right away. Reviewed-by: David Sterba dste...@suse.cz Test result (I used your reproducer): 3.16-rc3: FAIL 3.16-rc3 w/ your patch: PASS Reviewed-by: Satoru Takeuchi takeuchi_sat...@jp.fujitsu.com Tested-by: Satoru Takeuchi takeuchi_sat...@jp.fujitsu.com Thanks, Satoru -- To unsubscribe from this list: send the line unsubscribe linux-btrfs in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line unsubscribe linux-btrfs in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2] Btrfs: clear compress-force when remounting with compress option
On 07/02/2014 09:58 AM, Satoru Takeuchi wrote: (2014/07/02 1:04), David Sterba wrote: On Mon, Jun 30, 2014 at 10:51:25AM +0800, Wang Shilong wrote: Steps to reproduce: # mkfs.btrfs -f /dev/sdb # mount /dev/sdb /mnt -o compress-force=lzo # mount /dev/sdb /mnt -o remount,compress=zlib # cat /proc/mounts Remounting from compress-force to compress could not clear compress-force option. The problem is there is no way for users to clear compress-force option separately. Fix this problem by clearing @FORCE_COMPRESS flag when remounting to compress=xxx. Suggested-by: Tsutomu Itoh t-i...@jp.fujitsu.com Signed-off-by: Wang Shilong wangsl.f...@cn.fujitsu.com I'm sorry for lack of answer in the V1 thread, V2 is the minimal bugfix and I'm fine with it right away. Reviewed-by: David Sterba dste...@suse.cz Test result (I used your reproducer): 3.16-rc3: FAIL 3.16-rc3 w/ your patch: PASS Reviewed-by: Satoru Takeuchi takeuchi_sat...@jp.fujitsu.com Tested-by: Satoru Takeuchi takeuchi_sat...@jp.fujitsu.com Satoru, Thanks for doing this. ^_^ Regards, Wang Thanks, Satoru -- To unsubscribe from this list: send the line unsubscribe linux-btrfs in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html . -- To unsubscribe from this list: send the line unsubscribe linux-btrfs in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH v2] Btrfs: clear compress-force when remounting with compress option
Steps to reproduce: # mkfs.btrfs -f /dev/sdb # mount /dev/sdb /mnt -o compress-force=lzo # mount /dev/sdb /mnt -o remount,compress=zlib # cat /proc/mounts Remounting from compress-force to compress could not clear compress-force option. The problem is there is no way for users to clear compress-force option separately. Fix this problem by clearing @FORCE_COMPRESS flag when remounting to compress=xxx. Suggested-by: Tsutomu Itoh t-i...@jp.fujitsu.com Signed-off-by: Wang Shilong wangsl.f...@cn.fujitsu.com --- v1-v2: remove unnecessary message output which was addressed by David. --- fs/btrfs/super.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/fs/btrfs/super.c b/fs/btrfs/super.c index 4662d92..5aae8f0 100644 --- a/fs/btrfs/super.c +++ b/fs/btrfs/super.c @@ -513,6 +513,13 @@ int btrfs_parse_options(struct btrfs_root *root, char *options) btrfs_info(root-fs_info, btrfs: use %s compression, compress_type); + /* +* If we remount from compress-force=xxx to +* compress=xxx, we need clear FORCE_COMPRESS +* flag, otherwise, there is no way for users +* to disable forcible compression separately. +*/ + btrfs_clear_opt(info-mount_opt, FORCE_COMPRESS); } break; case Opt_ssd: -- 1.8.3.1 -- To unsubscribe from this list: send the line unsubscribe linux-btrfs in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html