From: Erwan Velu <e.velu@criteo.com>
To: "Martin K. Petersen" <martin.petersen@oracle.com>,
Erwan Velu <erwanaliasr1@gmail.com>
Cc: Don Brace <don.brace@microsemi.com>,
"James E.J. Bottomley" <jejb@linux.ibm.com>,
"esc.storagedev@microsemi.com" <esc.storagedev@microsemi.com>,
"linux-scsi@vger.kernel.org" <linux-scsi@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v3] scsi: smartpqi_init: Reporting 'logical unit failure'
Date: Mon, 11 Mar 2019 16:43:25 +0000 [thread overview]
Message-ID: <21c537b9-5d45-d170-cfdb-d49a3e36b924@criteo.com> (raw)
In-Reply-To: <yq1y35rzz31.fsf@oracle.com>
Le 06/03/2019 à 18:34, Martin K. Petersen a écrit :
> Erwan,
>
>> When the HARDWARE_ERROR/0x3e/0x1 case is triggered, the logical volume
>> is offlined. When reading the kernel log, the reason why the device
>> got offlined isn't reported to the user. This situation makes
>> difficult for admins to estimate the root cause of the issue they
>> analize.
While I was debugging this scenario, I was wondering if some other cases
were possible.
The current code is considering (sshdr.asc == 0x3e && sshdr.ascq ==
0x1), but what if ascq have a different value here ?
The specification (http://www.t10.org/lists/asc-num.htm#ASC_3E) reports
other sub-values like ASCQ==02 which means a timeout on the lun.
So, does the raid controllers supported by smartpqi can generates these
other values ? If so, how/where are they handled ?
I was considering at least, to a switch statement on sshdr.ascq with a
0x1 case on the current code and a a default one that prints at least a
message saying that a message got received but not handled.
Thanks !
Erwan,
prev parent reply other threads:[~2019-03-11 16:43 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-02-27 16:31 [PATCH] " Erwan Velu
2019-02-28 13:09 ` Erwan Velu
2019-02-28 20:03 ` Elliott, Robert (Persistent Memory)
2019-03-01 14:59 ` Erwan Velu
2019-03-01 14:58 ` [PATCH v2] " Erwan Velu
2019-03-01 15:26 ` James Bottomley
2019-03-01 15:43 ` Erwan Velu
2019-03-01 15:56 ` James Bottomley
2019-03-01 16:00 ` Erwan Velu
2019-03-01 16:08 ` [PATCH v3] " Erwan Velu
2019-03-05 22:30 ` Don.Brace
2019-03-06 17:34 ` Martin K. Petersen
2019-03-11 16:36 ` Erwan Velu
2019-03-11 16:43 ` Erwan Velu [this message]
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=21c537b9-5d45-d170-cfdb-d49a3e36b924@criteo.com \
--to=e.velu@criteo.com \
--cc=don.brace@microsemi.com \
--cc=erwanaliasr1@gmail.com \
--cc=esc.storagedev@microsemi.com \
--cc=jejb@linux.ibm.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
--cc=martin.petersen@oracle.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®