yeahuh / rpms / qemu-kvm

Forked from rpms/qemu-kvm 2 years ago
Clone
9ae3a8
From 0ae4c882404b4590c34bb9b03a86f9389413fd1c Mon Sep 17 00:00:00 2001
9ae3a8
From: Alex Williamson <alex.williamson@redhat.com>
9ae3a8
Date: Wed, 7 Sep 2016 14:09:52 +0200
9ae3a8
Subject: [PATCH 2/2] Add skip_dump flag to ignore memory region during dump
9ae3a8
9ae3a8
RH-Author: Alex Williamson <alex.williamson@redhat.com>
9ae3a8
Message-id: <20160907140817.21968.47551.stgit@gimli.home>
9ae3a8
Patchwork-id: 72264
9ae3a8
O-Subject: [RHEL7.3 qemu-kvm PATCH v2] Add skip_dump flag to ignore memory region during dump
9ae3a8
Bugzilla: 1373088
9ae3a8
RH-Acked-by: Paolo Bonzini <pbonzini@redhat.com>
9ae3a8
RH-Acked-by: Auger Eric <eric.auger@redhat.com>
9ae3a8
RH-Acked-by: Laszlo Ersek <lersek@redhat.com>
9ae3a8
9ae3a8
From: Nikunj A Dadhania <nikunj@linux.vnet.ibm.com>
9ae3a8
9ae3a8
Bugzilla: 1373088
9ae3a8
Brew: https://brewweb.engineering.redhat.com/brew/taskinfo?taskID=11713365
9ae3a8
Upstream: e4dc3f5909ab90520bc1a27b381c3017ff65ed68
9ae3a8
9ae3a8
The PCI MMIO might be disabled or the device in the reset state.
9ae3a8
Make sure we do not dump these memory regions.
9ae3a8
9ae3a8
Signed-off-by: Nikunj A Dadhania <nikunj@linux.vnet.ibm.com>
9ae3a8
Acked-by: Alex Williamson <alex.williamson@redhat.com>
9ae3a8
CC: Paolo Bonzini <pbonzini@redhat.com>
9ae3a8
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
9ae3a8
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
9ae3a8
---
9ae3a8
 hw/misc/vfio.c        |  1 +
9ae3a8
 include/exec/memory.h | 19 +++++++++++++++++++
9ae3a8
 memory.c              | 10 ++++++++++
9ae3a8
 memory_mapping.c      |  3 ++-
9ae3a8
 4 files changed, 32 insertions(+), 1 deletion(-)
9ae3a8
9ae3a8
diff --git a/hw/misc/vfio.c b/hw/misc/vfio.c
9ae3a8
index 36b9832..4fdc09a 100644
9ae3a8
--- a/hw/misc/vfio.c
9ae3a8
+++ b/hw/misc/vfio.c
9ae3a8
@@ -2596,6 +2596,7 @@ static int vfio_mmap_bar(VFIOBAR *bar, MemoryRegion *mem, MemoryRegion *submem,
9ae3a8
         }
9ae3a8
 
9ae3a8
         memory_region_init_ram_ptr(submem, name, size, *map);
9ae3a8
+        memory_region_set_skip_dump(submem);
9ae3a8
     } else {
9ae3a8
 empty_region:
9ae3a8
         /* Create a zero sized sub-region to make cleanup easy. */
9ae3a8
diff --git a/include/exec/memory.h b/include/exec/memory.h
9ae3a8
index 3bbe378..448d501 100644
9ae3a8
--- a/include/exec/memory.h
9ae3a8
+++ b/include/exec/memory.h
9ae3a8
@@ -126,6 +126,7 @@ struct MemoryRegion {
9ae3a8
     bool terminates;
9ae3a8
     bool romd_mode;
9ae3a8
     bool ram;
9ae3a8
+    bool skip_dump;
9ae3a8
     bool readonly; /* For RAM regions */
9ae3a8
     bool enabled;
9ae3a8
     bool rom_device;
9ae3a8
@@ -353,6 +354,24 @@ uint64_t memory_region_size(MemoryRegion *mr);
9ae3a8
 bool memory_region_is_ram(MemoryRegion *mr);
9ae3a8
 
9ae3a8
 /**
9ae3a8
+ * memory_region_is_skip_dump: check whether a memory region should not be
9ae3a8
+ *                             dumped
9ae3a8
+ *
9ae3a8
+ * Returns %true is a memory region should not be dumped(e.g. VFIO BAR MMAP).
9ae3a8
+ *
9ae3a8
+ * @mr: the memory region being queried
9ae3a8
+ */
9ae3a8
+bool memory_region_is_skip_dump(MemoryRegion *mr);
9ae3a8
+
9ae3a8
+/**
9ae3a8
+ * memory_region_set_skip_dump: Set skip_dump flag, dump will ignore this memory
9ae3a8
+ *                              region
9ae3a8
+ *
9ae3a8
+ * @mr: the memory region being queried
9ae3a8
+ */
9ae3a8
+void memory_region_set_skip_dump(MemoryRegion *mr);
9ae3a8
+
9ae3a8
+/**
9ae3a8
  * memory_region_is_romd: check whether a memory region is in ROMD mode
9ae3a8
  *
9ae3a8
  * Returns %true if a memory region is a ROM device and currently set to allow
9ae3a8
diff --git a/memory.c b/memory.c
9ae3a8
index a71d096..7bd6e87 100644
9ae3a8
--- a/memory.c
9ae3a8
+++ b/memory.c
9ae3a8
@@ -957,6 +957,11 @@ void memory_region_init_ram_ptr(MemoryRegion *mr,
9ae3a8
     mr->ram_addr = qemu_ram_alloc_from_ptr(size, ptr, mr);
9ae3a8
 }
9ae3a8
 
9ae3a8
+void memory_region_set_skip_dump(MemoryRegion *mr)
9ae3a8
+{
9ae3a8
+    mr->skip_dump = true;
9ae3a8
+}
9ae3a8
+
9ae3a8
 void memory_region_init_alias(MemoryRegion *mr,
9ae3a8
                               const char *name,
9ae3a8
                               MemoryRegion *orig,
9ae3a8
@@ -1047,6 +1052,11 @@ bool memory_region_is_ram(MemoryRegion *mr)
9ae3a8
     return mr->ram;
9ae3a8
 }
9ae3a8
 
9ae3a8
+bool memory_region_is_skip_dump(MemoryRegion *mr)
9ae3a8
+{
9ae3a8
+    return mr->skip_dump;
9ae3a8
+}
9ae3a8
+
9ae3a8
 bool memory_region_is_logging(MemoryRegion *mr)
9ae3a8
 {
9ae3a8
     return mr->dirty_log_mask;
9ae3a8
diff --git a/memory_mapping.c b/memory_mapping.c
9ae3a8
index 65082d8..a4d59b7 100644
9ae3a8
--- a/memory_mapping.c
9ae3a8
+++ b/memory_mapping.c
9ae3a8
@@ -203,7 +203,8 @@ static void guest_phys_blocks_region_add(MemoryListener *listener,
9ae3a8
     GuestPhysBlock *predecessor;
9ae3a8
 
9ae3a8
     /* we only care about RAM */
9ae3a8
-    if (!memory_region_is_ram(section->mr)) {
9ae3a8
+    if (!memory_region_is_ram(section->mr) ||
9ae3a8
+        memory_region_is_skip_dump(section->mr)) {
9ae3a8
         return;
9ae3a8
     }
9ae3a8
 
9ae3a8
-- 
9ae3a8
1.8.3.1
9ae3a8