linux-kernel - Re: [PATCH -v6 0/4] PCIe, PM, Add PCIe runtime D3cold support

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]

Date:	Sat, 23 Jun 2012 01:15:33 +0200
From:	"Rafael J. Wysocki" <rjw@...k.pl>
To:	Bjorn Helgaas <bhelgaas@...gle.com>,
	Huang Ying <ying.huang@...el.com>
Cc:	ming.m.lin@...el.com, linux-kernel@...r.kernel.org,
	linux-pci@...r.kernel.org, linux-pm@...r.kernel.org,
	Zheng Yan <zheng.z.yan@...el.com>
Subject: Re: [PATCH -v6 0/4] PCIe, PM, Add PCIe runtime D3cold support

On Saturday, June 23, 2012, Bjorn Helgaas wrote:
> On Mon, Jun 18, 2012 at 11:20 PM, Huang Ying <ying.huang@...el.com> wrote:
> > Changelog:
> >
> > v6:
> >
> > - Disable device D3cold during system suspend/hibernate with a new method per Rafeal's comments
> >
> > v5:
> >
> > - Disable device D3cold during system suspend/hibernate per Rafael's comments
> >
> > v4:
> >
> > - Minors fixes based on comments from Rafael
> >
> > v3:
> >
> > - Drop device.power.power_must_be_on, use pci_dev.no_d3cold instead
> > - Drop device.power.power_off_user, use pci_dev.d3cold_allowed instead
> > - Use black list instead of white list in pcie port runtime support per request from Bjorn
> > - Various fixes based on comments from Rafael
> >
> > v2:
> >
> > - Refreshed based on comments from Rafael
> >
> > [PATCH -v6 1/4] ACPI, PM, Specify lowest allowed state for device sleep
> > [PATCH -v6 2/4] PCIe, PM, Add runtime PM support to PCIe port
> > [PATCH -v6 3/4] PCI, PM, Do not call pci_set_power_state with PCI_D3cold
> > [PATCH -v6 4/4] PCIe, PM, Add PCIe runtime D3cold support
> 
> What kernel is this series based on?  Patch 4/4 doesn't apply to my
> tree.  This hunk fails because I don't have the "Some devices mustn't
> be in D3 ..." context:

I think it's based on an ACPI patch that didn't go in due to the problem
with the Len's tree.

Huang, can you please resend the series along with that patch?

Rafael


> @@ -1754,6 +1841,10 @@ int pci_prepare_to_sleep(struct pci_dev
>         if (target_state == PCI_POWER_ERROR)
>                 return -EIO;
> 
> +       /* D3cold during system suspend/hibernate is not supported */
> +       if (target_state > PCI_D3hot)
> +               target_state = PCI_D3hot;
> +
>         /* Some devices mustn't be in D3 during system sleep */
>         if (target_state == PCI_D3hot &&
>                         (dev->dev_flags & PCI_DEV_FLAGS_NO_D3_DURING_SLEEP))
> 
> 

--
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/