diff options
| author | Minwoo Im <minwoo.im@samsung.com> | 2023-04-18 09:26:21 +0900 |
|---|---|---|
| committer | Klaus Jensen <k.jensen@samsung.com> | 2023-06-28 11:13:42 +0200 |
| commit | cab1da59c2ff3bf08d0a7becf9b51e43a724a85c (patch) | |
| tree | 0c6fc6e838320b24f91fb5778be6f88c40e5197d /hw/nvme | |
| parent | 7491e0e4096f426de4a47fa241442c84e63c5a64 (diff) | |
hw/nvme: consider COPY command in nvme_aio_err
If we don't have NVME_CMD_COPY consideration in the switch statement in
nvme_aio_err(), it will go to have NVME_INTERNAL_DEV_ERROR and
`req->status` will be ovewritten to it. During the aio context, it
might set the NVMe status field like NVME_CMD_SIZE_LIMIT, but it's
overwritten in the nvme_aio_err().
Add consideration for the NVME_CMD_COPY not to overwrite the status at
the end of the function.
Signed-off-by: Minwoo Im <minwoo.im@samsung.com>
Reviewed-by: Klaus Jensen <k.jensen@samsung.com>
Signed-off-by: Klaus Jensen <k.jensen@samsung.com>
Diffstat (limited to 'hw/nvme')
| -rw-r--r-- | hw/nvme/ctrl.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/hw/nvme/ctrl.c b/hw/nvme/ctrl.c index 020f37a780..e031c2250a 100644 --- a/hw/nvme/ctrl.c +++ b/hw/nvme/ctrl.c @@ -1755,6 +1755,7 @@ static void nvme_aio_err(NvmeRequest *req, int ret) case NVME_CMD_WRITE: case NVME_CMD_WRITE_ZEROES: case NVME_CMD_ZONE_APPEND: + case NVME_CMD_COPY: status = NVME_WRITE_FAULT; break; default: |