[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v4 06/18] nvme: refactor nvme_addr_read
From: |
Klaus Jensen |
Subject: |
[PATCH v4 06/18] nvme: refactor nvme_addr_read |
Date: |
Wed, 22 Apr 2020 13:01:40 +0200 |
From: Klaus Jensen <address@hidden>
Pull the controller memory buffer check to its own function. The check
will be used on its own in later patches.
Signed-off-by: Klaus Jensen <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Reviewed-by: Maxim Levitsky <address@hidden>
Reviewed-by: Keith Busch <address@hidden>
---
hw/block/nvme.c | 16 ++++++++++++----
1 file changed, 12 insertions(+), 4 deletions(-)
diff --git a/hw/block/nvme.c b/hw/block/nvme.c
index 4058f2c79796..623a88be93dc 100644
--- a/hw/block/nvme.c
+++ b/hw/block/nvme.c
@@ -66,14 +66,22 @@
static void nvme_process_sq(void *opaque);
+static bool nvme_addr_is_cmb(NvmeCtrl *n, hwaddr addr)
+{
+ hwaddr low = n->ctrl_mem.addr;
+ hwaddr hi = n->ctrl_mem.addr + int128_get64(n->ctrl_mem.size);
+
+ return addr >= low && addr < hi;
+}
+
static void nvme_addr_read(NvmeCtrl *n, hwaddr addr, void *buf, int size)
{
- if (n->cmbsz && addr >= n->ctrl_mem.addr &&
- addr < (n->ctrl_mem.addr + int128_get64(n->ctrl_mem.size))) {
+ if (n->cmbsz && nvme_addr_is_cmb(n, addr)) {
memcpy(buf, (void *)&n->cmbuf[addr - n->ctrl_mem.addr], size);
- } else {
- pci_dma_read(&n->parent_obj, addr, buf, size);
+ return;
}
+
+ pci_dma_read(&n->parent_obj, addr, buf, size);
}
static int nvme_check_sqid(NvmeCtrl *n, uint16_t sqid)
--
2.26.2
- [PATCH v4 00/18] nvme: factor out cmb/pmr setup, Klaus Jensen, 2020/04/22
- [PATCH v4 01/18] nvme: fix pci doorbell size calculation, Klaus Jensen, 2020/04/22
- [PATCH v4 03/18] nvme: remove superfluous breaks, Klaus Jensen, 2020/04/22
- [PATCH v4 10/18] nvme: factor out device state setup, Klaus Jensen, 2020/04/22
- [PATCH v4 11/18] nvme: factor out block backend setup, Klaus Jensen, 2020/04/22
- [PATCH v4 06/18] nvme: refactor nvme_addr_read,
Klaus Jensen <=
- [PATCH v4 04/18] nvme: move device parameters to separate struct, Klaus Jensen, 2020/04/22
- [PATCH v4 02/18] nvme: rename trace events to pci_nvme, Klaus Jensen, 2020/04/22
- [PATCH v4 09/18] nvme: factor out property/constraint checks, Klaus Jensen, 2020/04/22
- [PATCH v4 05/18] nvme: use constants in identify, Klaus Jensen, 2020/04/22
- [PATCH v4 07/18] nvme: add max_ioqpairs device parameter, Klaus Jensen, 2020/04/22
- [PATCH v4 08/18] nvme: remove redundant cmbloc/cmbsz members, Klaus Jensen, 2020/04/22
- [PATCH v4 15/18] nvme: factor out cmb setup, Klaus Jensen, 2020/04/22
- [PATCH v4 14/18] nvme: factor out pci setup, Klaus Jensen, 2020/04/22
- [PATCH v4 12/18] nvme: add namespace helpers, Klaus Jensen, 2020/04/22
- [PATCH v4 13/18] nvme: factor out namespace setup, Klaus Jensen, 2020/04/22