2013-06-29 00:11:02 +00:00
|
|
|
From 9642978bbeba188570ec62fd989cad4da32589c3 Mon Sep 17 00:00:00 2001
|
2013-02-20 01:14:18 +00:00
|
|
|
From: Cole Robinson <crobinso@redhat.com>
|
|
|
|
Date: Tue, 19 Feb 2013 16:19:02 -0500
|
|
|
|
Subject: [PATCH] qxl: Add rom_size compat property, fix migration from 1.2
|
|
|
|
|
|
|
|
Commit 038c1879a00153b14bce113315b693e8c2944fa9 changed the qxl rom
|
|
|
|
size to 8192, which fixes incoming migration from qemu 1.0. However
|
|
|
|
from qemu 1.2 and 1.3 had rom size 16384, so incoming migration
|
|
|
|
from those versions is now broken.
|
|
|
|
|
|
|
|
Add a rom_size compat property. 1.2 and 1.3 get 16384, everything
|
|
|
|
else is 8192.
|
|
|
|
|
|
|
|
This isn't actually fool proof, since rom_size can be dependent on
|
|
|
|
the version of spice qemu is built against:
|
|
|
|
|
|
|
|
https://lists.gnu.org/archive/html/qemu-devel/2013-02/msg03154.html
|
|
|
|
|
|
|
|
However these sizes match what native Fedora packages get, so it's
|
|
|
|
good enough for now.
|
|
|
|
---
|
2013-05-21 16:12:56 +00:00
|
|
|
hw/display/qxl.c | 9 ++++-----
|
|
|
|
hw/i386/pc_piix.c | 16 ++++++++++++++++
|
2013-02-20 01:14:18 +00:00
|
|
|
2 files changed, 20 insertions(+), 5 deletions(-)
|
|
|
|
|
2013-05-21 16:12:56 +00:00
|
|
|
diff --git a/hw/display/qxl.c b/hw/display/qxl.c
|
|
|
|
index c475cb1..74f07c0 100644
|
|
|
|
--- a/hw/display/qxl.c
|
|
|
|
+++ b/hw/display/qxl.c
|
|
|
|
@@ -306,16 +306,14 @@ static inline uint32_t msb_mask(uint32_t val)
|
2013-02-20 01:14:18 +00:00
|
|
|
return mask;
|
|
|
|
}
|
|
|
|
|
|
|
|
-static ram_addr_t qxl_rom_size(void)
|
|
|
|
+static void check_qxl_rom_size(PCIQXLDevice *d)
|
|
|
|
{
|
|
|
|
uint32_t required_rom_size = sizeof(QXLRom) + sizeof(QXLModes) +
|
|
|
|
sizeof(qxl_modes);
|
|
|
|
- uint32_t rom_size = 8192; /* two pages */
|
|
|
|
|
|
|
|
required_rom_size = MAX(required_rom_size, TARGET_PAGE_SIZE);
|
|
|
|
required_rom_size = msb_mask(required_rom_size * 2 - 1);
|
|
|
|
- assert(required_rom_size <= rom_size);
|
|
|
|
- return rom_size;
|
|
|
|
+ assert(required_rom_size <= d->rom_size);
|
|
|
|
}
|
|
|
|
|
|
|
|
static void init_qxl_rom(PCIQXLDevice *d)
|
|
|
|
@@ -1979,7 +1977,7 @@ static int qxl_init_common(PCIQXLDevice *qxl)
|
|
|
|
pci_set_byte(&config[PCI_REVISION_ID], pci_device_rev);
|
|
|
|
pci_set_byte(&config[PCI_INTERRUPT_PIN], 1);
|
|
|
|
|
|
|
|
- qxl->rom_size = qxl_rom_size();
|
|
|
|
+ check_qxl_rom_size(qxl);
|
|
|
|
memory_region_init_ram(&qxl->rom_bar, "qxl.vrom", qxl->rom_size);
|
|
|
|
vmstate_register_ram(&qxl->rom_bar, &qxl->pci.qdev);
|
|
|
|
init_qxl_rom(qxl);
|
2013-05-21 16:12:56 +00:00
|
|
|
@@ -2302,6 +2300,7 @@ static Property qxl_properties[] = {
|
2013-02-20 01:14:18 +00:00
|
|
|
DEFINE_PROP_UINT32("vram64_size_mb", PCIQXLDevice, vram_size_mb, -1),
|
|
|
|
DEFINE_PROP_UINT32("vgamem_mb", PCIQXLDevice, vgamem_size_mb, 16),
|
|
|
|
DEFINE_PROP_INT32("surfaces", PCIQXLDevice, ssd.num_surfaces, 1024),
|
|
|
|
+ DEFINE_PROP_UINT32("rom_size", PCIQXLDevice, rom_size, 8192),
|
|
|
|
DEFINE_PROP_END_OF_LIST(),
|
|
|
|
};
|
|
|
|
|
2013-05-21 16:12:56 +00:00
|
|
|
diff --git a/hw/i386/pc_piix.c b/hw/i386/pc_piix.c
|
2013-06-29 00:11:02 +00:00
|
|
|
index 0787f38..e26c93d 100644
|
2013-05-21 16:12:56 +00:00
|
|
|
--- a/hw/i386/pc_piix.c
|
|
|
|
+++ b/hw/i386/pc_piix.c
|
|
|
|
@@ -369,6 +369,14 @@ static QEMUMachine pc_i440fx_machine_v1_4 = {
|
|
|
|
.driver = "e1000",\
|
|
|
|
.property = "autonegotiation",\
|
|
|
|
.value = "off",\
|
|
|
|
+ },{ \
|
|
|
|
+ .driver = "qxl", \
|
|
|
|
+ .property = "rom_size", \
|
|
|
|
+ .value = stringify(16384), \
|
|
|
|
+ },{\
|
|
|
|
+ .driver = "qxl-vga", \
|
|
|
|
+ .property = "rom_size", \
|
|
|
|
+ .value = stringify(16384), \
|
|
|
|
}
|
|
|
|
|
|
|
|
static QEMUMachine pc_machine_v1_3 = {
|
|
|
|
@@ -465,6 +473,14 @@ static QEMUMachine pc_machine_v1_2 = {
|
|
|
|
.driver = "virtio-blk-pci",\
|
|
|
|
.property = "config-wce",\
|
|
|
|
.value = "off",\
|
|
|
|
+ },{ \
|
|
|
|
+ .driver = "qxl", \
|
|
|
|
+ .property = "rom_size", \
|
|
|
|
+ .value = stringify(8192), \
|
|
|
|
+ },{\
|
|
|
|
+ .driver = "qxl-vga", \
|
|
|
|
+ .property = "rom_size", \
|
|
|
|
+ .value = stringify(8192), \
|
|
|
|
}
|
|
|
|
|
|
|
|
static QEMUMachine pc_machine_v1_1 = {
|