From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Brownell Subject: Re: 2.6.19-rc5 nasty ACPI regression, AE_TIME errors Date: Wed, 15 Nov 2006 17:10:26 -0800 Message-ID: <200611151710.26570.david-b@pacbell.net> References: <200611142303.47325.david-b@pacbell.net> <455B28B2.4010707@linux.intel.com> <200611151356.52985.david-b@pacbell.net> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from smtp103.sbc.mail.mud.yahoo.com ([68.142.198.202]:52362 "HELO smtp103.sbc.mail.mud.yahoo.com") by vger.kernel.org with SMTP id S1162141AbWKPBKb (ORCPT ); Wed, 15 Nov 2006 20:10:31 -0500 In-Reply-To: <200611151356.52985.david-b@pacbell.net> Content-Disposition: inline Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: Alexey Starikovskiy Cc: Len Brown , Linux Kernel list , linux-acpi@vger.kernel.org On Wednesday 15 November 2006 1:56 pm, David Brownell wrote: > On Wednesday 15 November 2006 6:48 am, Alexey Starikovskiy wrote: > > ec1.patch > >=20 > >=20 > > Always enable GPE after return from notify handler. > >=20 > > From: =A0Alexey Starikovskiy > >=20 > >=20 > > --- >=20 > Yes, this seems to resolve the regression as well as Len's ec_intr=3D= 0 boot param. Whoops, I spoke too soon. It does get rid of SOME of the AE_TIME error= s. 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 [Embed= dedControl] [20060707] ACPI Exception (dswexec-0458): AE_TIME, While resolving operands for [O= pcodeName unavailable] [2006070 7] ACPI Error (psparse-0537): Method parse/execution failed [\_TZ_.THRM._T= MP] (Node ffff810002032d10), AE _TIME In short, better but evidently not yet good enough... - Dave > IMO this should get merged into 2.6.19 ASAP ... >=20 >=20 > > =A0drivers/acpi/ec.c | =A0 =A02 -- > > =A01 files changed, 0 insertions(+), 2 deletions(-) > >=20 > > 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 > > =A0 > > =A0=A0=A0=A0=A0=A0=A0=A0if (value & ACPI_EC_FLAG_SCI) { > > =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0status =3D acpi_os_= execute(OSL_EC_BURST_HANDLER, acpi_ec_gpe_query, ec); > > -=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0return status =3D=3D = AE_OK ? > > -=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 =A0 =A0ACPI_INTERRUP= T_HANDLED : ACPI_INTERRUPT_NOT_HANDLED; > > =A0=A0=A0=A0=A0=A0=A0=A0} > > =A0=A0=A0=A0=A0=A0=A0=A0acpi_enable_gpe(NULL, ec->gpe_bit, ACPI_ISR= ); > > =A0=A0=A0=A0=A0=A0=A0=A0return status =3D=3D AE_OK ? >=20 - To unsubscribe from this list: send the line "unsubscribe linux-acpi" i= n the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html