Re: 2.6.19-rc5 nasty ACPI regression, AE_TIME errors

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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 [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux