Re: [PATCH] Set bit 1 in disabled processor's _STA

[Glauber Costa <glommer@redhat.com>]

On Sun, May 17, 2009 at 11:23:47AM +0300, Gleb Natapov wrote:
> On Fri, May 15, 2009 at 08:14:43AM -0400, Glauber Costa wrote:
> > This patch sets bits 1 in disabled processor's _STA.
> > According to the ACPI spec, this bit means:
> >  "Set if the device is enabled and decoding its resources."
> > 
> > Without it, Windows 2008 device manager shows the processors
> > as malfunctioning hardware.
> > 
> If you uncheck "show hidden devices" option in View menu of the
> device manager you should see only enabled CPUs (bit 2 of _STA).
Yes, that is what happens. But then, by checking "show hidden devices",
the disabled processor appears, but are listed as malfunctioning.

Because of this, SVVP tests break without this patch.

> This patch breaks resume from hibernate on windows 2008/vista.
Oh god... do you have any idea why?

which bits should we set for everything to work?
It appears to me that having bit 2 cleared should do the right thing,
but your test clearly indicate otherwise.