|
 |
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 |
|