Hi Geert, On Thu, Jan 26, 2017 at 6:53 PM, Geert Uytterhoeven <geert+renesas@xxxxxxxxx> wrote: > Currently, the IPMMU/VMSA driver supports 32-bit I/O Virtual Addresses > only, and thus sets io_pgtable_cfg.ias = 32. However, it doesn't force > a 32-bit IOVA space through the IOMMU Domain Geometry. > > Hence if a device (e.g. SYS-DMAC) rightfully configures a 40-bit DMA > mask, it will still be handed out a 40-bit IOVA, outside the 32-bit IOVA > space, leading to out-of-bounds accesses of the PGD when mapping the > IOVA. > > Force a 32-bit IOMMU Domain Geometry to fix this. Nice, thanks! > Signed-off-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> > --- > Should the generic code restrict the geometry based on IAS instead? Might make sense. Since this is a software policy limited by hardware it might be good to use as small IOVA space as possible to improve performance. So selecting IOVA space based on slave device or domain policy or something similar might be a good thing to do. Not sure how to tie that into the IOMMU subsystem though... Thanks! / magnus
