From: Lukas Wunner <lukas@wunner.de>
To: Mika Westerberg <mika.westerberg@linux.intel.com>
Cc: Bjorn Helgaas <helgaas@kernel.org>,
kilian.singer@quantumtechnology.info, linux-pci@vger.kernel.org,
"Rafael J. Wysocki" <rafael.j.wysocki@intel.com>
Subject: Re: PCI: Revert "PCI: Add runtime PM support for PCIe ports"
Date: Tue, 24 Jan 2017 05:53:20 +0100 [thread overview]
Message-ID: <20170124045320.GA27136@wunner.de> (raw)
In-Reply-To: <20170123211247.GB17297@lahna.fi.intel.com>
On Mon, Jan 23, 2017 at 11:12:47PM +0200, Mika Westerberg wrote:
> On Mon, Jan 23, 2017 at 02:33:35PM -0600, Bjorn Helgaas wrote:
> > On Tue, Jan 17, 2017 at 08:56:28AM -0600, Bjorn Helgaas wrote:
> > > On Tue, Dec 27, 2016 at 05:57:37PM -0600, Bjorn Helgaas wrote:
> > > > Hi Killian,
> > > >
> > > > Thanks for the report (https://bugzilla.kernel.org/show_bug.cgi?id=190861)
> > > > and all the debugging you've done. Below is a revert of the troublesome
> > > > commit. Can you test it and verify that it also fixes the problem?
> > > >
> > > > I assume Mika is looking at this and will have a better solution soon.
> > > > But if not, I'll queue this up for v4.10.
> > >
> > > Can somebody please summarize the current state of this issue? I
> > > assume somebody has already posted a better patch that should replace
> > > this naive revert, but I haven't been following the whole thread.
> >
> > This is somewhat frustrating. Is there a better patch than the revert
> > mentioned below? There was a lot of hullabaloo when I first posted
> > it, but I haven't seen a good alternative yet. I intended the revert
> > as a worst-case scenario fix, with the expectation that somebody would
> > fix the problem or at least avoid it without having to do the revert.
> > Maybe somebody posted that better fix and I just missed it?
>
> I understood that there is a patch here:
>
> https://patchwork.freedesktop.org/patch/132478/
>
> that is supposed to fix the issue. I'm waiting Kilian to test it.
That patch landed in Linus' tree tonight (commit 3846fd9b8600,
merge commit 3258943ddb90).
@Kilian: Could you retest with the tip of Linus' master branch?
Thanks,
Lukas
next prev parent reply other threads:[~2017-01-24 4:53 UTC|newest]
Thread overview: 115+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-12-27 23:57 PCI: Revert "PCI: Add runtime PM support for PCIe ports" Bjorn Helgaas
2016-12-28 9:17 ` Mika Westerberg
2016-12-28 11:29 ` Lukas Wunner
2016-12-28 16:18 ` Bjorn Helgaas
2016-12-29 9:58 ` Kilian Singer
2016-12-29 16:02 ` Kilian Singer
2016-12-29 16:20 ` Kilian Singer
2016-12-29 17:50 ` Lukas Wunner
2016-12-29 22:52 ` Kilian Singer
2016-12-29 23:02 ` Kilian Singer
2016-12-29 23:05 ` Kilian Singer
2016-12-29 23:48 ` Lukas Wunner
2016-12-29 23:20 ` Kilian Singer
2016-12-30 0:07 ` Lukas Wunner
2016-12-30 0:16 ` Kilian Singer
2016-12-30 0:24 ` Kilian Singer
2016-12-30 0:22 ` Rafael J. Wysocki
2016-12-30 0:39 ` Kilian Singer
2016-12-30 0:41 ` Rafael J. Wysocki
2016-12-30 0:45 ` Kilian Singer
2016-12-30 1:40 ` Rafael J. Wysocki
2016-12-30 1:50 ` Rafael J. Wysocki
2016-12-30 1:52 ` Rafael J. Wysocki
2016-12-30 13:37 ` Kilian Singer
2016-12-30 13:59 ` Kilian Singer
2016-12-30 14:44 ` Rafael J. Wysocki
2016-12-30 14:47 ` Rafael J. Wysocki
2017-01-02 12:22 ` Mika Westerberg
2017-01-03 17:12 ` Kilian Singer
2017-01-02 11:40 ` Lukas Wunner
2017-01-02 12:10 ` Mika Westerberg
2017-01-02 13:53 ` Mika Westerberg
2017-01-02 14:48 ` Mika Westerberg
2017-01-02 21:31 ` Rafael J. Wysocki
2017-01-03 9:51 ` Mika Westerberg
2017-01-03 15:15 ` Peter Wu
2017-01-03 16:11 ` Lukas Wunner
2017-01-03 16:31 ` Peter Wu
2017-01-03 16:44 ` Deucher, Alexander
2017-01-03 18:09 ` Lukas Wunner
2017-01-03 18:12 ` Bjorn Helgaas
2017-01-03 21:38 ` Rafael J. Wysocki
2017-01-03 21:52 ` Kilian Singer
2017-01-03 22:07 ` Rafael J. Wysocki
2017-01-03 22:25 ` Kilian Singer
2017-01-03 22:25 ` Bjorn Helgaas
2017-01-03 23:13 ` Rafael J. Wysocki
2017-01-04 0:05 ` Bjorn Helgaas
2017-01-04 1:09 ` Rafael J. Wysocki
2017-01-04 8:16 ` Lukas Wunner
2017-01-04 10:33 ` Kilian Singer
2017-01-04 12:29 ` Mika Westerberg
2017-01-04 15:50 ` Deucher, Alexander
2017-01-04 21:09 ` Peter Wu
2017-01-04 21:58 ` Rafael J. Wysocki
2017-01-04 23:21 ` David Airlie
2017-01-05 15:06 ` Lukas Wunner
2017-01-05 18:13 ` Peter Jones
2017-01-05 19:36 ` David Airlie
2017-01-09 15:11 ` Lyude Paul
2017-01-09 15:21 ` Hans de Goede
2017-01-09 18:48 ` Kilian Singer
2017-01-10 0:33 ` David Airlie
2017-01-10 9:17 ` Kilian Singer
2017-01-12 18:10 ` Lyude Paul
2017-01-24 4:59 ` Lukas Wunner
2017-01-24 19:09 ` Lyude Paul
2017-01-11 20:40 ` Lyude Paul
2017-01-12 1:13 ` Lyude Paul
2017-01-12 2:04 ` Lyude Paul
2017-01-12 2:12 ` Lukas Wunner
2017-01-17 15:55 ` Mika Westerberg
2017-01-17 18:06 ` Lyude Paul
2017-01-17 19:10 ` Bjorn Helgaas
2017-01-17 19:49 ` Lyude Paul
2017-01-07 11:45 ` Hans de Goede
2017-01-07 12:16 ` Lukas Wunner
2017-01-09 23:00 ` Peter Jones
2017-01-10 0:17 ` David Airlie
2017-01-10 1:24 ` Lukas Wunner
2017-01-10 2:15 ` David Airlie
2017-01-11 11:04 ` Hans de Goede
2017-01-11 13:24 ` Kilian Singer
2017-01-11 13:26 ` Hans de Goede
2017-01-11 16:24 ` Peter Jones
2017-01-11 19:20 ` Kilian Singer
2017-01-05 10:49 ` Mika Westerberg
2017-01-05 14:19 ` Rafael J. Wysocki
2017-01-05 14:20 ` Mika Westerberg
2017-01-05 14:23 ` Rafael J. Wysocki
2017-01-05 14:42 ` Lukas Wunner
2017-01-06 1:21 ` Rafael J. Wysocki
2017-01-07 6:50 ` Mika Westerberg
2017-01-07 11:35 ` Peter Wu
2017-01-07 12:19 ` Lukas Wunner
2017-01-07 12:36 ` Peter Wu
2017-01-08 14:05 ` Lukas Wunner
2017-01-04 21:55 ` Rafael J. Wysocki
2017-01-03 21:26 ` Rafael J. Wysocki
2017-01-03 17:37 ` Kilian Singer
2017-01-03 17:10 ` Kilian Singer
2017-01-03 16:59 ` Kilian Singer
2017-01-03 17:08 ` Kilian Singer
2016-12-30 0:19 ` Rafael J. Wysocki
2016-12-30 14:48 ` Rafael J. Wysocki
2017-01-17 14:56 ` Bjorn Helgaas
2017-01-17 15:49 ` Kilian Singer
2017-01-23 20:33 ` Bjorn Helgaas
2017-01-23 21:12 ` Mika Westerberg
2017-01-24 4:53 ` Lukas Wunner [this message]
2017-01-24 20:01 ` Bjorn Helgaas
2017-01-25 9:48 ` Mika Westerberg
2017-01-25 16:05 ` Kilian Singer
2017-01-25 16:31 ` Mika Westerberg
2017-01-25 17:58 ` Bjorn Helgaas
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=20170124045320.GA27136@wunner.de \
--to=lukas@wunner.de \
--cc=helgaas@kernel.org \
--cc=kilian.singer@quantumtechnology.info \
--cc=linux-pci@vger.kernel.org \
--cc=mika.westerberg@linux.intel.com \
--cc=rafael.j.wysocki@intel.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.