[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v4 1/8] aspeed: introduce a dummy ROM device to catc
From: |
Cédric Le Goater |
Subject: |
[Qemu-devel] [PATCH v4 1/8] aspeed: introduce a dummy ROM device to catch invalid writes |
Date: |
Thu, 19 Oct 2017 12:04:03 +0200 |
Some legacy firmwares access unimplemented addresses on the Aspeed SoC
(old U-Boot code using variables in the bss when it shouldn't do).
Let's add a dummy ROM device to catch the invalid writes and support
new board without using the 'ignore_memory_transaction_failures' flags.
Signed-off-by: Cédric Le Goater <address@hidden>
---
hw/arm/aspeed.c | 24 ++++++++++++++++++++++++
1 file changed, 24 insertions(+)
diff --git a/hw/arm/aspeed.c b/hw/arm/aspeed.c
index ab895ad490af..e44733153819 100644
--- a/hw/arm/aspeed.c
+++ b/hw/arm/aspeed.c
@@ -166,6 +166,19 @@ static void aspeed_board_init_flashes(AspeedSMCState *s,
const char *flashtype,
}
}
+static void boot_rom_rw_flash_write(void *opaque, hwaddr offset, uint64_t
value,
+ unsigned size)
+{
+ qemu_log_mask(LOG_GUEST_ERROR,
+ "%s: 0x%" HWADDR_PRIx " <- 0x%" PRIx64 " [%u]\n",
+ __func__, offset, value, size);
+}
+
+static const MemoryRegionOps boot_rom_rw_flash_ops = {
+ .write = boot_rom_rw_flash_write,
+ .endianness = DEVICE_NATIVE_ENDIAN,
+};
+
static void aspeed_board_init(MachineState *machine,
const AspeedBoardConfig *cfg)
{
@@ -209,6 +222,7 @@ static void aspeed_board_init(MachineState *machine,
if (drive0) {
AspeedSMCFlash *fl = &bmc->soc.fmc.flashes[0];
MemoryRegion *boot_rom = g_new(MemoryRegion, 1);
+ MemoryRegion *boot_rom_rw = g_new(MemoryRegion, 1);
/*
* create a ROM region using the default mapping window size of
@@ -221,6 +235,16 @@ static void aspeed_board_init(MachineState *machine,
memory_region_add_subregion(get_system_memory(), FIRMWARE_ADDR,
boot_rom);
write_boot_rom(drive0, FIRMWARE_ADDR, fl->size, &error_abort);
+
+ /*
+ * Create a fake ROM region to track invalid writes done by
+ * some legacy firmwares
+ */
+ memory_region_init_rom_device(boot_rom_rw, NULL,
&boot_rom_rw_flash_ops,
+ NULL, "aspeed.boot_rom_rw", fl->size,
+ &error_abort);
+ memory_region_add_subregion_overlap(get_system_memory(), FIRMWARE_ADDR,
+ boot_rom_rw, 0);
}
aspeed_board_binfo.kernel_filename = machine->kernel_filename;
--
2.13.6
- [Qemu-devel] [PATCH v4 0/8] aspeed: add a witherspoon-bmc machine, Cédric Le Goater, 2017/10/19
- [Qemu-devel] [PATCH v4 1/8] aspeed: introduce a dummy ROM device to catch invalid writes,
Cédric Le Goater <=
- [Qemu-devel] [PATCH v4 2/8] aspeed: remove ignore_memory_transaction_failures on all boards, Cédric Le Goater, 2017/10/19
- [Qemu-devel] [PATCH v4 3/8] aspeed: add support for the witherspoon-bmc board, Cédric Le Goater, 2017/10/19
- [Qemu-devel] [PATCH v4 4/8] aspeed: add an I2C RTC device to all machines, Cédric Le Goater, 2017/10/19
- [Qemu-devel] [PATCH v4 5/8] smbus: add a smbus_eeprom_init_one() routine, Cédric Le Goater, 2017/10/19
- [Qemu-devel] [PATCH v4 6/8] aspeed: Add EEPROM I2C devices, Cédric Le Goater, 2017/10/19
- [Qemu-devel] [PATCH v4 7/8] misc: add pca9552 LED blinker model, Cédric Le Goater, 2017/10/19
- [Qemu-devel] [PATCH v4 8/8] aspeed: add the pc9552 chips to the witherspoon machine, Cédric Le Goater, 2017/10/19