ocfs2: optimize bad declarations and redundant assignment
authorNorton.Zhu <norton.zhu@huawei.com>
Thu, 14 Jan 2016 23:17:09 +0000 (15:17 -0800)
committerLinus Torvalds <torvalds@linux-foundation.org>
Fri, 15 Jan 2016 00:00:49 +0000 (16:00 -0800)
In ocfs2_parse_options,

a) it's better to declare variables(small size) outside of while loop;

b) 'option' will be set by match_int, 'option = 0;' makes no sense, if
   match_int failed, it just goto bail and return.

Signed-off-by: Norton.Zhu <norton.zhu@huawei.com>
Reviewed-by: Joseph Qi <joseph.qi@huawei.com>
Cc: Gang He <ghe@suse.com>
Cc: Mark Fasheh <mfasheh@suse.de>
Acked-by: Joel Becker <jlbec@evilplan.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
fs/ocfs2/super.c

index 2de4c8a9340c267a16381faacbd0ce66c18d123f..64dcd787fa8043f87ccab4a49233c79a7a110402 100644 (file)
@@ -1280,6 +1280,8 @@ static int ocfs2_parse_options(struct super_block *sb,
        int status, user_stack = 0;
        char *p;
        u32 tmp;
+       int token, option;
+       substring_t args[MAX_OPT_ARGS];
 
        trace_ocfs2_parse_options(is_remount, options ? options : "(none)");
 
@@ -1298,9 +1300,6 @@ static int ocfs2_parse_options(struct super_block *sb,
        }
 
        while ((p = strsep(&options, ",")) != NULL) {
-               int token, option;
-               substring_t args[MAX_OPT_ARGS];
-
                if (!*p)
                        continue;
 
@@ -1367,7 +1366,6 @@ static int ocfs2_parse_options(struct super_block *sb,
                                mopt->atime_quantum = option;
                        break;
                case Opt_slot:
-                       option = 0;
                        if (match_int(&args[0], &option)) {
                                status = 0;
                                goto bail;
@@ -1376,7 +1374,6 @@ static int ocfs2_parse_options(struct super_block *sb,
                                mopt->slot = (s16)option;
                        break;
                case Opt_commit:
-                       option = 0;
                        if (match_int(&args[0], &option)) {
                                status = 0;
                                goto bail;
@@ -1388,7 +1385,6 @@ static int ocfs2_parse_options(struct super_block *sb,
                        mopt->commit_interval = HZ * option;
                        break;
                case Opt_localalloc:
-                       option = 0;
                        if (match_int(&args[0], &option)) {
                                status = 0;
                                goto bail;