On 18.08.2021 19:44, Julien Grall wrote: > On 18/08/2021 08:52, Jan Beulich wrote: >> Ranges checked by iomem_access_permitted() are inclusive; to permit a >> mapping there's no need for access to also have been granted for the >> subsequent page. > > Good catch! OOI, how did you find it?
In the course of my large-IOMMU-mappings work I ended up grep-ing for all uses of the function, and this one - while unrelated to that work - caught my eye. >> Fixes: 80f9c3167084 ("xen/arm: acpi: Map MMIO on fault in stage-2 page table >> for the hardware domain") >> Signed-off-by: Jan Beulich <jbeul...@suse.com> > > Reviewed-by: Julien Grall <jgr...@amazon.com> Thanks. Jan