On Mon, Apr 09, 2018 at 11:07:19PM +0300, Dmitry Osipenko wrote: > Page mapping could overwritten by an accident (a bug). We can catch this > case by checking 'VALID' bit of GART's page entry prior to mapping of a > page. Since that check introduces a small performance impact, it should be > enabled explicitly using new GART's kernel module 'debug' parameter. > > Signed-off-by: Dmitry Osipenko <digetx@xxxxxxxxx> > --- > drivers/iommu/tegra-gart.c | 13 +++++++++++++ > 1 file changed, 13 insertions(+) I think this was discussed before, but this really shouldn't be needed unless the IOVA allocator or GART driver are buggy. That said, I think this has some usefulness to find such bugs, so: Acked-by: Thierry Reding <treding@xxxxxxxxxx>
Attachment:
signature.asc
Description: PGP signature
