Re: [PATCH] Remove CONFIG_PM altogether, enable power management all the time

2011-02-09 Thread Mark Brown
On Tue, Feb 08, 2011 at 03:35:29PM -0800, Frank Rowand wrote: For 2.6.38-rc4, x86_64, CONFIG_NR_CPUS=4: size vmlinux text data bss dec hex filename 6553910 3555020 9994240 20103170 132c002 vmlinuxwithCONFIG_PM 6512652 3553116 9994240 20060008

Re: [PATCH] Remove CONFIG_PM altogether, enable power management all the time

2011-02-09 Thread Frank Rowand
On 02/09/11 09:07, Rafael J. Wysocki wrote: On Wednesday, February 09, 2011, Mark Brown wrote: On Tue, Feb 08, 2011 at 03:35:29PM -0800, Frank Rowand wrote: For 2.6.38-rc4, x86_64, CONFIG_NR_CPUS=4: size vmlinux text data bss dec hex filename 6553910 3555020

Re: [PATCH] Remove CONFIG_PM altogether, enable power management all the time

2011-02-09 Thread Mark Brown
On Wed, Feb 09, 2011 at 10:31:29AM -0800, Frank Rowand wrote: Raphael's patch will turn on CONFIG_PM in the correct circumstances, and leave it off when not needed by other config options. That means that the size overhead will _not_ be an issue for me because CONFIG_PM will not be enabled

Re: [PATCH] Remove CONFIG_PM altogether, enable power management all the time

2011-02-09 Thread Frank Rowand
On 02/09/11 10:40, Mark Brown wrote: On Wed, Feb 09, 2011 at 10:31:29AM -0800, Frank Rowand wrote: Raphael's patch will turn on CONFIG_PM in the correct circumstances, and leave it off when not needed by other config options. That means that the size overhead will _not_ be an issue for me

Re: [PATCH] Remove CONFIG_PM altogether, enable power management all the time

2011-02-09 Thread Mark Brown
On Wed, Feb 09, 2011 at 11:53:52AM -0800, Tim Bird wrote: On 02/09/2011 11:25 AM, Mark Brown wrote: Not really, the goal was to simplify the PM config options to ones that are actually useful and cut down on the number of silly combinations that the randconfigs turn up. CONFIG_PM is there

Re: [PATCH] Remove CONFIG_PM altogether, enable power management all the time

2011-02-09 Thread Alan Stern
On Wed, 9 Feb 2011, Mark Brown wrote: On Wed, Feb 09, 2011 at 11:53:52AM -0800, Tim Bird wrote: I'm confused. Do you plan to retain the option to turn off PM features completely, or not? I thought that's what CONFIG_PM did today. Raphael's patches do that in a much better way than my

Re: [PATCH] Remove CONFIG_PM altogether, enable power management all the time

2011-02-08 Thread Ingo Molnar
* Rafael J. Wysocki r...@sisk.pl wrote: I'd appreciate it if people could review/test it and drop their comments. Thanks, Rafael --- arch/x86/xen/Kconfig |2 +- drivers/acpi/Kconfig |1 - drivers/acpi/bus.c |4 +---

Re: [PATCH] Remove CONFIG_PM altogether, enable power management all the time

2011-02-08 Thread Tim Bird
On Tue, Feb 8, 2011 at 4:21 AM, Ingo Molnar mi...@elte.hu wrote: Rafael, could you do a defconfig-ish x86 build with and without CONFIG_PM, and post the 'size vmlinux' comparison - so that we can see the size difference? We make some things CONFIG_EXPERT configurable just to enable folks

Re: [PATCH] Remove CONFIG_PM altogether, enable power management all the time

2011-02-08 Thread Frank Rowand
On 02/08/11 04:21, Ingo Molnar wrote: snip Also, i've Cc:-ed Linus, to check whether the idea to make power management a permanent, core portion of Linux has any obvious downsides we missed. Rafael, could you do a defconfig-ish x86 build with and without CONFIG_PM, and post the 'size

Re: [PATCH] Remove CONFIG_PM altogether, enable power management all the time

2011-02-08 Thread Ingo Molnar
* Tim Bird tbird...@gmail.com wrote: On Tue, Feb 8, 2011 at 4:21 AM, Ingo Molnar mi...@elte.hu wrote: Rafael, could you do a defconfig-ish x86 build with and without CONFIG_PM, and post the 'size vmlinux' comparison - so that we can see the size difference? We make some things