Unverified Commit b6f2a7b1 authored Nov 19, 2019 by Chengguang Xu Committed by Michael Bestas Apr 16, 2024

erofs: remove unnecessary output in erofs_show_options()

We have already handled cache_strategy option carefully,
so incorrect setting could not pass option parsing.
Meanwhile, print 'cache_strategy=(unknown)' can cause
failure on remount.

Link: https://lore.kernel.org/r/20191119115049.3401-1-cgxu519@mykernel.net


Change-Id: Ic63fcce3f5f0aca5eef37f5b738149f15444c3a8
Signed-off-by: Chengguang Xu <cgxu519@mykernel.net>
Reviewed-by: Gao Xiang <gaoxiang25@huawei.com>
Reviewed-by: Chao Yu <yuchao0@huawei.com>
Signed-off-by: Gao Xiang <gaoxiang25@huawei.com>

parent 6ed32473

fs/erofs/super.c

+0 −3

Original line number	Diff line number	Diff line
		@@ -598,9 +598,6 @@ static int erofs_show_options(struct seq_file seq, struct dentry root)
		seq_puts(seq, ",cache_strategy=readahead");
		} else if (sbi->cache_strategy == EROFS_ZIP_CACHE_READAROUND) {
		seq_puts(seq, ",cache_strategy=readaround");
		} else {
		seq_puts(seq, ",cache_strategy=(unknown)");
		DBG_BUGON(1);
		}
		#endif
		return 0;