NACK: [PATCH 1/1] [SRU][OEM-5.6] UBUNTU: SAUCE: pci: s3 resume takes too long time, 7s.

[Koba Ko]

Sorry for my typo, BguLink is typo of BugLink.

On Tue, May 5, 2020 at 6:25 PM <koba.ko at canonical.com> wrote:

> From: Koba Ko <koba.ko at canonical.com>
>
> From:
> https://lore.kernel.org/linux-pci/20200416083245.73957-1-mika.westerberg@linux.intel.com/
>
> BguLink: https://bugs.launchpad.net/bugs/1876844
>
> Since PCIe spec mandates that all downstream ports that support
> speeds greater than 5 GT/s must support data link layer active
> reporting so use that here to determine when the delay should
> be issued.
>
> Tested-by: Kai-Heng Feng <kai.heng.feng at canonical.com>
> Signed-off-by: Mika Westerberg <mika.westerberg at linux.intel.com>
> Signed-off-by: Koba Ko <koba.ko at canonical.com>
> (cherry picked from commit ff407506230ccab096fff2d6de04a04a8b55bdc4)
> Signed-off-by: Koba Ko <koba.ko at canonical.com>
> ---
>  drivers/pci/pci.c | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/pci/pci.c b/drivers/pci/pci.c
> index d828ca835a98..aa4d4de1e2e1 100644
> --- a/drivers/pci/pci.c
> +++ b/drivers/pci/pci.c
> @@ -4765,7 +4765,13 @@ void pci_bridge_wait_for_secondary_bus(struct
> pci_dev *dev)
>         if (!pcie_downstream_port(dev))
>                 return;
>
> -       if (pcie_get_speed_cap(dev) <= PCIE_SPEED_5_0GT) {
> +       /*
> +        * Since PCIe spec mandates that all downstream ports that
> +        * support speeds greater than 5 GT/s must support data link
> +        * layer active reporting we use that here to determine when the
> +        * delay should be issued.
> +        */
> +       if (!dev->link_active_reporting) {
>                 pci_dbg(dev, "waiting %d ms for downstream link\n", delay);
>                 msleep(delay);
>         } else {
> --
> 2.17.1
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20200511/b6754d5b/attachment.html>