All of lore.kernel.org
 help / color / mirror / Atom feed
* [Ocfs2-devel] [PATCH] ocfs2: Optimize bad declarations and redundant assignment
@ 2015-11-16  2:06 Norton.Zhu
  2015-11-16  3:14 ` Joseph Qi
  0 siblings, 1 reply; 3+ messages in thread
From: Norton.Zhu @ 2015-11-16  2:06 UTC (permalink / raw)
  To: ocfs2-devel

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>
---
 fs/ocfs2/super.c | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/fs/ocfs2/super.c b/fs/ocfs2/super.c
index 403c566..8a6b48f 100644
--- a/fs/ocfs2/super.c
+++ b/fs/ocfs2/super.c
@@ -1278,6 +1278,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)");

@@ -1296,9 +1298,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;

@@ -1356,7 +1355,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;
@@ -1365,7 +1363,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;
@@ -1377,7 +1374,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;
-- 
1.8.4.3

^ permalink raw reply related	[flat|nested] 3+ messages in thread
* [Ocfs2-devel] [PATCH] ocfs2: Optimize bad declarations and redundant assignment
  2015-11-16  2:06 [Ocfs2-devel] [PATCH] ocfs2: Optimize bad declarations and redundant assignment Norton.Zhu
@ 2015-11-16  3:14 ` Joseph Qi
  2015-11-16  6:54   ` Gang He
  0 siblings, 1 reply; 3+ messages in thread
From: Joseph Qi @ 2015-11-16  3:14 UTC (permalink / raw)
  To: ocfs2-devel

On 2015/11/16 10:06, Norton.Zhu wrote:
> 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>

> ---
>  fs/ocfs2/super.c | 8 ++------
>  1 file changed, 2 insertions(+), 6 deletions(-)
> 
> diff --git a/fs/ocfs2/super.c b/fs/ocfs2/super.c
> index 403c566..8a6b48f 100644
> --- a/fs/ocfs2/super.c
> +++ b/fs/ocfs2/super.c
> @@ -1278,6 +1278,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)");
> 
> @@ -1296,9 +1298,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;
> 
> @@ -1356,7 +1355,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;
> @@ -1365,7 +1363,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;
> @@ -1377,7 +1374,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;
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread
* [Ocfs2-devel] [PATCH] ocfs2: Optimize bad declarations and redundant assignment
  2015-11-16  3:14 ` Joseph Qi
@ 2015-11-16  6:54   ` Gang He
  0 siblings, 0 replies; 3+ messages in thread
From: Gang He @ 2015-11-16  6:54 UTC (permalink / raw)
  To: ocfs2-devel

Thanks a lot.
Gang


>>> 
> On 2015/11/16 10:06, Norton.Zhu wrote:
>> 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>
Reviewed-by: Gang He <ghe@suse.com>

> 
>> ---
>>  fs/ocfs2/super.c | 8 ++------
>>  1 file changed, 2 insertions(+), 6 deletions(-)
>> 
>> diff --git a/fs/ocfs2/super.c b/fs/ocfs2/super.c
>> index 403c566..8a6b48f 100644
>> --- a/fs/ocfs2/super.c
>> +++ b/fs/ocfs2/super.c
>> @@ -1278,6 +1278,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)");
>> 
>> @@ -1296,9 +1298,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;
>> 
>> @@ -1356,7 +1355,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;
>> @@ -1365,7 +1363,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;
>> @@ -1377,7 +1374,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;
>> 
> 
> 
> 
> _______________________________________________
> Ocfs2-devel mailing list
> Ocfs2-devel at oss.oracle.com 
> https://oss.oracle.com/mailman/listinfo/ocfs2-devel

^ permalink raw reply	[flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-11-16  6:54 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-11-16  2:06 [Ocfs2-devel] [PATCH] ocfs2: Optimize bad declarations and redundant assignment Norton.Zhu
2015-11-16  3:14 ` Joseph Qi
2015-11-16  6:54   ` Gang He

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.