On Wed, 25 Jul 2007, Len Brown wrote:
> On Wednesday 25 July 2007 14:48, Linus Torvalds wrote:
>
> > ... ACPI now seems to select CPU hotplug. Why?
>
> ACPI=y SMP=y systems require SUSPEND_SMP=y for system sleep support,
> and that requires HOTPLUG_CPU=y.
.. and why do you think I want system sleep support? That's the bug.
> Note that ACPI=y SMP=n systems do not need it,
> and thus will not select HOTPLUG_CPU=y
I want SMP, dammit. This machine has multi-core. And I want ACPI. I just
don't want the system sleep thing.
I didn't have it before, I don't need it, I don't want it.
> My assumption is that if somebody selects CONFIG_ACPI,
> that 99% of the time, they intend that to include support for
> the ACPI hooks for system sleep states.
That wasn't true before, and it makes no sense what-so-ever as an
assumption.
The system sleep states are mostly usable on laptops. Not always even
then. They are seldom used on desktops or servers, but both of those are
often SMP machines, and certainly want ACPI.
So your "99%" makes no sense.
> So today we have this:
>
> menuconfig ACPI
> ...
> select HOTPLUG_CPU if X86 && SMP
> select SUSPEND_SMP if X86 && SMP
But why the *hell* is this dependent on ACPI?
Why not just do ACPI_SLEEP, and have *that* do "select
HOTPLUG_CPU/SUSPEND_CPU".
> Which I think leads to fewer surprises, and less complicated code.
That makes no sense. You're tying together things that have *nothing* to
do with each other.
As mentioned (and as is *obvious*), pretty much everybody who has a
multi-core CPU on x86 wants ACPI and SMP. But the set of people who want
to sw-suspend such a machine is *much* smaller. There is no 99%.
Linus
-
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]
