linux-kernel - Re: [PATCH] rcu: Fix the code style issue in rcutree_trace

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-ID: <20121029094903.GB3027@linux.vnet.ibm.com>
Date:	Mon, 29 Oct 2012 02:49:03 -0700
From:	"Paul E. McKenney" <paulmck@...ux.vnet.ibm.com>
To:	Michael Wang <wangyun@...ux.vnet.ibm.com>
Cc:	linux-kernel@...r.kernel.org, dipankar@...ibm.com,
	fengguang.wu@...el.com
Subject: Re: [PATCH] rcu: Fix the code style issue in rcutree_trace_init()

On Mon, Oct 29, 2012 at 11:02:13AM +0800, Michael Wang wrote:
> This patch fix the code style issue caused by:
> 
> 	commit 622afdb3bd6288837a37bac137dfeff4e771798d
> 	"rcu: Optimize the 'rcudata' for RCU trace"

Queued, thank you Michael.

							Thanx, Paul

> Signed-off-by: Michael Wang <wangyun@...ux.vnet.ibm.com>
> Reported-by: Fengguang Wu <fengguang.wu@...el.com>
> ---
>  kernel/rcutree_trace.c |   60 ++++++++++++++++++++++++------------------------
>  1 files changed, 30 insertions(+), 30 deletions(-)
> 
> diff --git a/kernel/rcutree_trace.c b/kernel/rcutree_trace.c
> index 73abfb0..e84679c 100644
> --- a/kernel/rcutree_trace.c
> +++ b/kernel/rcutree_trace.c
> @@ -436,44 +436,44 @@ static int __init rcutree_trace_init(void)
>  		if (!rspdir)
>  			goto free_out;
> 
> -			retval = debugfs_create_file("rcudata", 0444,
> -					rspdir, rsp, &rcudata_fops);
> -			if (!retval)
> -				goto free_out;
> +		retval = debugfs_create_file("rcudata", 0444,
> +				rspdir, rsp, &rcudata_fops);
> +		if (!retval)
> +			goto free_out;
> 
> -			retval = debugfs_create_file("rcuexp", 0444,
> -					rspdir, rsp, &rcuexp_fops);
> -			if (!retval)
> -				goto free_out;
> +		retval = debugfs_create_file("rcuexp", 0444,
> +				rspdir, rsp, &rcuexp_fops);
> +		if (!retval)
> +			goto free_out;
> 
> -			retval = debugfs_create_file("rcu_pending", 0444,
> -					rspdir, rsp, &rcu_pending_fops);
> -			if (!retval)
> -				goto free_out;
> +		retval = debugfs_create_file("rcu_pending", 0444,
> +				rspdir, rsp, &rcu_pending_fops);
> +		if (!retval)
> +			goto free_out;
> 
> -			retval = debugfs_create_file("rcubarrier", 0444,
> -					rspdir, rsp, &rcubarrier_fops);
> -			if (!retval)
> -				goto free_out;
> +		retval = debugfs_create_file("rcubarrier", 0444,
> +				rspdir, rsp, &rcubarrier_fops);
> +		if (!retval)
> +			goto free_out;
> 
>  #ifdef CONFIG_RCU_BOOST
> -			if (rsp == &rcu_preempt_state) {
> -				retval = debugfs_create_file("rcuboost", 0444,
> -					rspdir, NULL, &rcu_node_boost_fops);
> -				if (!retval)
> -					goto free_out;
> -			}
> -#endif
> -
> -			retval = debugfs_create_file("rcugp", 0444,
> -					rspdir, rsp, &rcugp_fops);
> +		if (rsp == &rcu_preempt_state) {
> +			retval = debugfs_create_file("rcuboost", 0444,
> +				rspdir, NULL, &rcu_node_boost_fops);
>  			if (!retval)
>  				goto free_out;
> +		}
> +#endif
> 
> -			retval = debugfs_create_file("rcuhier", 0444,
> -					rspdir, rsp, &rcuhier_fops);
> -			if (!retval)
> -				goto free_out;
> +		retval = debugfs_create_file("rcugp", 0444,
> +				rspdir, rsp, &rcugp_fops);
> +		if (!retval)
> +			goto free_out;
> +
> +		retval = debugfs_create_file("rcuhier", 0444,
> +				rspdir, rsp, &rcuhier_fops);
> +		if (!retval)
> +			goto free_out;
>  	}
> 
>  	retval = debugfs_create_file("rcutorture", 0444, rcudir,
> -- 
> 1.7.4.1
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/