Linux Power Management development
 help / color / mirror / Atom feed
From: "Rafael J. Wysocki" <rjw@rjwysocki.net>
To: Thomas Gleixner <tglx@linutronix.de>
Cc: Linus Torvalds <torvalds@linux-foundation.org>,
	Bjorn Helgaas <helgaas@kernel.org>,
	Maarten Lankhorst <dev@mblankhorst.nl>,
	Michal Hocko <mhocko@kernel.org>,
	Andy Lutomirski <luto@kernel.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	the arch/x86 maintainers <x86@kernel.org>,
	Daniel Vetter <daniel.vetter@intel.com>,
	Bjorn Helgaas <bhelgaas@google.com>,
	"Rafael J. Wysocki" <rafael.j.wysocki@intel.com>,
	linux-pci@vger.kernel.org, linux-pm@vger.kernel.org
Subject: Re: Linux 4.15-rc2: Regression in resume from ACPI S3
Date: Wed, 13 Dec 2017 23:48:45 +0100	[thread overview]
Message-ID: <15300261.gs7DNfzHs2@aspire.rjw.lan> (raw)
In-Reply-To: <alpine.DEB.2.20.1712132201380.1885@nanos>

On Wednesday, December 13, 2017 10:06:40 PM CET Thomas Gleixner wrote:
> On Wed, 13 Dec 2017, Thomas Gleixner wrote:
> > On Wed, 13 Dec 2017, Thomas Gleixner wrote:
> > > On Wed, 13 Dec 2017, Linus Torvalds wrote:
> > > 
> > > > On Wed, Dec 13, 2017 at 8:41 AM, Thomas Gleixner <tglx@linutronix.de> wrote:
> > > > >
> > > > > Definitely. That was fragile forever but puzzles me is that I can't figure
> > > > > out what now causes that spurious interrupt to surface out of the blue.
> > > > 
> > > > Perhaps just timing?
> > > 
> > > That's what I'm trying to figure out right now, because that is the only
> > > sensible explanation left. The whole machinery of suspend is exactly the
> > > same with and without the vector changes. I instrumented all functions
> > > involved and the picture is the same. I even do not see any fundamental
> > > timing differences where one would say: That's it.
> > > 
> > > What puzzles me even more is that in the range of commits I'm fiddling with
> > > there is no other change than the vector management stuff and the point
> > > where it breaks makes no sense at all. The point Maarten bisected it to
> > > works nicely here, so that might just point to a very subtle timing issue.
> > 
> > After doing more debugging on this it turns out that this looks like a
> > legacy interrupt coming in. The vector number is always 55, which is legacy
> > IRQ 7 as seen from the PIC. The corresponding IOAPIC interrupt pin is
> > masked and vector 55 is completely unused.
> > 
> > More questions than answers. Still investigating.
> 
> And it does not explain Maartens report which gets a spurious vector 33 on
> CPU4 after the non boot cpus have been brought online again. And that's the
> vector which was assigned before the affinity was moved by unplugging CPU4.
> 
> Hrmpf. Even more mystery to solve.

Any chance to look at /proc/interrupts from a machine where that can be
reproduced?

I'm also curious if that can be reproduced by doing CPU offline/online
without suspending?

  reply	other threads:[~2017-12-13 22:49 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <168050887.sZlTFXWCmO@aspire.rjw.lan>
     [not found] ` <CA+55aFwsMuHUBQz5kDNwRf17JnasXMWjvmLq5qXGH-694yeq1w@mail.gmail.com>
     [not found]   ` <20171206121452.GA6320@dhcp22.suse.cz>
     [not found]     ` <db81e89d-eee6-868c-dc10-c028f4c4be84@mblankhorst.nl>
     [not found]       ` <alpine.DEB.2.20.1712061344560.1724@nanos>
     [not found]         ` <0f1d3d63-fa10-5cef-8014-81753dc60243@mblankhorst.nl>
     [not found]           ` <alpine.DEB.2.20.1712061514420.1724@nanos>
     [not found]             ` <57c8679e-1b88-c9ad-2299-2bea7560b28f@mblankhorst.nl>
     [not found]               ` <alpine.DEB.2.20.1712081129450.1840@nanos>
     [not found]                 ` <alpine.DEB.2.20.1712131507160.1885@nanos>
2017-12-13 16:23                   ` Linux 4.15-rc2: Regression in resume from ACPI S3 Bjorn Helgaas
2017-12-13 16:41                     ` Thomas Gleixner
2017-12-13 17:45                       ` Linus Torvalds
2017-12-13 18:19                         ` Thomas Gleixner
2017-12-13 20:52                           ` Thomas Gleixner
2017-12-13 21:06                             ` Thomas Gleixner
2017-12-13 22:48                               ` Rafael J. Wysocki [this message]
2017-12-14 11:54                               ` Thomas Gleixner
2017-12-14 12:12                                 ` Rafael J. Wysocki
2017-12-14 12:30                                   ` Thomas Gleixner
2017-12-14 15:30                                     ` Rafael J. Wysocki
2017-12-14 15:52                                       ` Thomas Gleixner
2017-12-14 15:54                                         ` Rafael J. Wysocki
2017-12-14 16:17                                           ` Maarten Lankhorst
2017-12-15  2:07                                           ` [PATCH] PCI / PM: Force devices to D0 in pci_pm_thaw_noirq() Rafael J. Wysocki
2017-12-15 14:28                                             ` Rafael J. Wysocki
2017-12-15 18:30                                             ` Bjorn Helgaas
2017-12-15 23:44                                               ` Rafael J. Wysocki
2017-12-14 13:24                                 ` Linux 4.15-rc2: Regression in resume from ACPI S3 Thomas Gleixner
2017-12-14 19:03                                 ` Linus Torvalds
2017-12-14 22:36                                   ` Thomas Gleixner
2017-12-14 22:47                                     ` Linus Torvalds
2017-12-15  9:05                                       ` Thomas Gleixner
2017-12-15  0:34                                     ` Rafael J. Wysocki
2017-12-13 22:39                           ` Rafael J. Wysocki
2017-12-13 23:26                             ` Rafael J. Wysocki

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=15300261.gs7DNfzHs2@aspire.rjw.lan \
    --to=rjw@rjwysocki.net \
    --cc=bhelgaas@google.com \
    --cc=daniel.vetter@intel.com \
    --cc=dev@mblankhorst.nl \
    --cc=helgaas@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pci@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=luto@kernel.org \
    --cc=mhocko@kernel.org \
    --cc=rafael.j.wysocki@intel.com \
    --cc=tglx@linutronix.de \
    --cc=torvalds@linux-foundation.org \
    --cc=x86@kernel.org \
    /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