[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [RFC PATCH 1/3] hw/block/nvme: set NVME_DNR in a single place
From: |
Klaus Jensen |
Subject: |
Re: [RFC PATCH 1/3] hw/block/nvme: set NVME_DNR in a single place |
Date: |
Wed, 10 Feb 2021 21:19:43 +0100 |
On Feb 11 04:52, Minwoo Im wrote:
> @@ -945,6 +945,11 @@ static void nvme_post_cqes(void *opaque)
> static void nvme_enqueue_req_completion(NvmeCQueue *cq, NvmeRequest *req)
> {
> assert(cq->cqid == req->sq->cqid);
> +
> + if (req->status != NVME_SUCCESS) {
> + req->status |= NVME_DNR;
> + }
There are status codes where we do not set the DNR bit (e.g. Data
Transfer Error, and that might be the only one actually).
Maybe a switch such that we do not explicitly set DNR for Data Transfer
Error (and any other errors we identify), but only if we set it earlier
in the stack.
signature.asc
Description: PGP signature