exfat: implement "quiet" option for setattr

Signed-off-by: Park Ju Hyung <[email protected]>

exfat_fs.h

@@ -207,6 +207,8 @@ struct exfat_mount_options {
 	unsigned short allow_utime;
 	/* charset for filename input/display */
 	char *iocharset;
+	/* fake return success on setattr(e.g. chmods/chowns) */
+	unsigned char quiet;
 	/* on error: continue, panic, remount-ro */
 	enum exfat_error_mode errors;
 	unsigned utf8:1, /* Use of UTF-8 character set */

file.c

@@ -303,7 +303,7 @@ int exfat_setattr(struct dentry *dentry, struct iattr *attr)
 	    attr->ia_size > i_size_read(inode)) {
 		error = exfat_cont_expand(inode, attr->ia_size);
 		if (error || attr->ia_valid == ATTR_SIZE)
-			return error;
+			goto out;
 		attr->ia_valid &= ~ATTR_SIZE;
 	}
 
@@ -317,8 +317,11 @@ int exfat_setattr(struct dentry *dentry, struct iattr *attr)
 
 	error = setattr_prepare(dentry, attr);
 	attr->ia_valid = ia_valid;
-	if (error)
+	if (error) {
+		if (sbi->options.quiet)
+			error = 0;
 		goto out;
+	}
 
 	if (((attr->ia_valid & ATTR_UID) &&
 	     !uid_eq(attr->ia_uid, sbi->options.fs_uid)) ||
@@ -330,6 +333,12 @@ int exfat_setattr(struct dentry *dentry, struct iattr *attr)
 		goto out;
 	}
 
+	if (error) {
+		if (sbi->options.quiet)
+			error = 0;
+		goto out;
+	}
+
 	/*
 	 * We don't return -EPERM here. Yes, strange, but this is too
 	 * old behavior.

super.c

@@ -146,6 +146,8 @@ static int exfat_show_options(struct seq_file *m, struct dentry *root)
 	seq_printf(m, ",fmask=%04o,dmask=%04o", opts->fs_fmask, opts->fs_dmask);
 	if (opts->allow_utime)
 		seq_printf(m, ",allow_utime=%04o", opts->allow_utime);
+	if (opts->quiet)
+		seq_puts(m, ",quiet");
 	if (opts->utf8)
 		seq_puts(m, ",iocharset=utf8");
 	else if (sbi->nls_io)
@@ -203,6 +205,7 @@ enum {
 	Opt_fmask,
 	Opt_allow_utime,
 	Opt_charset,
+	Opt_quiet,
 	Opt_err_cont,
 	Opt_err_panic,
 	Opt_err_ro,
@@ -224,6 +227,7 @@ static const match_table_t exfat_tokens = {
 	{Opt_fmask, "fmask=%o"},
 	{Opt_allow_utime, "allow_utime=%o"},
 	{Opt_charset, "iocharset=%s"},
+	{Opt_quiet, "quiet"},
 	{Opt_err_cont, "errors=continue"},
 	{Opt_err_panic, "errors=panic"},
 	{Opt_err_ro, "errors=remount-ro"},
@@ -277,6 +281,9 @@ static int __exfat_parse_option(struct super_block *sb, char *p, substring_t *ar
 			return -ENOMEM;
 		opts->iocharset = tmpstr;
 		break;
+	case Opt_quiet:
+		opts->quiet = 1;
+		break;
 	case Opt_err_cont:
 		opts->errors = EXFAT_ERRORS_CONT;
 		break;