linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Pavel Machek <pavel@ucw.cz>
To: Wilmer van der Gaast <wilmer@gaast.net>, bhelgaas@google.com
Cc: rafael.j.wysocki@intel.com, linux-kernel@vger.kernel.org
Subject: Re: Machine crashes right *after* ~successful resume
Date: Sun, 12 Oct 2014 22:40:32 +0200	[thread overview]
Message-ID: <20141012204032.GA11171@amd> (raw)
In-Reply-To: <543AA2FE.20105@gaast.net>

Bjorn, any ideas?

Would it be feasible to revert 2e8b... to see if it fixes it on 3.17?

Thanks,
									Pavel

On Sun 2014-10-12 16:49:18, Wilmer van der Gaast wrote:
> Hello,
> 
> Many thanks for your response!
> 
> On 12-10-14 15:30, Pavel Machek wrote:
> >
> >Has it ever worked ok? ...aha, in 3.10, ok.
> >
> Correct. And I've tried a few more kernels now, compiled on my own. 3.17
> still has this issue, 3.10 is completely fine all the way up to 3.10.57
> (I've tested just under 50 cycles last night). 3.11 I tried but it seems to
> have other suspend-resume stability issues not present anymore in later
> kernels, I've mostly not used those results.
> 
> git bisect: I've finally succeeded! I've tried automating it completely, but
> sadly Gigabyte couldn't be bothered wiring up the motherboard to make the
> watchdog work. :-(
> 
> The culprit appears to be this one: 2e8b5f621dbe29425906852c6079afb6b28720cb
> 
> Merge: 07f2daa fed2451
> Author: Bjorn Helgaas <bhelgaas@google.com>
> Date:   Wed Aug 28 20:55:41 2013 -0600
> 
>     Merge branch 'pci/misc' into next
> 
>     * pci/misc:
>       PCI: Remove pcie_cap_has_devctl()
>       PCI: Support PCIe Capability Slot registers only for ports with slots
>       PCI: Remove PCIe Capability version checks
>       PCI: Allow PCIe Capability link-related register access for switches
>       PCI: Add offsets of PCIe capability registers
>       PCI: Tidy bitmasks and spacing of PCIe capability definitions
>       PCI: Remove obsolete comment reference to pci_pcie_cap2()
>       PCI: Clarify PCI_EXP_TYPE_PCI_BRIDGE comment
>       PCI: Rename PCIe capability definitions to follow convention
>       PCI: Disable decoding for BAR sizing only when it was actually enabled
>       PCI: Add comment about needing pci_msi_off() even when
> CONFIG_PCI_MSI=n
>       PCI: Add pcibios_pm_ops for optional arch-specific hibernate
> functionality
> 
> I've then tried to narrow down which of the merged changes is my issue but
> with no luck, possibly because there's a problem with a combination of one
> of these changes, and a change that was not in the pci/misc branch at the
> time. I could do a manual test instead.
> 
> >>I've already tried to skip the NVidia + VMware modules at boot time (as you
> >>can see from the logs they're not loaded at any point), but it didn't help.
> >>I could try omitting more modules.
> >Yes, try with minimal modules (and no s2ram) would be nice.
> >									
> I've tried unloading a bunch of modules (sound and NIC IIRC), same results.
> I can try this again with an even more minimal set. If this improves the
> situation, I'll post again.
> 
> 
> Wilmer van der Gaast.
> 

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

  reply	other threads:[~2014-10-12 20:40 UTC|newest]

Thread overview: 51+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-10-07 23:20 Machine crashes right *after* ~successful resume Wilmer van der Gaast
2014-10-12 14:30 ` Pavel Machek
2014-10-12 15:49   ` Wilmer van der Gaast
2014-10-12 20:40     ` Pavel Machek [this message]
2014-10-12 23:47       ` Wilmer van der Gaast
2014-10-13 15:06       ` Rafael J. Wysocki
2014-10-15 11:16         ` Wilmer van der Gaast
2014-10-15 13:58           ` Bjorn Helgaas
2014-10-15 18:39             ` Yinghai Lu
2014-10-15 23:34               ` Wilmer van der Gaast
2014-10-16  4:32                 ` Yinghai Lu
2014-10-16  9:36                   ` Wilmer van der Gaast
2014-10-16 16:36                     ` Yinghai Lu
2014-10-16 21:08                       ` Wilmer van der Gaast
2014-10-18 21:28                         ` Yinghai Lu
2014-10-18 23:57                           ` Wilmer van der Gaast
2014-10-19  4:29                             ` Yinghai Lu
2014-10-19 10:48                               ` Wilmer van der Gaast
2014-10-21 21:40                               ` Wilmer van der Gaast
2014-10-21 23:15                                 ` Yinghai Lu
2014-10-22 12:53                                   ` Wilmer van der Gaast
2014-10-26 21:53                                     ` Yinghai Lu
2014-10-27 10:50                                       ` Wilmer van der Gaast
2014-10-27 18:23                                         ` Yinghai Lu
2014-10-27 22:22                                           ` Wilmer van der Gaast
2014-10-27 23:41                                             ` Yinghai Lu
2014-10-28  0:03                                               ` Wilmer van der Gaast
2014-10-28  1:12                                                 ` Yinghai Lu
2014-10-28  4:03                                                   ` Yinghai Lu
2014-10-28 10:23                                                     ` Wilmer van der Gaast
2014-10-28 23:34                                                   ` Wilmer van der Gaast
2014-10-29  5:17                                                     ` Yinghai Lu
2014-10-29  9:37                                                       ` Wilmer van der Gaast
2014-10-30  0:53                                                         ` Yinghai Lu
2014-10-30 10:36                                                           ` Wilmer van der Gaast
2014-10-30 16:57                                                             ` Yinghai Lu
2014-10-30 21:54                                                               ` Wilmer van der Gaast
2014-10-30 23:02                                                                 ` Yinghai Lu
2014-10-30 23:24                                                                   ` Wilmer van der Gaast
2014-10-31  0:43                                                                     ` Yinghai Lu
2014-10-31  2:13                                                                       ` Yinghai Lu
2014-10-31  9:39                                                                         ` Wilmer van der Gaast
2014-10-31 16:11                                                                           ` Yinghai Lu
2014-10-31 21:13                                                                             ` Wilmer van der Gaast
2014-10-31 21:22                                                                               ` Yinghai Lu
2014-10-31 23:18                                                                                 ` Yinghai Lu
2014-11-01  0:00                                                                                   ` Wilmer van der Gaast
2014-11-01  2:10                                                                                     ` Yinghai Lu
2014-11-02 23:16                                                                                       ` Wilmer van der Gaast
2014-10-27 21:21                                         ` Pavel Machek
2014-10-19  8:07                             ` Pavel Machek

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=20141012204032.GA11171@amd \
    --to=pavel@ucw.cz \
    --cc=bhelgaas@google.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rafael.j.wysocki@intel.com \
    --cc=wilmer@gaast.net \
    /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).