Re: [PATCH 1/6] hw/block/nvme: fix shutdown/reset logic

[Keith Busch]

On Mon, Jan 11, 2021 at 01:32:18PM +0100, Klaus Jensen wrote:
> From: Klaus Jensen <k.jensen@samsung.com>
> 
> A shutdown is only about flushing stuff. It is the host that should
> delete any queues, so do not perform a reset here.
> 
> Also, on shutdown, make sure that the PMR is flushed if in use.
> 
> Fixes: 368f4e752cf9 ("hw/block/nvme: Process controller reset and shutdown 
> differently")
> Signed-off-by: Klaus Jensen <k.jensen@samsung.com>

Yes, and exiting a controller shutdown state requires a controller
reset, so a functioning host shouldn't require the controller behave the
way it's currently done.

Reviewed-by: Keith Busch <kbusch@kernel.org>