9ae3a8
From 8f25ff7e1496ce8a26edacfa01c64f3a98c564a0 Mon Sep 17 00:00:00 2001
9ae3a8
From: Gerd Hoffmann <kraxel@redhat.com>
9ae3a8
Date: Fri, 22 Jul 2016 09:34:39 +0200
9ae3a8
Subject: [PATCH 2/4] qxl: store memory region and offset instead of pointer
9ae3a8
 for guest slots
9ae3a8
9ae3a8
RH-Author: Gerd Hoffmann <kraxel@redhat.com>
9ae3a8
Message-id: <1469180081-28522-3-git-send-email-kraxel@redhat.com>
9ae3a8
Patchwork-id: 71314
9ae3a8
O-Subject: [RHEL-7.3 qemu-kvm PATCH v2 2/4] qxl: store memory region and offset instead of pointer for guest slots
9ae3a8
Bugzilla: 1355730
9ae3a8
RH-Acked-by: Laurent Vivier <lvivier@redhat.com>
9ae3a8
RH-Acked-by: John Snow <jsnow@redhat.com>
9ae3a8
RH-Acked-by: Laszlo Ersek <lersek@redhat.com>
9ae3a8
9ae3a8
Store MemoryRegion and offset instead of a pointer for each qxl memory
9ae3a8
slot, so we can easily figure in which memory region an qxl object
9ae3a8
stored.
9ae3a8
9ae3a8
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
9ae3a8
Message-id: 1466597244-5938-2-git-send-email-kraxel@redhat.com
9ae3a8
(cherry picked from commit 3cb5158f15604a9f50287f2f06777d5835ff4c15)
9ae3a8
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
9ae3a8
---
9ae3a8
 hw/display/qxl.c | 15 +++++++++++----
9ae3a8
 hw/display/qxl.h |  3 ++-
9ae3a8
 2 files changed, 13 insertions(+), 5 deletions(-)
9ae3a8
9ae3a8
diff --git a/hw/display/qxl.c b/hw/display/qxl.c
9ae3a8
index 5e1ecd8..de5770e 100644
9ae3a8
--- a/hw/display/qxl.c
9ae3a8
+++ b/hw/display/qxl.c
9ae3a8
@@ -1232,6 +1232,7 @@ static int qxl_add_memslot(PCIQXLDevice *d, uint32_t slot_id, uint64_t delta,
9ae3a8
     int pci_region;
9ae3a8
     pcibus_t pci_start;
9ae3a8
     pcibus_t pci_end;
9ae3a8
+    MemoryRegion *mr;
9ae3a8
     intptr_t virt_start;
9ae3a8
     QXLDevMemSlot memslot;
9ae3a8
     int i;
9ae3a8
@@ -1278,11 +1279,11 @@ static int qxl_add_memslot(PCIQXLDevice *d, uint32_t slot_id, uint64_t delta,
9ae3a8
 
9ae3a8
     switch (pci_region) {
9ae3a8
     case QXL_RAM_RANGE_INDEX:
9ae3a8
-        virt_start = (intptr_t)memory_region_get_ram_ptr(&d->vga.vram);
9ae3a8
+        mr = &d->vga.vram;
9ae3a8
         break;
9ae3a8
     case QXL_VRAM_RANGE_INDEX:
9ae3a8
     case 4 /* vram 64bit */:
9ae3a8
-        virt_start = (intptr_t)memory_region_get_ram_ptr(&d->vram_bar);
9ae3a8
+        mr = &d->vram_bar;
9ae3a8
         break;
9ae3a8
     default:
9ae3a8
         /* should not happen */
9ae3a8
@@ -1290,6 +1291,7 @@ static int qxl_add_memslot(PCIQXLDevice *d, uint32_t slot_id, uint64_t delta,
9ae3a8
         return 1;
9ae3a8
     }
9ae3a8
 
9ae3a8
+    virt_start = (intptr_t)memory_region_get_ram_ptr(mr);
9ae3a8
     memslot.slot_id = slot_id;
9ae3a8
     memslot.slot_group_id = MEMSLOT_GROUP_GUEST; /* guest group */
9ae3a8
     memslot.virt_start = virt_start + (guest_start - pci_start);
9ae3a8
@@ -1299,7 +1301,8 @@ static int qxl_add_memslot(PCIQXLDevice *d, uint32_t slot_id, uint64_t delta,
9ae3a8
     qxl_rom_set_dirty(d);
9ae3a8
 
9ae3a8
     qemu_spice_add_memslot(&d->ssd, &memslot, async);
9ae3a8
-    d->guest_slots[slot_id].ptr = (void*)memslot.virt_start;
9ae3a8
+    d->guest_slots[slot_id].mr = mr;
9ae3a8
+    d->guest_slots[slot_id].offset = memslot.virt_start - virt_start;
9ae3a8
     d->guest_slots[slot_id].size = memslot.virt_end - memslot.virt_start;
9ae3a8
     d->guest_slots[slot_id].delta = delta;
9ae3a8
     d->guest_slots[slot_id].active = 1;
9ae3a8
@@ -1366,6 +1369,7 @@ void *qxl_phys2virt(PCIQXLDevice *qxl, QXLPHYSICAL pqxl, int group_id)
9ae3a8
 {
9ae3a8
     uint64_t offset;
9ae3a8
     uint32_t slot;
9ae3a8
+    void *ptr;
9ae3a8
 
9ae3a8
     switch (group_id) {
9ae3a8
     case MEMSLOT_GROUP_HOST:
9ae3a8
@@ -1375,7 +1379,10 @@ void *qxl_phys2virt(PCIQXLDevice *qxl, QXLPHYSICAL pqxl, int group_id)
9ae3a8
         if (!qxl_get_check_slot_offset(qxl, pqxl, &slot, &offset)) {
9ae3a8
             return NULL;
9ae3a8
         }
9ae3a8
-        return qxl->guest_slots[slot].ptr + offset;
9ae3a8
+        ptr = memory_region_get_ram_ptr(qxl->guest_slots[slot].mr);
9ae3a8
+        ptr += qxl->guest_slots[slot].offset;
9ae3a8
+        ptr += offset;
9ae3a8
+        return ptr;
9ae3a8
     }
9ae3a8
     return NULL;
9ae3a8
 }
9ae3a8
diff --git a/hw/display/qxl.h b/hw/display/qxl.h
9ae3a8
index 5247ce9..f3f51e2 100644
9ae3a8
--- a/hw/display/qxl.h
9ae3a8
+++ b/hw/display/qxl.h
9ae3a8
@@ -49,7 +49,8 @@ typedef struct PCIQXLDevice {
9ae3a8
 
9ae3a8
     struct guest_slots {
9ae3a8
         QXLMemSlot     slot;
9ae3a8
-        void           *ptr;
9ae3a8
+        MemoryRegion   *mr;
9ae3a8
+        uint64_t       offset;
9ae3a8
         uint64_t       size;
9ae3a8
         uint64_t       delta;
9ae3a8
         uint32_t       active;
9ae3a8
-- 
9ae3a8
1.8.3.1
9ae3a8