public inbox for linux-acpi@vger.kernel.org
 help / color / mirror / Atom feed
* Re: PATCH: Don't enable GPE if GPE dispatch failed
       [not found] ` <1309311174.5389.0.camel@rui>
@ 2011-06-29  1:35   ` Zhang Rui
       [not found]     ` <BANLkTinqc5FeP9x3B=QjQ0RaXJPytYzn4A@mail.gmail.com>
  0 siblings, 1 reply; 2+ messages in thread
From: Zhang Rui @ 2011-06-29  1:35 UTC (permalink / raw)
  To: H.J. Lu; +Cc: acpi@linux.intel.com, linux-acpi@vger.kernel.org

CC linux-acpi mail list.

On Wed, 2011-06-29 at 09:32 +0800, Zhang, Rui wrote:
> Hi,
> 
> can you send me the acpidump output of this machine please?
> 
> thanks,
> rui
> 
> On Wed, 2011-06-29 at 06:05 +0800, Lu, Hongjiu wrote:
> > Hi,
> > 
> > I have a broken BIOS, which generates thousands of kernel messages:
> > 
> > Jun 28 14:13:26 gnu-18 kernel: [  628.704003] ACPI Exception: AE_NOT_EXIST, while evaluating GPE method [_L01] (20110112/evgpe-551)
> > Jun 28 14:13:26 gnu-18 kernel: [  628.826752] ACPI Error: No handler for Region [ECF2] (ffff88013763b6c0) [EmbeddedControl] (20110112/evregion-369)
> > Jun 28 14:13:26 gnu-18 kernel: [  628.826761] ACPI Error: Region EmbeddedControl(0x3) has no handler (20110112/exfldio-292)
> > Jun 28 14:13:26 gnu-18 kernel: [  628.826768] ACPI Error: Method parse/execution failed [\_GPE._L01] (Node ffff88013765a488), AE_NOT_EXIST (20110112/psparse-536)
> > 
> > This patch avoids enabling GPE if GPE dispatch failed.  Now I got only
> > one kernel message:
> > 
> > Jun 28 14:38:51 gnu-18 kernel: [    0.937479] ACPI Error: No handler for Region [ECF2] (ffff88013763b6c0) [EmbeddedControl] (20110112/evregion-369)
> > Jun 28 14:38:51 gnu-18 kernel: [    0.937485] ACPI Error: Region EmbeddedControl(0x3) has no handler (20110112/exfldio-292)
> > Jun 28 14:38:51 gnu-18 kernel: [    0.937490] ACPI Error: Method parse/execution failed [\_GPE._L01] (Node ffff88013765a488), AE_NOT_EXIST (20110112/psparse-536)
> > Jun 28 14:38:51 gnu-18 kernel: [    0.937502] ACPI Exception: AE_NOT_EXIST, while evaluating GPE method [_L01] (20110112/evgpe-551)
> > 
> > 
> > 
> > H.J.
> > ---
> > diff --git a/drivers/acpi/acpica/evgpe.c b/drivers/acpi/acpica/evgpe.c
> > index f472521..5b1f4b2 100644
> > --- a/drivers/acpi/acpica/evgpe.c
> > +++ b/drivers/acpi/acpica/evgpe.c
> > @@ -559,11 +559,13 @@ static void ACPI_SYSTEM_XFACE acpi_ev_asynch_execute_gpe_method(void *context)
> >  
> >  	/* Defer enabling of GPE until all notify handlers are done */
> >  
> > -	status = acpi_os_execute(OSL_NOTIFY_HANDLER,
> > -				 acpi_ev_asynch_enable_gpe,
> > -				 local_gpe_event_info);
> > -	if (ACPI_FAILURE(status)) {
> > -		ACPI_FREE(local_gpe_event_info);
> > +	if (!ACPI_FAILURE(status)) {
> > +		status = acpi_os_execute(OSL_NOTIFY_HANDLER,
> > +					 acpi_ev_asynch_enable_gpe,
> > +					 local_gpe_event_info);
> > +		if (ACPI_FAILURE(status)) {
> > +			ACPI_FREE(local_gpe_event_info);
> > +		}
> >  	}
> >  	return_VOID;
> >  }
> > 
> > _______________________________________________
> > acpi mailing list
> > acpi@linux.intel.com
> > http://linux.intel.com/mailman/listinfo/acpi
> 
> 
> _______________________________________________
> acpi mailing list
> acpi@linux.intel.com
> http://linux.intel.com/mailman/listinfo/acpi



^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: PATCH: Don't enable GPE if GPE dispatch failed
       [not found]     ` <BANLkTinqc5FeP9x3B=QjQ0RaXJPytYzn4A@mail.gmail.com>
@ 2011-06-29  6:19       ` Zhang Rui
  0 siblings, 0 replies; 2+ messages in thread
From: Zhang Rui @ 2011-06-29  6:19 UTC (permalink / raw)
  To: H.J. Lu; +Cc: acpi@linux.intel.com, linux-acpi@vger.kernel.org

Hi, H.J.

IMO, the real problem is that EC address space handler is not installed.
please attach the full dmesg output of your patched kernel after boot.

thanks,
rui

On Wed, 2011-06-29 at 10:51 +0800, H.J. Lu wrote:
> On Tue, Jun 28, 2011 at 6:35 PM, Zhang Rui <rui.zhang@intel.com> wrote:
> > CC linux-acpi mail list.
> >
> > On Wed, 2011-06-29 at 09:32 +0800, Zhang, Rui wrote:
> >> Hi,
> >>
> >> can you send me the acpidump output of this machine please?
> >>
> >> thanks,
> >> rui
> >>
> >> On Wed, 2011-06-29 at 06:05 +0800, Lu, Hongjiu wrote:
> >> > Hi,
> >> >
> >> > I have a broken BIOS, which generates thousands of kernel messages:
> >> >
> >> > Jun 28 14:13:26 gnu-18 kernel: [  628.704003] ACPI Exception: AE_NOT_EXIST, while evaluating GPE method [_L01] (20110112/evgpe-551)
> >> > Jun 28 14:13:26 gnu-18 kernel: [  628.826752] ACPI Error: No handler for Region [ECF2] (ffff88013763b6c0) [EmbeddedControl] (20110112/evregion-369)
> >> > Jun 28 14:13:26 gnu-18 kernel: [  628.826761] ACPI Error: Region EmbeddedControl(0x3) has no handler (20110112/exfldio-292)
> >> > Jun 28 14:13:26 gnu-18 kernel: [  628.826768] ACPI Error: Method parse/execution failed [\_GPE._L01] (Node ffff88013765a488), AE_NOT_EXIST (20110112/psparse-536)
> >> >
> >> > This patch avoids enabling GPE if GPE dispatch failed.  Now I got only
> >> > one kernel message:
> >> >
> >> > Jun 28 14:38:51 gnu-18 kernel: [    0.937479] ACPI Error: No handler for Region [ECF2] (ffff88013763b6c0) [EmbeddedControl] (20110112/evregion-369)
> >> > Jun 28 14:38:51 gnu-18 kernel: [    0.937485] ACPI Error: Region EmbeddedControl(0x3) has no handler (20110112/exfldio-292)
> >> > Jun 28 14:38:51 gnu-18 kernel: [    0.937490] ACPI Error: Method parse/execution failed [\_GPE._L01] (Node ffff88013765a488), AE_NOT_EXIST (20110112/psparse-536)
> >> > Jun 28 14:38:51 gnu-18 kernel: [    0.937502] ACPI Exception: AE_NOT_EXIST, while evaluating GPE method [_L01] (20110112/evgpe-551)
> >> >
> >> >
> >> >
> >> > H.J.
> >> > ---
> >> > diff --git a/drivers/acpi/acpica/evgpe.c b/drivers/acpi/acpica/evgpe.c
> >> > index f472521..5b1f4b2 100644
> >> > --- a/drivers/acpi/acpica/evgpe.c
> >> > +++ b/drivers/acpi/acpica/evgpe.c
> >> > @@ -559,11 +559,13 @@ static void ACPI_SYSTEM_XFACE acpi_ev_asynch_execute_gpe_method(void *context)
> >> >
> >> >     /* Defer enabling of GPE until all notify handlers are done */
> >> >
> >> > -   status = acpi_os_execute(OSL_NOTIFY_HANDLER,
> >> > -                            acpi_ev_asynch_enable_gpe,
> >> > -                            local_gpe_event_info);
> >> > -   if (ACPI_FAILURE(status)) {
> >> > -           ACPI_FREE(local_gpe_event_info);
> >> > +   if (!ACPI_FAILURE(status)) {
> >> > +           status = acpi_os_execute(OSL_NOTIFY_HANDLER,
> >> > +                                    acpi_ev_asynch_enable_gpe,
> >> > +                                    local_gpe_event_info);
> >> > +           if (ACPI_FAILURE(status)) {
> >> > +                   ACPI_FREE(local_gpe_event_info);
> >> > +           }
> >> >     }
> >> >     return_VOID;
> >> >  }
> >> >
> 
> Here it is.
> 
> 



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2011-06-29  6:17 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <20110628220535.GA19846@intel.com>
     [not found] ` <1309311174.5389.0.camel@rui>
2011-06-29  1:35   ` PATCH: Don't enable GPE if GPE dispatch failed Zhang Rui
     [not found]     ` <BANLkTinqc5FeP9x3B=QjQ0RaXJPytYzn4A@mail.gmail.com>
2011-06-29  6:19       ` Zhang Rui

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox