fs/affs/super.c: use affs_set_opt()

Replace direct mount option assignation by affs_set_opt() macro.

Signed-off-by: Fabian Frederick <fabf@skynet.be>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
This commit is contained in:
Fabian Frederick 2015-04-16 12:48:21 -07:00 committed by Linus Torvalds
parent 6bf445ce35
commit 34f2483538
1 changed files with 16 additions and 15 deletions

View File

@ -227,22 +227,22 @@ parse_options(char *options, kuid_t *uid, kgid_t *gid, int *mode, int *reserved,
if (match_octal(&args[0], &option))
return 0;
*mode = option & 0777;
*mount_opts |= AFFS_MOUNT_SF_SETMODE;
affs_set_opt(*mount_opts, SF_SETMODE);
break;
case Opt_mufs:
*mount_opts |= AFFS_MOUNT_SF_MUFS;
affs_set_opt(*mount_opts, SF_MUFS);
break;
case Opt_notruncate:
*mount_opts |= AFFS_MOUNT_SF_NO_TRUNCATE;
affs_set_opt(*mount_opts, SF_NO_TRUNCATE);
break;
case Opt_prefix:
*prefix = match_strdup(&args[0]);
if (!*prefix)
return 0;
*mount_opts |= AFFS_MOUNT_SF_PREFIX;
affs_set_opt(*mount_opts, SF_PREFIX);
break;
case Opt_protect:
*mount_opts |= AFFS_MOUNT_SF_IMMUTABLE;
affs_set_opt(*mount_opts, SF_IMMUTABLE);
break;
case Opt_reserved:
if (match_int(&args[0], reserved))
@ -258,7 +258,7 @@ parse_options(char *options, kuid_t *uid, kgid_t *gid, int *mode, int *reserved,
*gid = make_kgid(current_user_ns(), option);
if (!gid_valid(*gid))
return 0;
*mount_opts |= AFFS_MOUNT_SF_SETGID;
affs_set_opt(*mount_opts, SF_SETGID);
break;
case Opt_setuid:
if (match_int(&args[0], &option))
@ -266,10 +266,10 @@ parse_options(char *options, kuid_t *uid, kgid_t *gid, int *mode, int *reserved,
*uid = make_kuid(current_user_ns(), option);
if (!uid_valid(*uid))
return 0;
*mount_opts |= AFFS_MOUNT_SF_SETUID;
affs_set_opt(*mount_opts, SF_SETUID);
break;
case Opt_verbose:
*mount_opts |= AFFS_MOUNT_SF_VERBOSE;
affs_set_opt(*mount_opts, SF_VERBOSE);
break;
case Opt_volume: {
char *vol = match_strdup(&args[0]);
@ -435,30 +435,31 @@ static int affs_fill_super(struct super_block *sb, void *data, int silent)
case MUFS_FS:
case MUFS_INTLFFS:
case MUFS_DCFFS:
sbi->s_flags |= AFFS_MOUNT_SF_MUFS;
affs_set_opt(sbi->s_flags, SF_MUFS);
/* fall thru */
case FS_INTLFFS:
case FS_DCFFS:
sbi->s_flags |= AFFS_MOUNT_SF_INTL;
affs_set_opt(sbi->s_flags, SF_INTL);
break;
case MUFS_FFS:
sbi->s_flags |= AFFS_MOUNT_SF_MUFS;
affs_set_opt(sbi->s_flags, SF_MUFS);
break;
case FS_FFS:
break;
case MUFS_OFS:
sbi->s_flags |= AFFS_MOUNT_SF_MUFS;
affs_set_opt(sbi->s_flags, SF_MUFS);
/* fall thru */
case FS_OFS:
sbi->s_flags |= AFFS_MOUNT_SF_OFS;
affs_set_opt(sbi->s_flags, SF_OFS);
sb->s_flags |= MS_NOEXEC;
break;
case MUFS_DCOFS:
case MUFS_INTLOFS:
sbi->s_flags |= AFFS_MOUNT_SF_MUFS;
affs_set_opt(sbi->s_flags, SF_MUFS);
case FS_DCOFS:
case FS_INTLOFS:
sbi->s_flags |= AFFS_MOUNT_SF_INTL | AFFS_MOUNT_SF_OFS;
affs_set_opt(sbi->s_flags, SF_INTL);
affs_set_opt(sbi->s_flags, SF_OFS);
sb->s_flags |= MS_NOEXEC;
break;
default: