[PATCH] nvme: move nvme_stop_keep_alive() back to original position

Hannes Reinecke hare at suse.de
Tue Nov 21 06:41:52 PST 2023


On 11/21/23 10:23, Sagi Grimberg wrote:
> 
>> Stopping keep-alive not only stops the keep-alive workqueue,
>> but also needs to be synchronized with I/O termination as we
>> must not send a keep-alive command after all I/O had been
>> terminated.
>> So to avoid any regressions move the call to stop_keep_alive()
>> back to its original position and ensure that keep-alive is
>> correctly stopped failing to setup the admin queue.
>>
>> Fixes: 4733b65d82bd ("nvme: start keep-alive after admin queue setup")
> 
> It should be folded in I guess. I suggest to either suggest an
> alternative phrasing of the patch or resend an alternative patch
> set.
> 
Well, the original patch has already been merged to the nvme-6.7 tree,
so I've attached the 'Fixes' tag.
But let me know how you'd like to play it.

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