> [szepe@pinerecords.com]
>
> > > > Yes, but what I do not understand is why cpufreq need power management.
> > >
> > > CONFIG_PM is a dummy option, it does not link any code into the kernel
> > > by itself.
> >
> > Actually, it does:
> > ...
> >
> > But, I agree with your change anyway.
>
> Trouble is, the same goes for ACPI -- it doesn't require that CONFIG_PM
> code be present.
>
> I think the correct x86 solution would be to introduce a real dummy
> option for the menus, and imply CONFIG_PM if APM or swsusp (the two
> options that seem to actually need CONFIG_PM code) is enabled.
Patrick, Bruno, what is your opinion on the following replacement
patch?
-- Tomas Szepe <szepe@pinerecords.com>diff -urN a/arch/i386/Kconfig b/arch/i386/Kconfig --- a/arch/i386/Kconfig 2003-08-05 18:39:25.000000000 +0200 +++ b/arch/i386/Kconfig 2003-08-05 18:56:48.000000000 +0200 @@ -797,13 +797,13 @@ endmenu -menu "Power management options (ACPI, APM)" +menu "Power management support" depends on !X86_VOYAGER config PM - bool "Power Management support" + bool "APM and software suspend options" ---help--- - "Power Management" means that parts of your computer are shut + "Power management" provides for parts of your computer to be shut off or put into a power conserving "sleep" mode if they are not being used. There are two competing standards for doing this: APM and ACPI. If you want to use either one, say Y here and then also @@ -820,37 +820,6 @@ will issue the hlt instruction if nothing is to be done, thereby sending the processor to sleep and saving power. -config SOFTWARE_SUSPEND - bool "Software Suspend (EXPERIMENTAL)" - depends on EXPERIMENTAL && PM && SWAP - ---help--- - Enable the possibilty of suspendig machine. It doesn't need APM. - You may suspend your machine by 'swsusp' or 'shutdown -z <time>' - (patch for sysvinit needed). - - It creates an image which is saved in your active swaps. By the next - booting the, pass 'resume=/path/to/your/swap/file' and kernel will - detect the saved image, restore the memory from - it and then it continues to run as before you've suspended. - If you don't want the previous state to continue use the 'noresume' - kernel option. However note that your partitions will be fsck'd and - you must re-mkswap your swap partitions/files. - - Right now you may boot without resuming and then later resume but - in meantime you cannot use those swap partitions/files which were - involved in suspending. Also in this case there is a risk that buffers - on disk won't match with saved ones. - - SMP is supported ``as-is''. There's a code for it but doesn't work. - There have been problems reported relating SCSI. - - This option is about getting stable. However there is still some - absence of features. - - For more information take a look at Documentation/swsusp.txt. - -source "drivers/acpi/Kconfig" - config APM tristate "Advanced Power Management BIOS support" depends on PM @@ -996,6 +965,10 @@ a work-around for a number of buggy BIOSes. Switch this option on if your computer crashes instead of powering off properly. +source "kernel/Kconfig.swsusp" + +source "drivers/acpi/Kconfig" + source "arch/i386/kernel/cpu/cpufreq/Kconfig" endmenu diff -urN a/arch/i386/kernel/cpu/cpufreq/Kconfig b/arch/i386/kernel/cpu/cpufreq/Kconfig --- a/arch/i386/kernel/cpu/cpufreq/Kconfig 2003-07-10 23:30:33.000000000 +0200 +++ b/arch/i386/kernel/cpu/cpufreq/Kconfig 2003-08-05 18:42:00.000000000 +0200 @@ -2,8 +2,6 @@ # CPU Frequency scaling # -menu "CPU Frequency scaling" - config CPU_FREQ bool "CPU Frequency scaling" help @@ -16,6 +14,8 @@ If in doubt, say N. +if CPU_FREQ + source "drivers/cpufreq/Kconfig" config CPU_FREQ_TABLE @@ -162,4 +162,4 @@ If in doubt, say N. -endmenu +endif diff -urN a/arch/x86_64/Kconfig b/arch/x86_64/Kconfig --- a/arch/x86_64/Kconfig 2003-08-05 18:39:26.000000000 +0200 +++ b/arch/x86_64/Kconfig 2003-08-05 18:41:45.000000000 +0200 @@ -296,34 +296,7 @@ sending the processor to limited sleep and saving power. However using ACPI will likely save more power. -config SOFTWARE_SUSPEND - bool "Software Suspend (EXPERIMENTAL)" - depends on EXPERIMENTAL && PM && SWAP - ---help--- - Enable the possibilty of suspending the machine. It doesn't need APM. - You may suspend your machine by 'swsusp' or 'shutdown -z <time>' - (patch for sysvinit needed). - - It creates an image which is saved in your active swaps. On the next - boot, pass the 'resume=/path/to/your/swap/file' option and the kernel - will detect the saved image, restore the memory from - it, and then continue to run as before you suspended. - If you don't want the previous state to continue, use the 'noresume' - kernel option. However, note that your partitions will be fsck'd and - you must re-mkswap your swap partitions/files. - - Right now you may boot without resuming and then later resume but - in the meantime you cannot use those swap partitions/files which were - involved in suspending. Also in this case there is a risk that buffers - on disk won't match with saved ones. - - SMP is supported ``as-is''. There's code for it but doesn't work. - There have been problems reported relating to SCSI. - - This option is close to getting stable. However there is still some - absence of features. - - For more information take a look at Documentation/swsusp.txt. +source "kernel/Kconfig.swsusp" source "drivers/acpi/Kconfig" diff -urN a/drivers/acpi/Kconfig b/drivers/acpi/Kconfig --- a/drivers/acpi/Kconfig 2003-06-22 22:27:39.000000000 +0200 +++ b/drivers/acpi/Kconfig 2003-08-05 18:46:01.000000000 +0200 @@ -2,8 +2,6 @@ # ACPI Configuration # -menu "ACPI Support" - config ACPI bool "ACPI Support" if X86 depends on !X86_VISWS @@ -36,6 +34,8 @@ available at: <http://www.acpi.info> +if ACPI + config ACPI_HT_ONLY bool "CPU Enumeration Only" depends on X86 && ACPI && X86_LOCAL_APIC @@ -236,5 +236,4 @@ depends on IA64 && (!IA64_HP_SIM || IA64_SGI_SN) default y -endmenu - +endif diff -urN a/kernel/Kconfig.swsusp b/kernel/Kconfig.swsusp --- a/kernel/Kconfig.swsusp 1970-01-01 01:00:00.000000000 +0100 +++ b/kernel/Kconfig.swsusp 2003-08-05 18:41:45.000000000 +0200 @@ -0,0 +1,26 @@ +config SOFTWARE_SUSPEND + bool "Software Suspend (EXPERIMENTAL)" + depends on EXPERIMENTAL && PM && SWAP + ---help--- + Enable the possibility of suspending the machine. This feature + doesn't require APM. You may suspend your machine by 'swsusp' + or 'shutdown -z <time>' (with the appropriate patch for sysvinit). + + The suspend procedure will create an image that is saved in one + of your active swap partitions (swap files are not supported). + + On the next boot, passing the kernel boot option + 'resume=/path/to/your/swap/partition' will cause the kernel + to detect the saved image, restore system memory from it and + then continue to run as before you suspended. If you don't + want the previous state restored, use the 'noresume' kernel + boot option. However, note that in this case your partitions + will appear to be damaged and you will need to re-mkswap. + + You are not advised to boot without resuming and then resume + later. There is a high probability that buffers on disk won't + match with the saved ones. + + SMP is NOT supported at this time. + + For more information take a look at Documentation/swsusp.txt. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
This archive was generated by hypermail 2b29 : Thu Aug 07 2003 - 22:00:29 EST