linux-kernel - Re: [RFC][PATCH 3/4] PCI / ACPI PM: Platform support for PCI PME wake-up

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20090913225518.GA31787@srcf.ucam.org>
Date:	Sun, 13 Sep 2009 23:55:18 +0100
From:	Matthew Garrett <mjg59@...f.ucam.org>
To:	"Rafael J. Wysocki" <rjw@...k.pl>
Cc:	pm list <linux-pm@...ts.linux-foundation.org>,
	Linux PCI <linux-pci@...r.kernel.org>,
	Len Brown <lenb@...nel.org>,
	LKML <linux-kernel@...r.kernel.org>,
	Jesse Barnes <jbarnes@...tuousgeek.org>,
	Shaohua Li <shaohua.li@...el.com>,
	ACPI Devel Maling List <linux-acpi@...r.kernel.org>
Subject: Re: [RFC][PATCH 3/4] PCI / ACPI PM: Platform support for PCI PME
	wake-up

On Mon, Sep 14, 2009 at 12:53:05AM +0200, Rafael J. Wysocki wrote:
> On Monday 14 September 2009, Matthew Garrett wrote:
> > On Sun, Sep 13, 2009 at 11:24:03PM +0200, Rafael J. Wysocki wrote:
> > > +	} else if (!dev->wakeup.flags.run_wake) {
> > > +		acpi_set_gpe_type(dev->wakeup.gpe_device,
> > > +					dev->wakeup.gpe_number,
> > > +					ACPI_GPE_TYPE_WAKE);
> > 
> > Is this going to work for cases where we have multiple devices attached 
> > to the same GPE? The common one is EHCI, where both EHCI HCDs will be 
> > one a single GPE. If we wake one, that'll then disable the GPE for the 
> > other. Further wakeup events will then be lost.
> 
> You're right, I overlooked that.  Some kind of refcounting is needed here.

I've sent patches to implement this at the GPE level, which also change 
the API for requesting them. I'm waiting on feedback from Bob Moore.

> > > +	if (device->wakeup.flags.valid)
> > > +		acpi_install_notify_handler(device->handle, ACPI_SYSTEM_NOTIFY,
> > > +						pci_acpi_device_wakeup,
> > > +						&dev->dev);
> > > +
> > 
> > I think this will fail for the root bridge if acpiphp has already 
> > grabbed it to check for hotplug events.
> 
> The root bridge is not a struct pci_dev and this is called only for these.

Are you sure? When I did this I saw conflicts with acpiphp. I may have 
my terminology confused. I sent another patch that abstract that out.

-- 
Matthew Garrett | mjg59@...f.ucam.org
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/