[PATCH 3/3] nvme: enable retries for authentication commands
Hannes Reinecke
hare at suse.de
Sun Jan 28 22:24:11 PST 2024
On 1/29/24 07:00, Christoph Hellwig wrote:
> On Sat, Jan 27, 2024 at 10:37:46AM +0100, hare at kernel.org wrote:
>> From: Hannes Reinecke <hare at suse.de>
>>
>> Authentication commands are normal NVMe commands, and as such
>> can return a status where NVME_SC_DNR is not set, indicating
>> that the command should be retried.
>
> What command is not normal?
>
Things like 'property set' or 'property get', which are used in eg
nvmf_reg_read() et al. They are used to retrieve or get register
settings, and it's not quite clear if it's safe (or prudent) to retry them.
>> This patch enables retries by setting NVME_SUBMIT_RETRY for
>> __nvme_submit_sync_cmd().
>
> I think what should go into the commit log is why the retries are
> desirable here, and not for other admin commands.
>
Okay.
> Otherwise this looks good to me.
Cheers,
Hannes
--
Dr. Hannes Reinecke Kernel Storage Architect
hare at suse.de +49 911 74053 688
SUSE Software Solutions GmbH, Maxfeldstr. 5, 90409 Nürnberg
HRB 36809 (AG Nürnberg), GF: Ivo Totev, Andrew McDonald,
Werner Knoblich
More information about the Linux-nvme
mailing list