Current 'iotlb_to_section()' API only finds the sections from CPU's AddressSpace but not other AddressSpaces. However, If address space translation is over IOMMUMemoryRegion, like ARM MPC device, it could return section from other AddressSpaces, which will break 'iotlb_to_section()'.
To fix this bug, store complete MemoryRegionSection pointer in CPUTLBEntryFull, and we can directly use section pointer in it w/o need of 'iotlb_to_section()' API. Besides ARM MPC device, the upcoming RISC-V wgChecker [1] and RISC-V IOPMP [2] devices also require this bugfix. [1] RISC-V WG: https://patchew.org/QEMU/[email protected]/ [2] RISC-V IOPMP: https://patchew.org/QEMU/[email protected]/ Changed since v1: - Move io_prepare() argument change in a preliminary patch - Rename 'xlat' to 'xlat_offset' and fix the comment Jim Shu (3): accel/tcg: Send full CPUTLBEntryFull struct into io_prepare() accel/tcg: Fix iotlb_to_section() for different AddressSpace system/physmem: Remove the assertion of page-aligned section number accel/tcg/cputlb.c | 32 +++++++++++++++----------------- include/accel/tcg/iommu.h | 15 --------------- include/exec/cputlb.h | 4 ++-- include/hw/core/cpu.h | 17 +++++++++-------- system/physmem.c | 31 ------------------------------- 5 files changed, 26 insertions(+), 73 deletions(-) -- 2.43.0
