qemu-block
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [RFC PATCH 6/6] hw/block/nvme: Make device target agnostic


From: Philippe Mathieu-Daudé
Subject: Re: [RFC PATCH 6/6] hw/block/nvme: Make device target agnostic
Date: Mon, 4 May 2020 17:36:22 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.5.0

+Keith new email

On 5/4/20 11:46 AM, Philippe Mathieu-Daudé wrote:
The NVMe device should not use target specific API.
Use memory_region_do_writeback() (which was introduced
in commit 61c490e25e0, after the NVMe emulated device
was added) to replace qemu_ram_writeback().

Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
---
RFC because I have no clue how dirty_log_mask works.

Cc: Paolo Bonzini <address@hidden>
Cc: Beata Michalska <address@hidden>
---
  hw/block/nvme.c        | 4 +---
  hw/block/Makefile.objs | 2 +-
  2 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/hw/block/nvme.c b/hw/block/nvme.c
index 9b453423cf..9b0ac0ea2a 100644
--- a/hw/block/nvme.c
+++ b/hw/block/nvme.c
@@ -46,7 +46,6 @@
  #include "qapi/visitor.h"
  #include "sysemu/hostmem.h"
  #include "sysemu/block-backend.h"
-#include "exec/ram_addr.h"
#include "qemu/log.h"
  #include "qemu/module.h"
@@ -1207,8 +1206,7 @@ static uint64_t nvme_mmio_read(void *opaque, hwaddr addr, 
unsigned size)
           */
          if (addr == 0xE08 &&
              (NVME_PMRCAP_PMRWBM(n->bar.pmrcap) & 0x02)) {
-            qemu_ram_writeback(n->pmrdev->mr.ram_block,
-                               0, n->pmrdev->size);
+            memory_region_do_writeback(&n->pmrdev->mr, 0, n->pmrdev->size);
          }
          memcpy(&val, ptr + addr, size);
      } else {
diff --git a/hw/block/Makefile.objs b/hw/block/Makefile.objs
index 47960b5f0d..8855c22656 100644
--- a/hw/block/Makefile.objs
+++ b/hw/block/Makefile.objs
@@ -13,6 +13,6 @@ common-obj-$(CONFIG_SH4) += tc58128.o
obj-$(CONFIG_VIRTIO_BLK) += virtio-blk.o
  obj-$(CONFIG_VHOST_USER_BLK) += vhost-user-blk.o
-obj-$(CONFIG_NVME_PCI) += nvme.o
+common-obj-$(CONFIG_NVME_PCI) += nvme.o
obj-y += dataplane/





reply via email to

[Prev in Thread] Current Thread [Next in Thread]