[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <455C907A.8080600@linux.intel.com>
Date: Thu, 16 Nov 2006 19:23:22 +0300
From: Alexey Starikovskiy <alexey.y.starikovskiy@...ux.intel.com>
To: David Brownell <david-b@...bell.net>
CC: Len Brown <lenb@...nel.org>,
Linux Kernel list <linux-kernel@...r.kernel.org>,
linux-acpi@...r.kernel.org
Subject: Re: 2.6.19-rc5 nasty ACPI regression, AE_TIME errors
David Brownell wrote:
> On Thursday 16 November 2006 7:41 am, Alexey Starikovskiy wrote:
>
>> Looks like either EC GPE or whole ACPI irq got disabled... Could you
>> check that ACPI interrupts still arrive after
>> you notice AE_TIME?
>>
>
> If I unplug AC, /proc/interrrupts reports 2 IRQs going to ACPI.
> Then on replug, it reports 10-12 IRQs going to ACPI.
>
What is the state of kacpid and kacpid_notify ?
>
>
>> Also, may be attached patch will help?
>>
>
> I'l give it a try next time I can reboot (presumably by this PM).
>
> - Dave
>
>
>
>> Regards,
>> Alex.
>>
>> David Brownell wrote:
>>
>>> On Wednesday 15 November 2006 1:56 pm, David Brownell wrote:
>>>
>>>
>>>> On Wednesday 15 November 2006 6:48 am, Alexey Starikovskiy wrote:
>>>>
>>>>
>>>>> ec1.patch
>>>>>
>>>>>
>>>>> Always enable GPE after return from notify handler.
>>>>>
>>>>> From: Alexey Starikovskiy <alexey.y.starikovskiy@...ux.intel.com>
>>>>>
>>>>>
>>>>> ---
>>>>>
>>>>>
>>>> Yes, this seems to resolve the regression as well as Len's ec_intr=0 boot param.
>>>>
>>>>
>>> Whoops, I spoke too soon. It does get rid of SOME of the AE_TIME errors. But
>>> the system is still confused about whether or not the AC is connected, and
>>> whether the battery is charging or not; and the CPU is still relatively hot.
>>> Even with this patch I later got:
>>>
>>> ACPI Exception (evregion-0424): AE_TIME, Returned by Handler for [EmbeddedControl] [20060707]
>>> ACPI Exception (dswexec-0458): AE_TIME, While resolving operands for [OpcodeName unavailable] [2006070
>>> 7]
>>> ACPI Error (psparse-0537): Method parse/execution failed [\_TZ_.THRM._TMP] (Node ffff810002032d10), AE
>>> _TIME
>>>
>>> In short, better but evidently not yet good enough...
>>>
>>> - Dave
>>>
>>>
>>>
>>>
>>>
>>>> IMO this should get merged into 2.6.19 ASAP ...
>>>>
>>>>
>>>>
>>>>
>>>>> drivers/acpi/ec.c | 2 --
>>>>> 1 files changed, 0 insertions(+), 2 deletions(-)
>>>>>
>>>>> diff --git a/drivers/acpi/ec.c b/drivers/acpi/ec.c
>>>>> index e6d4b08..937eafc 100644
>>>>> --- a/drivers/acpi/ec.c
>>>>> +++ b/drivers/acpi/ec.c
>>>>> @@ -465,8 +465,6 @@ static u32 acpi_ec_gpe_handler(void *dat
>>>>>
>>>>> if (value & ACPI_EC_FLAG_SCI) {
>>>>> status = acpi_os_execute(OSL_EC_BURST_HANDLER, acpi_ec_gpe_query, ec);
>>>>> - return status == AE_OK ?
>>>>> - ACPI_INTERRUPT_HANDLED : ACPI_INTERRUPT_NOT_HANDLED;
>>>>> }
>>>>> acpi_enable_gpe(NULL, ec->gpe_bit, ACPI_ISR);
>>>>> return status == AE_OK ?
>>>>>
>>>>>
>>
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists