All of lore.kernel.org
 help / color / mirror / Atom feed
From: marc@merlins.org (Marc MERLIN)
Subject: [PATCH 5/5] nvme/pci: Complete all stuck requests
Date: Wed, 13 Dec 2017 19:36:33 -0800	[thread overview]
Message-ID: <20171214033633.4oncj3kcsvxv4kic@merlins.org> (raw)
In-Reply-To: <20170313153319.fmy6ww72fjtx74xq@merlins.org> <20170313143649.GC6994@localhost.localdomain> <20170311022612.GB28220@merlins.org> <20170215181435.36saagrln25im7ml@merlins.org>

On Mon, Mar 13, 2017@08:33:19AM -0700, Marc MERLIN wrote:
> On Mon, Mar 13, 2017@10:36:50AM -0400, Keith Busch wrote:
> > On Fri, Mar 10, 2017@06:26:12PM -0800, Marc MERLIN wrote:
> > > Hi Keith,
> > > 
> > > From what I think I've seen, your patches did get in. Thank you again for
> > > getting them through.
> > > 
> > > I'm assuming that they were too late for 4.10 but would be in 4.11, correct?
> > > 
> > > The 4.10 branch you gave me to test sadly has multiple other bugs that make
> > > it not very desirable on my laptop for every day use, so I'm looking at
> > > switching to a standard released kernel as soon as practical.
> > > I'm also happy to install your patches on top of a release kernel, if they
> > > do apply until they make it to the next mainline release (but don't do extra
> > > rebase for me if that's needed, I'm ok waiting if they don't apply as is)
> > 
> > No problem, thanks for reporting the issue and sticking with it through
> > all the delays.
> > 
> > This was too late for 4.10, but applied with the 4.11 merge. The commit
> > is here:
> > 
> >   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=302ad8cc09339ea261eef58a8d5f4a116a8ffda5
>  
> Thank yuou.
> 
> > Sorry to hear about the other bugs. I think this should be readily
> > portable to the 4.9 longterm stable brach if we get the OK to apply the
> > new blk-mq API's. I'll do some testing this week and check with the
> > maintainers.
> 
> No worries, I didn't expect a random in between release kernel to be
> fully stable. If I just need to wait for 4.11, I can certainly do that.

So, I was all happy and all with 4.11, and it still works fine.
S3 sleep just works, hibernate just works.
And then 4.12 came, and S3 sleep would not work reliably
4.13 came, same thing
4.14, again, same thing :(

Currently when I put the laptop to sleep, it sometimes, but just
sometimes, keeps the thinkpad light in a slow pulsing light as if it
were still in sleep mode, and just never wakes up.

Sadly, this is crappy to bisect since it's not reliably reproducible.

Before I go through that pain, are there any suggestions or ideas I
should try?

Thanks,
Marc
-- 
"A mouse is a device used to point at the xterm you want to type in" - A.S.R.
Microsoft is to operating systems ....
                                      .... what McDonalds is to gourmet cooking
Home page: http://marc.merlins.org/                         | PGP 1024R/763BE901

  reply	other threads:[~2017-12-14  3:36 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-02-10 23:15 [PATCH 0/5] NVMe pci fixes, for-4.11 Keith Busch
2017-02-10 23:15 ` [PATCH 1/5] nvme/pci: Disable on removal when disconnected Keith Busch
2017-02-13 10:18   ` Johannes Thumshirn
2017-02-13 13:51   ` Christoph Hellwig
2017-02-10 23:15 ` [PATCH 2/5] nvme/pci: Cancel work after watchdog disabled Keith Busch
2017-02-13 10:25   ` Johannes Thumshirn
2017-02-13 13:51   ` Christoph Hellwig
2017-02-10 23:15 ` [PATCH 3/5] nvme/core: Fix race kicking freed request_queue Keith Busch
2017-02-13 10:33   ` Johannes Thumshirn
2017-02-13 13:53   ` Christoph Hellwig
2017-02-10 23:15 ` [PATCH 4/5] nvme/pci: No special case for queue busy on IO Keith Busch
2017-02-13 13:53   ` Christoph Hellwig
2017-02-10 23:15 ` [PATCH 5/5] nvme/pci: Complete all stuck requests Keith Busch
2017-02-15  9:50   ` Sagi Grimberg
2017-02-15 15:46     ` Keith Busch
2017-02-15 16:04       ` Marc MERLIN
2017-02-15 17:36         ` J Freyensee
2017-02-16  9:12         ` Sagi Grimberg
2017-02-16 22:51           ` Keith Busch
2017-02-17  8:25             ` Christoph Hellwig
2017-02-15 18:14   ` Marc MERLIN
2017-12-14  3:36     ` Marc MERLIN [this message]
2018-02-28  2:22       ` Marc MERLIN
2017-02-17 15:27   ` Christoph Hellwig
2017-02-17 16:33     ` Keith Busch
2017-02-20 10:05       ` Christoph Hellwig
2017-02-21 15:57         ` Keith Busch
2017-02-22  7:17           ` Christoph Hellwig
2017-02-22 14:45             ` Keith Busch
2017-02-23 15:06               ` Christoph Hellwig
2017-02-23 15:21                 ` Keith Busch
2017-02-23 15:16                   ` Christoph Hellwig
2017-02-21 21:55       ` Sagi Grimberg
2017-02-21 23:26         ` Keith Busch
2017-02-15  9:40 ` [PATCH 0/5] NVMe pci fixes, for-4.11 Sagi Grimberg
     [not found] <20170313153319.fmy6ww72fjtx74xq@merlins.org>
     [not found] ` <20170313143649.GC6994@localhost.localdomain>

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=20171214033633.4oncj3kcsvxv4kic@merlins.org \
    --to=marc@merlins.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 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.