linux-pci.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Christoph Hellwig <hch@lst.de>
To: "Rafael J. Wysocki" <rjw@rjwysocki.net>
Cc: Keith Busch <kbusch@kernel.org>,
	Mario Limonciello <Mario.Limonciello@dell.com>,
	Kai-Heng Feng <kai.heng.feng@canonical.com>,
	Keith Busch <keith.busch@intel.com>,
	Christoph Hellwig <hch@lst.de>, Sagi Grimberg <sagi@grimberg.me>,
	linux-nvme <linux-nvme@lists.infradead.org>,
	Linux PM <linux-pm@vger.kernel.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Rajat Jain <rajatja@google.com>,
	Linux PCI <linux-pci@vger.kernel.org>,
	Bjorn Helgaas <helgaas@kernel.org>
Subject: Re: [PATCH] nvme-pci: Do not prevent PCI bus-level PM from being used
Date: Wed, 7 Aug 2019 12:43:53 +0200	[thread overview]
Message-ID: <20190807104353.GA11356@lst.de> (raw)
In-Reply-To: <1893355.EP2830DdO9@kreacher>

> +	if (pm_suspend_via_firmware() || !ctrl->npss || !pcie_aspm_enabled(pdev)) {



> +	mutex_lock(&aspm_lock);
> +	aspm_enabled = bridge->link_state ? bridge->link_state->aspm_enabled : 0;

Please fix the overly long lines..

  parent reply	other threads:[~2019-08-07 10:43 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <4323ed84dd07474eab65699b4d007aaf@AUSX13MPC105.AMER.DELL.COM>
     [not found] ` <CAJZ5v0iDQ4=kTUgW94tKGt7oJzA_3uVU_M6HAMbNCRXwp_do8A@mail.gmail.com>
     [not found]   ` <47415939.KV5G6iaeJG@kreacher>
     [not found]     ` <20190730144134.GA12844@localhost.localdomain>
     [not found]       ` <100ba4aff1c6434a81e47774ab4acddc@AUSX13MPC105.AMER.DELL.COM>
     [not found]         ` <8246360B-F7D9-42EB-94FC-82995A769E28@canonical.com>
     [not found]           ` <20190730191934.GD13948@localhost.localdomain>
     [not found]             ` <7d3e0b8ba1444194a153c93faa1cabb3@AUSX13MPC105.AMER.DELL.COM>
     [not found]               ` <20190730213114.GK13948@localhost.localdomain>
     [not found]                 ` <CAJZ5v0gxfeMN8eCNRjcXmUOkReVsdozb3EccaYMpnmSHu3771g@mail.gmail.com>
     [not found]                   ` <20190731221956.GB15795@localhost.localdomain>
2019-08-07  9:53                     ` [PATCH] nvme-pci: Do not prevent PCI bus-level PM from being used Rafael J. Wysocki
2019-08-07 10:14                       ` Rafael J. Wysocki
2019-08-07 10:43                       ` Christoph Hellwig [this message]
2019-08-07 14:37                       ` Keith Busch
2019-08-08  8:36                     ` [PATCH] nvme-pci: Allow PCI bus-level PM to be used if ASPM is disabled Rafael J. Wysocki
2019-08-08  8:48                       ` Christoph Hellwig
2019-08-08  9:06                         ` Rafael J. Wysocki
2019-08-08 10:03                     ` [PATCH v2 0/2] " Rafael J. Wysocki
2019-08-08 10:06                       ` [PATCH v2 1/2] PCI: PCIe: ASPM: Introduce pcie_aspm_enabled_mask() Rafael J. Wysocki
2019-08-08 13:15                         ` Bjorn Helgaas
2019-08-08 14:48                           ` Rafael J. Wysocki
2019-08-08 10:10                       ` [PATCH v2 2/2] nvme-pci: Allow PCI bus-level PM to be used if ASPM is disabled Rafael J. Wysocki
2019-08-08 13:43                         ` Bjorn Helgaas
2019-08-08 14:47                           ` Rafael J. Wysocki
2019-08-08 17:06                             ` Rafael J. Wysocki
2019-08-08 18:39                             ` Bjorn Helgaas
2019-08-08 20:01                               ` Keith Busch
2019-08-08 20:05                               ` Mario.Limonciello
2019-08-08 20:41                               ` Rafael J. Wysocki
2019-08-09  4:47                                 ` Bjorn Helgaas
2019-08-09  8:04                                   ` Rafael J. Wysocki
2019-08-08 21:51                     ` [PATCH v3 0/2] " Rafael J. Wysocki
2019-08-08 21:55                       ` [PATCH v3 1/2] PCI: PCIe: ASPM: Introduce pcie_aspm_enabled() Rafael J. Wysocki
2019-08-09  4:50                         ` Bjorn Helgaas
2019-08-09  8:00                           ` Rafael J. Wysocki
2019-10-07 22:34                         ` Bjorn Helgaas
2019-10-08  9:27                           ` Rafael J. Wysocki
2019-10-08 21:16                             ` Bjorn Helgaas
2019-10-08 22:54                               ` Rafael J. Wysocki
2019-10-09 12:49                                 ` Bjorn Helgaas
2019-08-08 21:58                       ` [PATCH v3 2/2] nvme-pci: Allow PCI bus-level PM to be used if ASPM is disabled Rafael J. Wysocki
2019-08-08 22:13                       ` [PATCH v3 0/2] " Keith Busch
2019-08-09  8:05                         ` Rafael J. Wysocki
2019-08-09 14:52                           ` Keith Busch

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20190807104353.GA11356@lst.de \
    --to=hch@lst.de \
    --cc=Mario.Limonciello@dell.com \
    --cc=helgaas@kernel.org \
    --cc=kai.heng.feng@canonical.com \
    --cc=kbusch@kernel.org \
    --cc=keith.busch@intel.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-nvme@lists.infradead.org \
    --cc=linux-pci@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=rajatja@google.com \
    --cc=rjw@rjwysocki.net \
    --cc=sagi@grimberg.me \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).