[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