[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 00/11] memory: Forbid mapping AddressSpace root MemoryRegi
From: |
Cédric Le Goater |
Subject: |
Re: [PATCH v2 00/11] memory: Forbid mapping AddressSpace root MemoryRegion |
Date: |
Mon, 19 Apr 2021 09:17:13 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.8.1 |
Hello,
On 4/17/21 12:30 PM, Philippe Mathieu-Daudé wrote:
> 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.
So I will wait for this patchset to be merged before sending the
aspeed queue. Are there any blocking aspects to it ?
Thanks,
C.
- Re: [PATCH v2 04/11] hw/pci-host: Rename Raven ASIC PCI bridge as raven.c, (continued)
- [PATCH v2 05/11] hw/pci-host/raven: Add PCI_IO_BASE_ADDR definition, Philippe Mathieu-Daudé, 2021/04/17
- [PATCH v2 06/11] hw/pci-host/raven: Assert PCI I/O AddressSpace is based at 0x80000000, Philippe Mathieu-Daudé, 2021/04/17
- [PATCH v2 07/11] hw/pci-host/raven: Use MR alias for AS root, not sysbus mapped MR, Philippe Mathieu-Daudé, 2021/04/17
- [RFC PATCH v2 08/11] hw/pci-host/raven: Remove pointless alias mapping onto system bus, Philippe Mathieu-Daudé, 2021/04/17
- [PATCH v2 09/11] hw/pci-host/prep: Do not directly map bus-master region onto main bus, Philippe Mathieu-Daudé, 2021/04/17
- [PATCH v2 10/11] memory: Make sure root MR won't be added as subregion, Philippe Mathieu-Daudé, 2021/04/17
- [PATCH v2 11/11] hw/pci-host/raven: Remove temporary assertion 'root MR is zero-based', Philippe Mathieu-Daudé, 2021/04/17
- Re: [PATCH v2 00/11] memory: Forbid mapping AddressSpace root MemoryRegion,
Cédric Le Goater <=
- Re: [PATCH v2 00/11] memory: Forbid mapping AddressSpace root MemoryRegion, Peter Xu, 2021/04/20