[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v2 00/11] memory: Forbid mapping AddressSpace root MemoryRegion
From: |
Philippe Mathieu-Daudé |
Subject: |
[PATCH v2 00/11] memory: Forbid mapping AddressSpace root MemoryRegion |
Date: |
Sat, 17 Apr 2021 12:30:17 +0200 |
Hi,
This series is the result of a long thread with Peter:
https://www.mail-archive.com/qemu-devel@nongnu.org/msg788366.html
and IRC chats...
AddressSpace are physical address view and shouldn't be using
non-zero base address. The correct way to map a MR used as AS
root is to use a MR alias.
Fix the current incorrect uses, then forbid further use.
Since v1:
- Split the Raven patch in multiple changes, easier to follow/review
(https://www.mail-archive.com/qemu-devel@nongnu.org/msg791116.html)
Note, the Aspeed patches are already queued in Cédric tree. I had
to cherry-pick them from his tree to have the series pass CI.
Cédric Le Goater (1):
hw/aspeed/smc: Use the RAM memory region for DMAs
Peter Xu (1):
memory: Make sure root MR won't be added as subregion
Philippe Mathieu-Daudé (9):
hw/arm/aspeed: Do not directly map ram container onto main address bus
hw/arm/aspeed: Do not sysbus-map mmio flash region directly, use alias
hw/pci-host: Rename Raven ASIC PCI bridge as raven.c
hw/pci-host/raven: Add PCI_IO_BASE_ADDR definition
hw/pci-host/raven: Assert PCI I/O AddressSpace is based at 0x80000000
hw/pci-host/raven: Use MR alias for AS root, not sysbus mapped MR
hw/pci-host/raven: Remove pointless alias mapping onto system bus
hw/pci-host/prep: Do not directly map bus-master region onto main bus
hw/pci-host/raven: Remove temporary assertion 'root MR is zero-based'
include/exec/memory.h | 1 +
include/hw/ssi/aspeed_smc.h | 1 +
hw/arm/aspeed.c | 8 ++++++--
hw/pci-host/{prep.c => raven.c} | 19 ++++++++++---------
hw/ssi/aspeed_smc.c | 10 +++++-----
softmmu/memory.c | 2 ++
MAINTAINERS | 2 +-
hw/pci-host/Kconfig | 2 +-
hw/pci-host/meson.build | 2 +-
hw/ppc/Kconfig | 2 +-
10 files changed, 29 insertions(+), 20 deletions(-)
rename hw/pci-host/{prep.c => raven.c} (96%)
--
2.26.3
- [PATCH v2 00/11] memory: Forbid mapping AddressSpace root MemoryRegion,
Philippe Mathieu-Daudé <=
- [PATCH v2 02/11] hw/aspeed/smc: Use the RAM memory region for DMAs, Philippe Mathieu-Daudé, 2021/04/17
- [PATCH v2 03/11] hw/arm/aspeed: Do not sysbus-map mmio flash region directly, use alias, Philippe Mathieu-Daudé, 2021/04/17
- [PATCH v2 04/11] hw/pci-host: Rename Raven ASIC PCI bridge as raven.c, Philippe Mathieu-Daudé, 2021/04/17
- [PATCH v2 05/11] hw/pci-host/raven: Add PCI_IO_BASE_ADDR definition, Philippe Mathieu-Daudé, 2021/04/17