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
| ||
|
Date: Tue, 06 Nov 2012 08:33:11 +0800 From: Huang Ying <ying.huang@...el.com> To: Bjorn Helgaas <bhelgaas@...gle.com> Cc: Len Brown <lenb@...nel.org>, linux-kernel@...r.kernel.org, "Rafael J. Wysocki" <rjw@...k.pl>, linux-acpi@...r.kernel.org, linux-pm@...r.kernel.org, linux-pci@...r.kernel.org Subject: Re: [PATCH RESEND] ACPI/PCI: Make PCI devices notified when its power resource turned on On Mon, 2012-11-05 at 17:02 -0700, Bjorn Helgaas wrote: > On Wed, Sep 26, 2012 at 7:59 AM, Huang Ying <ying.huang@...el.com> wrote: > > Sorry for bothering. It appears that this patch should go through ACPI > > tree because code changed are under drivers/acpi. > > > > A set of power resources may be shared by multiple devices. When all > > devices share one set of power resources is put into D3_COLD state, > > the power resources will be turned off. When one of the device is > > waked, the power resource will be turned on and all devices share it > > will be powered on to D0uninitialized state. These devices should be > > resumed, so that they can get opportunity to go to low power state > > later. > > > > Signed-off-by: Huang Ying <ying.huang@...el.com> > > Acked-by: Rafael J. Wysocki <rjw@...k.pl> > > Cc: Bjorn Helgaas <bhelgaas@...gle.com> > > I had applied this to my for-linus branch, headed for v3.7, but had to > drop it because it introduced these warnings: > > drivers/acpi/pci_bind.c: In function 'acpi_pci_unbind': > drivers/acpi/pci_bind.c:48:2: warning: passing argument 1 of > 'acpi_power_resource_unregister_device' from incompatible pointer type > [enabled by default] > include/acpi/acpi_bus.h:346:6: note: expected 'struct device *' but > argument is of type 'struct pci_dev *' > drivers/acpi/pci_bind.c: In function 'acpi_pci_bind': > drivers/acpi/pci_bind.c:75:2: warning: passing argument 1 of > 'acpi_power_resource_register_device' from incompatible pointer type > [enabled by default] > include/acpi/acpi_bus.h:345:5: note: expected 'struct device *' but > argument is of type 'struct pci_dev *' > > Apparently this wasn't tested :-( It's my fault. I apologize for that. It appear that I turn a RFC to PATCH without testing. Will be more careful for next time. Best Regards, Huang Ying > > --- > > drivers/acpi/pci_bind.c | 2 ++ > > 1 file changed, 2 insertions(+) > > > > --- a/drivers/acpi/pci_bind.c > > +++ b/drivers/acpi/pci_bind.c > > @@ -45,6 +45,7 @@ static int acpi_pci_unbind(struct acpi_d > > > > device_set_run_wake(&dev->dev, false); > > pci_acpi_remove_pm_notifier(device); > > + acpi_power_resource_unregister_device(dev, device->handle); > > > > if (!dev->subordinate) > > goto out; > > @@ -71,6 +72,7 @@ static int acpi_pci_bind(struct acpi_dev > > return 0; > > > > pci_acpi_add_pm_notifier(device, dev); > > + acpi_power_resource_register_device(dev, device->handle); > > if (device->wakeup.flags.run_wake) > > device_set_run_wake(&dev->dev, true); > > -- 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/
Powered by blists - more mailing lists