All of lore.kernel.org
 help / color / mirror / Atom feed
From: keith.busch@intel.com (Keith Busch)
Subject: [PATCH 2/2] nvme/quirk: Add a delay before checking for adapter readiness
Date: Tue, 12 Apr 2016 20:02:16 +0000	[thread overview]
Message-ID: <20160412200216.GA4183@localhost.localdomain> (raw)
In-Reply-To: <201604121948.u3CJmrdR002584@d03av01.boulder.ibm.com>

On Tue, Apr 12, 2016@02:48:46PM -0500, Murali N Iyer wrote:
> Jeff, David,
> 
> You might not see the issue 100% of the time if you don't activate the
> different slot. Try with any kernel that provides "reset_controller" interface.
> RHEL 7.2, Ubuntu 16.04 etc. does.
> 
> Example: you are running with FW slot #1
> 
> # nvme fw-log /dev/nvme4
> Firmware Log for device:/dev/nvme4
> afi : 0x11
> frs1 : 0x3430315050494d4b (KMIPP104)
> frs2 : 0x3430315050494d4b (KMIPP104)
> frs3 : 0x3430315050494d4b (KMIPP104)
> frs4 : 0x3430315050494d4b (KMIPP104)
> 
> # echo 1 > /sys/class/nvme/nvme4/reset_controller ==.> This might work most of
> the time
> 
> Now, activate slot #4 and then reset_controller
> 
> # nvme fw-activate /dev/nvme4 -a 2 -s 4
> Success activating firmware action:2 slot:4
> 
> # echo 1 > /sys/class/nvme/nvme4/reset_controller ==> This generates EEH

As older kernels did not export the reset_controller interface, unloading
and reloading the driver with modprobe should reproduce the sequence
you're describing. The method is not as convenient, but should be enough
for a 3rd party with the same controller to confirm results.

  parent reply	other threads:[~2016-04-12 20:02 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-04-11 20:15 [PATCH 1/2] nvme: Fix pci_device_id table by adding the most generic devices in the end Guilherme G. Piccoli
2016-04-11 20:15 ` [PATCH 2/2] nvme/quirk: Add a delay before checking for adapter readiness Guilherme G. Piccoli
2016-04-12 17:56   ` Jeff Lien
     [not found]     ` <201604121948.u3CJmrdR002584@d03av01.boulder.ibm.com>
2016-04-12 20:02       ` Keith Busch [this message]
2016-04-12 20:13         ` Guilherme G. Piccoli
2016-04-12 17:58   ` David Darrington
2016-04-12 20:12     ` Guilherme G. Piccoli
2016-04-12 20:31       ` David Darrington
2016-04-11 20:48 ` [PATCH 1/2] nvme: Fix pci_device_id table by adding the most generic devices in the end Matthew Wilcox
2016-04-11 20:57   ` Guilherme G. Piccoli

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=20160412200216.GA4183@localhost.localdomain \
    --to=keith.busch@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.