Re: [PATCH 0/3] ACPI / PM: Make ACPI-based PCI wakeup work for the "freeze" sleep state

Rafael J. Wysocki
2014-07-23T12:17:55Z

On Wednesday, July 23, 2014 01:38:40 PM Peter Zijlstra wrote:
> On Wed, Jul 23, 2014 at 01:43:59PM +0200, Rafael J. Wysocki wrote:
> > On Wednesday, July 23, 2014 09:28:50 AM Peter Zijlstra wrote:
> > > On Wed, Jul 23, 2014 at 12:26:20AM +0200, Rafael J. Wysocki wrote:
> > > > All of that indicates that the machine in question has WoL based on native PCIe
> > > > PME signaling. In that case it doesn't wake up from the "freeze" state, because
> > > > some code is missing.
> > >
> > > Didn't wake, but it did show:
> > >
> > > 0000:00:01.0:pcie01 1 1 0 0 0 0 0 99207 0
> > > LNXPWRBN:00 1 1 0 0 0 0 0 99191 0
> > >
> > > So at least something's moving, although its not quite working yet.
> >
> > That may be because I forgot about one piece, sorry about that.
> > This patch:
> >
> > https://patchwork.kernel.org/patch/4526561/
> >
> > (which I think is in tip already) is needed to make enable_irq_wake()
> > work with the "freeze" state.
> >
> > Can you please apply this in addition and retest?
>
> I appear to have that hunk already (I'm running -tip based kernels).

I see.

> (also, patchwork blows chunks :/)

That's never happened to me (and there's only one hunk in that patch).

Anyway, the count for 0000:00:01.0:pcie01 means that the wakeup source
has been activated, so it should have woken it up in theory.

Unless, of course, it was activated after the power button wakeup.

It looks like the stuff works on the hardware level, though, so we should be
able to make the wakeup work too. I'll have another look later today.

Rafael