qemu-ppc
[Top][All Lists]
Advanced

[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




reply via email to

[Prev in Thread] Current Thread [Next in Thread]