Quoting Matthew Auld (2018-05-31 15:43:14) > On 31 May 2018 at 12:35, Chris Wilson <[email protected]> wrote: > > From: Jon Bloomfield <[email protected]> > > > > We can set a bit inside the ppGTT PTE to indicate a page is read-only; > > writes from the GPU will be discarded. We can use this to protect pages > > and in particular support read-only userptr mappings (necessary for > > importing PROT_READ vma). > > > > Signed-off-by: Jon Bloomfield <[email protected]> > > Signed-off-by: Chris Wilson <[email protected]> > > Cc: Joonas Lahtinen <[email protected]> > > Cc: Matthew Auld <[email protected]> > > Reviewed-by: Joonas Lahtinen <[email protected]> > > fwiw I didn't see any surprises with ro 2M/64K PTEs when testing locally. > > For the series: > Reviewed-by: Matthew Auld <[email protected]>
I think I'd like to see a selftest also exercising the ro nature before pushing. Thanks for the review, stick around there'll be another patch to come :) -Chris _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
