Re: [linux-pm] commit 'ACPICA: Minimize the differences between linux GPE code and ACPICA code base' breaks EC GPE on my system

[Maxim Levitsky <maximlevitsky@gmail.com>]

On Fri, 2010-06-11 at 22:59 +0200, Rafael J. Wysocki wrote: 
> On Friday, June 11, 2010, Rafael J. Wysocki wrote:
> > On Friday, June 11, 2010, Maxim Levitsky wrote:
> > > On Fri, 2010-06-11 at 21:43 +0200, Rafael J. Wysocki wrote: 
> > > > On Friday, June 11, 2010, Rafael J. Wysocki wrote:
> > > > > On Friday, June 11, 2010, Maxim Levitsky wrote:
> > > > > > Just bisected it.
> > > > > > 
> > > > > > I also tried linux-acpi-next/test, and no change.
> > > > > > 
> > > > > > The sympthoms are that EC does't sent any GPEs, and therefore battery
> > > > > > insert/removal events don't show up.
> > > > > > 
> > > > > > It can be see by doing 'grep  . /sys/firmware/acpi/interrupts/*'
> > > > > > With regression the line is shown like this:
> > > > > > 
> > > > > > 
> > > > > > /sys/firmware/acpi/interrupts/gpe1C:   1	enabled
> > > > > > 
> > > > > > Without regression it is
> > > > > > 
> > > > > > /sys/firmware/acpi/interrupts/gpe1C:   22889	enabled
> > > > > > 
> > > > > > and steadily increasing.
> > > > > > 
> > > > > > After suspend/resume, regression disappears.
> > > > > 
> > > > > Hmm.
> > > > > 
> > > > > Can you please apply the following patches:
> > > > > 
> > > > > https://patchwork.kernel.org/patch/104903/
> > > > > https://patchwork.kernel.org/patch/104912/
> > > > > https://patchwork.kernel.org/patch/104909/
> > > > > https://patchwork.kernel.org/patch/104911/
> > > > > https://patchwork.kernel.org/patch/104910/
> > > > > 
> > > > > on top of current -git and see if the problem is still there?
> > > > 
> > > > Also, regardless of whether or not this helps, please try to revert only the
> > > > changes made by the "guilty" commit in drivers/acpi/acpica/evxface.c and see
> > > > if that helps (this revert will conflict with the patches above, so you'll need
> > > > to unapply them before).
> > > > 
> > > > Thanks,
> > > > Rafael
> > > > --
> > > > 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
> > > 
> > > Yes, reverting changes in 'drivers/acpi/acpica/evxface.c' do restore
> > > correct behavior.
> > 
> > Good.  I think we'll need to revert them for .35, then.
> > 
> > I'll prepare a patch and send it to Len.
> 
> Can you check if the patch below fixes the issue?
Yes it does. Thanks!

Best regards,
Maxim Levitsky