Re: [PATCH] drivers/idle: make intel_idle.c driver more explicitly non-modular

2016-04-07 Thread Daniel Lezcano
On Tue, Apr 05, 2016 at 09:33:12AM +0200, rcoch...@linutronix.de wrote: > On Tue, Apr 05, 2016 at 05:53:47AM +, Brown, Len wrote: > > > On Tue, Apr 05, 2016 at 04:20:47AM +, Brown, Len wrote: > > > > No, I do not believe that cpuidle should bother > > > > supporting changing idle drivers at

Re: [PATCH] drivers/idle: make intel_idle.c driver more explicitly non-modular

2016-04-05 Thread Paul Gortmaker
[Re: [PATCH] drivers/idle: make intel_idle.c driver more explicitly non-modular] On 05/04/2016 (Tue 05:11) rcoch...@linutronix.de wrote: > On Mon, Apr 04, 2016 at 03:55:35PM -0400, Paul Gortmaker wrote: > > > This was done in commit 6ce9cd8669fa1195fdc21643370e34523c7ac988 > &

Re: [PATCH] drivers/idle: make intel_idle.c driver more explicitly non-modular

2016-04-05 Thread rcochran
On Tue, Apr 05, 2016 at 05:53:47AM +, Brown, Len wrote: > > On Tue, Apr 05, 2016 at 04:20:47AM +, Brown, Len wrote: > > > No, I do not believe that cpuidle should bother > > > supporting changing idle drivers at run-time. > > > > Huh? But you just said, "it would be good to be able to unl

RE: [PATCH] drivers/idle: make intel_idle.c driver more explicitly non-modular

2016-04-04 Thread Brown, Len
H] drivers/idle: make intel_idle.c driver more > explicitly non-modular > > On Tue, Apr 05, 2016 at 04:20:47AM +, Brown, Len wrote: > > The first idle driver to register with cpuidle wins. > > > > intel_idle should always get the opportunity > > to probe and reg

Re: [PATCH] drivers/idle: make intel_idle.c driver more explicitly non-modular

2016-04-04 Thread rcochran
On Tue, Apr 05, 2016 at 04:20:47AM +, Brown, Len wrote: > The first idle driver to register with cpuidle wins. > > intel_idle should always get the opportunity > to probe and register before acpi_idle (processor_idle.c) > > When intel_idle was allowed to be modular, > some distros build their

RE: [PATCH] drivers/idle: make intel_idle.c driver more explicitly non-modular

2016-04-04 Thread Brown, Len
The first idle driver to register with cpuidle wins. intel_idle should always get the opportunity to probe and register before acpi_idle (processor_idle.c) When intel_idle was allowed to be modular, some distros build their kernel and loaded modules such that acpi_idle could probe first. In such

Re: [PATCH] drivers/idle: make intel_idle.c driver more explicitly non-modular

2016-04-04 Thread rcochran
On Mon, Apr 04, 2016 at 03:55:35PM -0400, Paul Gortmaker wrote: > > This was done in commit 6ce9cd8669fa1195fdc21643370e34523c7ac988 > > ("intel_idle: disable module support") since "...the module capability > > is cauing more trouble than it is worth." The reason given in that commit was that "it

Re: [PATCH] drivers/idle: make intel_idle.c driver more explicitly non-modular

2016-04-04 Thread Paul Gortmaker
[[PATCH] drivers/idle: make intel_idle.c driver more explicitly non-modular] On 27/03/2016 (Sun 13:29) Paul Gortmaker wrote: > The Kconfig for this driver is currently declared with: > > config INTEL_IDLE > bool "Cpuidle Driver for Intel Processors" > > ...meaning that it currently is n