Re: [PATCH] powernow-k8: depend on acpi-processor for SMP systems

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Dave Jones wrote:
On Thu, May 17, 2007 at 02:13:42PM -0400, Len Brown wrote:

 > > Index: linux/arch/i386/kernel/cpu/cpufreq/Kconfig
 > > ===================================================================
 > > --- linux.orig/arch/i386/kernel/cpu/cpufreq/Kconfig
 > > +++ linux/arch/i386/kernel/cpu/cpufreq/Kconfig
 > > @@ -81,6 +81,7 @@ config X86_POWERNOW_K7_ACPI
 > >  config X86_POWERNOW_K8
 > >  	tristate "AMD Opteron/Athlon64 PowerNow!"
 > >  	select CPU_FREQ_TABLE
 > > +	select ACPI_PROCESSOR if SMP
 > >  	depends on EXPERIMENTAL
 > >  	help
 > >  	  This adds the CPUFreq driver for mobile AMD Opteron/Athlon64 processors.
> > Unfortunately this patch will not actually enable ACPI_PROCESSOR in
 > the SMP=y ACPI=n case.  "select" doesn't work for targets that
 > have dependencies.

I don't think we can fix this perfectly tbh, but the above at
least gets us close for the majority of users.

Are there many x86-64 users that don't enable acpi ?

	Dave

I've just always compiled acpi_processor in, it's only logical that if you are using a power management feature, that you compile in the power management interface, and if your stuff deals directly with the cpu, you may want to compile the acpi_processor driver in. The only reason I knew to do that though, was because i go through each option. Someone else looking to just enable cpufreq, would skip the sub-drivers of ACPI, and never know better. We dont suggest anywhere in the cpufreq driver, we dont mention restrictions or limits of the driver without acpi, and we certainly dont select it or make it dependent (except silently and invisibly to the user). Every other cpufreq driver demands acpi. In windows you have to have acpi, the p states are called acpi p states everywhere. The problem here is that the author to the powernow_k8 driver found a way to get some cpufreq functionality without acpi. So to make everyone happy, maybe we should have the silently selected/deselected driver exposed to the user, as a sub-driver.
->  Powernow K8 / athlon64  cpufreq driver     y/m/n
------->    ACPI support      y/m/n


we do this silently in the Kconfig, i dont see the problem with exposing it so we dont have to muck around forcing people to use acpi that dont need to and selecting drivers they may not really need. -
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux