I have to say that I always found nvme_kill_queues interface somewhat
odd.
It is..
its a core function that unquiesces the admin/io queues
assuming that they were stopped at some point by the driver.
If now there is no dependency between unquiesce and blk_mark_disk_dead,
maybe it would be a good idea to move the unquiescing to the drivers
which can pair with the quiesce itself, and rename it to
nvme_mark_namespaces_dead() or something?
Yes, something like that is probably a good idea. This entire area
is a bit of a mess to say it mildly, so maybe we need to sort that
out first.
I agree.