Re: [RFC][PATCH 3/4] PCI / ACPI PM: Platform support for PCI PME wake-up (rev. 2)

["Rafael J. Wysocki" <rjw@sisk.pl>]

On Friday 09 October 2009, Matthew Garrett wrote:
> On Fri, Oct 09, 2009 at 12:54:18AM +0200, Rafael J. Wysocki wrote:
> 
> > Unfortunately, the subset of PCI devices that have GPEs associated
> > with them is quite limited and the other devices have to rely on
> > the GPEs associated with their upstream bridges and, possibly, the
> > root bridge to generate ACPI wake-up events in response to PMEs from
> > them.  Moreover, ACPI devices tend to share wake-up GPEs, in which
> > cases it makes sense to install an ACPI notify handler for only one
> > of them and "bind" the other devices to it.  Furthermore, ACPI-based
> > PCI hotplug also uses ACPI notify handlers that in general may
> > conflict with the PM notify handlers, unless this issue is
> > specifically taken care of.
> 
> I'm not sure this approach is correct - for instance, from a Dell 
> system:
> 
>         Method (_L0D, 0, NotSerialized)
>         {
>             Store (SMI (0xC6, 0x00), Local0)
>             If (And (Local0, 0x01))
>             {
>                 Notify (\_SB.PCI0.AZAL, 0x02)
>             }
> 
>             If (And (Local0, 0x02))
>             {
>                 Notify (\_SB.PCI0.EHCI, 0x02)
>             }
> 
>             If (And (Local0, 0x04))
>             {
>                 Notify (\_SB.PCI0.EHC2, 0x02)
>             }
>         }
> 
> All these devices share a GPE, but registering only one notify handler 
> would result in us missing the notification in some cases.

Ok, I see.  The notification may be device-specific, although if we receive
one, we'll need to check all devices sharing the GPE anyway.

I think that's not going to be difficult to take into account, but I won't be
able to work on this until after the KS.

> I also suspect that this should wait until the GPE rework is done.

It certainly is better if the GPE rework is done first, although I'm not sure
it's strictly necessary.  At least I don't see why exactly at the moment.

Thanks,
Rafael