[RFC v4 1/1] nvme: add passthrough admin error logging opt-in
Chaitanya Kulkarni
chaitanyak at nvidia.com
Sun Apr 9 14:32:10 PDT 2023
On 4/7/23 00:48, hch at lst.de wrote:
> On Fri, Apr 07, 2023 at 04:45:59AM +0000, Chaitanya Kulkarni wrote:
>> + const char *device_name = dev_name(nr->ctrl->device);
>>
>> + if (ns)
>> + device_name = ns->disk ? ns->disk->disk_name : "?";
> I don't see how ns->disk could ever be NULL here. So this could
> be a simple
>
> ns ? ns->disk->disk->name : dev_name(nr->ctrl->device)
>
> in the parameter list. We'd still need to also call
> nvme_get_admin_opcode_str to get the actual command name as well.
>
sent out a tested V5 with all the fixes I mentioned and your
suggestion...
-ck
More information about the Linux-nvme
mailing list