fs/affs/super.c: use affs_set_opt()
authorFabian Frederick <fabf@skynet.be>
Thu, 16 Apr 2015 19:48:21 +0000 (12:48 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Fri, 17 Apr 2015 13:04:08 +0000 (09:04 -0400)
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>
fs/affs/super.c

index 5623b494bdf519d44d321b5f396c2ab0320c975d..32bfad4234f782216a1174a4e4e0dad2396a80d9 100644 (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 @@ got_root:
        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: