Re: [PATCH] amd76x_pm: C3 powersaving for AMD K7

[Tony Lindgren <tony@atomide.com>]

* Andrew Morton <akpm@osdl.org> [060131 19:35]:
> Joerg Sommrey <jo@sommrey.de> wrote:
> >
> > This is a processor idle module for AMD SMP 760MP(X) based systems.
> >  The patch was originally written by Tony Lindgren and has been around
> >  since 2002.  It enables C2 mode on AMD SMP systems and thus saves
> >  about 70 - 90 W of energy in the idle mode compared to the default idle
> >  mode.  The idle function has been rewritten and is now free of locking
> >  issues and is independent from the number of CPUs.  The impact
> >  from this module on the system clock and on i/o transfer are now fairly
> >  low.
> > 
> >  This patch can also be found at
> >  http://www.sommrey.de/amd76x_pm/amd76x_pm-2.6.15-4.patch
> > 
> >  In this version more locking was added to make sure all or no CPU enter
> >  C3 mode.
> > 
> >  Signed-off-by: Joerg Sommrey <jo@sommrey.de>
> 
> Thanks.  I'll merge this into -mm and shall plague the ACPI guys with it. 
> They have said discouraging things about board-specific drivers in the
> past.  We shall see.
> 
> Tony, could you please review section 11 of Documentation/SubmittingPatches
> and if OK, send a Signed-off-by:?

Sure, here you are:

Signed-off-by: Tony Lindgren <tony@atomide.com>
 
> Some minor pointlets, and a bug:
> 
> 
> Should CONFIG_AMD76X_PM depend on ACPI?

It works also without ACPI.

But in the long run something like this could be added for ACPI to allow
easy addition of custom PM modules. That was discussed briefly on the ACPI
list and people seemed to be OK with adding system for custom PM modules.

And if it was integrated with ACPI, only enabling the C states should be
enough, and then ACPI would just know how to use the C states...

Regards,

Tony
-
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/