[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH for-7.2 v2 1/6] hw/nvme: fix accidental reintroduction of redunda
From: |
Klaus Jensen |
Subject: |
[PATCH for-7.2 v2 1/6] hw/nvme: fix accidental reintroduction of redundant code |
Date: |
Thu, 10 Nov 2022 08:05:18 +0100 |
From: Klaus Jensen <k.jensen@samsung.com>
Commit 44219b6029fc ("hw/nvme: 64-bit pi support") accidentially
reintroduced code that was removed in commit a6de6ed5092c ("hw/nvme:
move format parameter parsing").
It is beneign, but get rid of it anyway.
Fixes: 44219b6029fc ("hw/nvme: 64-bit pi support")
Signed-off-by: Klaus Jensen <k.jensen@samsung.com>
---
hw/nvme/ctrl.c | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/hw/nvme/ctrl.c b/hw/nvme/ctrl.c
index e23af4db91ae..918af03d32be 100644
--- a/hw/nvme/ctrl.c
+++ b/hw/nvme/ctrl.c
@@ -5847,9 +5847,6 @@ static void nvme_format_bh(void *opaque)
NvmeFormatAIOCB *iocb = opaque;
NvmeRequest *req = iocb->req;
NvmeCtrl *n = nvme_ctrl(req);
- uint32_t dw10 = le32_to_cpu(req->cmd.cdw10);
- uint8_t lbaf = dw10 & 0xf;
- uint8_t pi = (dw10 >> 5) & 0x7;
uint16_t status;
int i;
@@ -5871,7 +5868,7 @@ static void nvme_format_bh(void *opaque)
goto done;
}
- status = nvme_format_check(iocb->ns, lbaf, pi);
+ status = nvme_format_check(iocb->ns, iocb->lbaf, iocb->pi);
if (status) {
req->status = status;
goto done;
--
2.38.1
- [PATCH for-7.2 v2 0/6] hw/nvme: misc fixes, Klaus Jensen, 2022/11/10
- [PATCH for-7.2 v2 1/6] hw/nvme: fix accidental reintroduction of redundant code,
Klaus Jensen <=
- [PATCH for-7.2 v2 2/6] hw/nvme: fix cancellation of format operations, Klaus Jensen, 2022/11/10
- [PATCH for-7.2 v2 4/6] hw/nvme: fix cancellation handling in zone reset, Klaus Jensen, 2022/11/10
- [PATCH for-7.2 v2 5/6] hw/nvme: fix cancellation handling in dsm, Klaus Jensen, 2022/11/10
- [PATCH for-7.2 v2 6/6] hw/nvme: fix numzrwa handling, Klaus Jensen, 2022/11/10
- [PATCH for-7.2 v2 3/6] hw/nvme: fix flush cancel, Klaus Jensen, 2022/11/10