Re: clockevents: fix resume logic

Thomas Gleixner
2007-09-11T18:38:18Z

On Tue, 2007-09-11 at 11:25 -0700, Andrew Morton wrote:
> > It evidently assumes cpuidle to be present, which is not in the mainline.
>
> Bear in mind that the cpuidle patch fixes resume-from-ram when cpuidle is
> disabled in config.
>
> > It seems to me that the total effect of this one and the hackpatch is that
> > the C states are not handled any more.
>
> hm.
>
> dmesg without the cpuidle patch: http://userweb.kernel.org/~akpm/dmesg-bad.txt
> dmesg with the cpuidle patch: http://userweb.kernel.org/~akpm/dmesg-good.txt
> difference: http://userweb.kernel.org/~akpm/dmesg-diff.txt
>
> there doesn't seem to be a lot of difference in the time handling, except
> there are large changes in when things happen in the bootup sequence.

The question is whether the system goes into C2 with the patch applied.

Can you please provide the output of /proc/acpi/processor/CPU0/power for
both the bad and the good one ?

tglx