Re: [PATCH] Quirk to fix suspend/resume on Lenovo Edge 11,13,14,15

[Manoj Iyer]

Also, attaching dmesg with my patch applied. Please note the warning
message is printed from dmi_ignore_irq0_timer_override() in
acpi/boot.c, printed before setting acpi_skip_timer_override = 1;

Cheers
Manoj

On Thu, Jan 13, 2011 at 2:33 PM, Manoj Iyer <manoj.iyer@xxxxxxxxxxxxx> wrote:
> I have attached 4 dmesg outputs from AMD based thinkpad Edge 11.
>
> 1. dmesg output with kernel parameter acpi_skip_timer_override after
> suspend/resume
> 2. dmesg output with kernel parameter acpi_skip_timer_override after system boot
> 3. dmesg output with nohpet kernel parameter after system boot
> 4. dmesg output of stock kernel with no kernel parameters.
>
> I also noticed certain inaccuracies in my patch, but once we decide
> what the right approach to the problem is I will be happy to send out
> a new patch.
>
> Many thanks
> Manoj Iyer
>
> On Thu, Jan 13, 2011 at 1:41 PM, Thomas Gleixner <tglx@xxxxxxxxxxxxx> wrote:
>> On Thu, 13 Jan 2011, Borislav Petkov wrote:
>>
>>> On Thu, Jan 13, 2011 at 08:13:42PM +0100, Thomas Gleixner wrote:
>>> > > Well, Andreas did boot with 'hpet=verbose' on an affected machine here
>>> > > and did a suspend/resume and the hpet config registers looked ok before
>>> > > suspend and after resume. It might be that the HPET is temporarily
>>> > > "insane" while resume lasts but we don't have any hard facts confirming
>>> >
>>> > And you have no explanation at all why applying the irq pin routing
>>> > quirk makes HPETs temporal insanity go away magically :)
>>>
>>> But after the HPET counter wraps around, the machine is alive again.
>>> Which means that the IRQ0 pin2 override is only temporarily needed after
>>> resume... Strange.
>>
>> Thinking more about it:
>>
>> Case 1: IRQ0 pin2 override applied
>>
>>     Resume hangs until HPET wraps around and issues another interrupt
>>
>> Case 2: IRQ0 pin2 override ignored via quirk
>>
>>     Resume just works
>>
>> So the question is what is restored _AFTER_ the HPET is reprogrammed
>> in the resume path ?
>>
>> The HPET reprogramming happens via timekeeping_resume() which is in
>> the sysdev part of resume. ioapic, apic, iommus etc. are also resumed
>> via the sysdev_class. So what makes sure that the ordering of these is
>> correct?
>>
>> AFAICT nothing :)
>>
>> We need information about the resume order of sysdev_class and the
>> difference of the pin routings in the quirk non/quirk case.
>>
>> Thanks,
>>
>>        tglx
>>
>
>
>
> --
> --manjo
>



-- 
--manjo

Attachment: dmesg.dmiquirk-kernel
Description: Binary data