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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 16 Jul 2014 20:48:07 -0400 From: Pranith Kumar <bobby.prani@...il.com> To: paulmck@...ux.vnet.ibm.com CC: Josh Triplett <josh@...htriplett.org>, Steven Rostedt <rostedt@...dmis.org>, Mathieu Desnoyers <mathieu.desnoyers@...icios.com>, Lai Jiangshan <laijs@...fujitsu.com>, Andrew Morton <akpm@...ux-foundation.org>, Frederic Weisbecker <fweisbec@...il.com>, Michal Hocko <mhocko@...e.cz>, Matt Turner <mattst88@...il.com>, Paul Gortmaker <paul.gortmaker@...driver.com>, 蔡正龙 <zhenglong.cai@...c.com.cn>, Kevin Hilman <khilman@...aro.org>, Tejun Heo <tj@...nel.org>, Fabian Frederick <fabf@...net.be>, David Howells <dhowells@...hat.com>, Peter Foley <pefoley2@...oley.com>, Eric Paris <eparis@...hat.com>, Dwight Engen <dwight.engen@...cle.com>, "Eric W. Biederman" <ebiederm@...ssion.com>, "H. Peter Anvin" <hpa@...or.com>, open list <linux-kernel@...r.kernel.org> Subject: Re: [RFC PATCH 1/1] rcu: Allow user to ovveride RCU_NOCB_CPU_ALL at boot time On 07/16/2014 08:26 PM, Paul E. McKenney wrote: > On Wed, Jul 16, 2014 at 06:38:08PM -0400, Pranith Kumar wrote: >> A kernel built with RCU_NOCB_CPU_ALL build time option will offload callbacks >> from all CPUs. The user cannot override this behavior without recompiling the >> kernel with the RCU_NOCB_CPU_ALL option turned off. >> >> This commit allows the user to override the build-time option by using the >> rcu_nocbs= boot time option without needing to recompile the kernel. >> >> Please note that this is how NO_HZ_FULL_ALL build time option works and this >> commit makes it work similar to that. >> >> Signed-off-by: Pranith Kumar <bobby.prani@...il.com> Hi Paul, > I cannot accept this patch. For one thing, tick_nohz_init_all() looks > a bit on the unconditional side when CONFIG_NO_HZ_FULL_ALL=y. For > another thing, we really do not want to be handing the user a tool that > allows CPUs that are nohz_full to not be no-CBs CPUs. For another thing, I thought the latest patch does not allow that by ORing the nohz_full and rcu_nocbs mask. Doesn't it? All nohz_full CPUs will be nocb CPUS. We can mention this explicitly in the kernel-parameters.txt file. > if we add this and it turns out to be a bad idea, it will be difficult > to take it back -- someone somewhere will no doubt have scripted the > boot parameter. This option already exists in the kernel when RCU_NOCB_CPU_ALL is not set. The user can pass in rcu_nocbs= at boot time. I am not sure we are adding anything new with this patch. Most of the distro kernels set RCU_NOCB_CPU_ALL and rightly so, as it is the suggested and most appropriate option. This patch will make it easier for users who want to specify nocb CPUs for their needs, without having to recompile the kernel. > > Thanx, Paul > > -- Pranith -- 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/
Powered by blists - more mailing lists