On 10 April 2013 00:00, Tejun Heo tj@kernel.org wrote:
On Tue, Apr 09, 2013 at 01:00:59PM +0530, Viresh Kumar wrote:
+#ifdef CONFIG_WQ_POWER_EFFICIENT +static bool wq_power_efficient = 0; +module_param_named(power_efficient, wq_power_efficient, bool, 0444); +#endif
I don't think we need to make the whole thing configurable. Turning it off isn't gonna save much - my gut tells me it's gonna be four instructions. :)
What I meant was that the default value for the parameter would probably be need to be configurable so that mobile people don't have to include the kernel param all the time or patch the kernel themselves.
I didn't get it completely.. Are you asking to set the default value to 1 instead to keep it enabled by default if config option is selected?