Hi Lucas, On Wed, Apr 17, 2019 at 03:50:15PM +0200, Lucas Stach wrote: > > Hi all, > > v1 cover letter: > > the following patches finally implement one of the longstanding TODO > items in the etnaviv driver: per-process address spaces. They are only > enabled for MMUv2, as switching the MMU context on MMUv1 would require > a full stop of the FE, which I deemed too expensive to do on potentially > every submitted commandstream. > > For now this only provides better isolation between GPU clients, but it > is also a big step in the direction of supporting softpin. Softpin will > later be used by GC7000 userspace drivers to deal with texture descriptors > without the need to add even more relocation interfaces to the etnaviv > UAPI. > > The changes are big and pretty disruptive, so I acknowledge that they > aren't prime targets for a quick review, but I would appreciate a > second pairs of eyes on them. > > Changes since v1: > - fixed an issue where a debugsfs read could run into a kernel NULL > ptr dereference due to no current MMU context > - fixed an issue where the current MMU context could be destroyed > due to the DRM client going away, while it is still in use by > an active GPU job > - more extensive testing on GC880, GC2000, GC3000 and GC7000 I gave this series a test on GC7000 and it looks good. I'll do some more testing over the next week. Cheers, -- Guido _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
