Searched hist:"6 da24341866fa940fd7d575788a2319514941c77" (Results 1 – 1 of 1) sorted by relevance
/openbmc/qemu/hw/i386/ |
H A D | intel_iommu.c | diff b1ab8f9cc554e3c92200d44ce9d65bab50be183e Tue Apr 04 15:37:31 CDT 2023 Peter Maydell <peter.maydell@linaro.org> Revert "memory: Optimize replay of guest mapping"
This reverts commit 6da24341866fa940fd7d575788a2319514941c77 ("memory: Optimize replay of guest mapping").
This change breaks the mps3-an547 board under TCG (and probably other TCG boards using an IOMMU), which now assert:
$ ./build/x86/qemu-system-arm --machine mps3-an547 -serial stdio -kernel /tmp/an547-mwe/build/test.elf qemu-system-arm: ../../softmmu/memory.c:1903: memory_region_register_iommu_notifier: Assertion `n->end <= memory_region_size(mr)' failed.
This is because tcg_register_iommu_notifier() registers an IOMMU notifier which covers the entire address space, so the assertion added in this commit is not correct.
For the 8.0 release, just revert this commit as it is only an optimization.
Fixes: 6da24341866f ("memory: Optimize replay of guest mapping") Signed-off-by: Peter Maydell <peter.maydell@linaro.org> Signed-off-by: Michael S. Tsirkin <mst@redhat.com> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Message-id: 917c1c552b2d1b732f9a86c6a90684c3a5e4cada.1680640587.git.mst@redhat.com diff 6da24341866fa940fd7d575788a2319514941c77 Wed Feb 15 00:52:38 CST 2023 Zhenzhong Duan <zhenzhong.duan@intel.com> memory: Optimize replay of guest mapping
On x86, there are two notifiers registered due to vtd-ir memory region splitting the whole address space. During replay of the address space for each notifier, the whole address space is scanned which is unnecessory.
We only need to scan the space belong to notifier montiored space.
Assert when notifier is used to monitor beyond iommu memory region's address space.
Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com> Message-Id: <20230215065238.713041-1-zhenzhong.duan@intel.com> Acked-by: Peter Xu <peterx@redhat.com> Acked-by: Jason Wang <jasowang@redhat.com> Reviewed-by: Michael S. Tsirkin <mst@redhat.com> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
|