From: Alexey Starikovskiy <alexey.y.starikovskiy@linux.intel.com>
To: David Brownell <david-b@pacbell.net>
Cc: Len Brown <lenb@kernel.org>,
Linux Kernel list <linux-kernel@vger.kernel.org>,
linux-acpi@vger.kernel.org
Subject: Re: 2.6.19-rc5 nasty ACPI regression, AE_TIME errors
Date: Sat, 18 Nov 2006 19:18:23 +0300 [thread overview]
Message-ID: <455F324F.3090208@linux.intel.com> (raw)
In-Reply-To: <200611171304.00889.david-b@pacbell.net>
Please take a look at 7466, they seem to fight same problem, so may be
removing same patch will work...
And Linus is about to drop it anyway...
Regards,
Alex.
David Brownell wrote:
> On Thursday 16 November 2006 10:22 pm, David Brownell wrote:
>
>> On Thursday 16 November 2006 7:41 am, Alexey Starikovskiy wrote:
>>
>>> --- a/drivers/acpi/ec.c
>>> +++ b/drivers/acpi/ec.c
>>> @@ -467,8 +467,8 @@ static u32 acpi_ec_gpe_handler(void *dat
>>> status = acpi_os_execute(OSL_EC_BURST_HANDLER, acpi_ec_gpe_query, ec);
>>> }
>>> acpi_enable_gpe(NULL, ec->gpe_bit, ACPI_ISR);
>>> - return status == AE_OK ?
>>> - ACPI_INTERRUPT_HANDLED : ACPI_INTERRUPT_NOT_HANDLED;
>>> + WARN_ON(ACPI_FAILURE(status));
>>> + return ACPI_INTERRUPT_HANDLED;
>>> }
>>>
>>>
>> Strange ... applying this on top of the previous patch seems to work
>> much better, but that WARN_ON hasn't triggered. At least, not yet.
>> Updating to RC6, with your two patches installed...
>>
>
> 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
>
> OK, I don't get the WARN_ON when these happen, so it's got to be one of the
> other EC updates.
>
> It'd be nice if this were easily reproducible ...
>
> - Dave
>
> -
> To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
next prev parent reply other threads:[~2006-11-18 16:18 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-11-15 7:03 2.6.19-rc5 nasty ACPI regression, AE_TIME errors David Brownell
2006-11-15 7:48 ` Len Brown
2006-11-15 14:48 ` Alexey Starikovskiy
2006-11-15 21:56 ` David Brownell
2006-11-16 1:10 ` David Brownell
2006-11-16 15:41 ` Alexey Starikovskiy
2006-11-16 16:03 ` David Brownell
2006-11-16 16:23 ` Alexey Starikovskiy
2006-11-16 16:53 ` David Brownell
2006-11-17 6:22 ` David Brownell
2006-11-17 21:04 ` David Brownell
2006-11-18 16:18 ` Alexey Starikovskiy [this message]
2006-11-15 18:46 ` David Brownell
2006-11-21 14:47 ` Janosch Machowinski
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=455F324F.3090208@linux.intel.com \
--to=alexey.y.starikovskiy@linux.intel.com \
--cc=david-b@pacbell.net \
--cc=lenb@kernel.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox