On Thu, 27 May 2021 at 02:37, Daniel Vetter <[email protected]> wrote: > > Media userspace was the last userspace to still use them, and they > converted now too: > > https://github.com/intel/media-driver/commit/144020c37770083974bedf59902b70b8f444c799 > > This means no reason anymore to make relocations faster than they've > been for the first 9 years of gem. This code was added in > > commit 7dd4f6729f9243bd7046c6f04c107a456bda38eb > Author: Chris Wilson <[email protected]> > Date: Fri Jun 16 15:05:24 2017 +0100 > > drm/i915: Async GPU relocation processing > > Furthermore there's pretty strong indications it's buggy, since the > code to use it by default as the only option had to be reverted: > > commit ad5d95e4d538737ed3fa25493777decf264a3011 > Author: Dave Airlie <[email protected]> > Date: Tue Sep 8 15:41:17 2020 +1000 > > Revert "drm/i915/gem: Async GPU relocations only" > > This code just disables gpu relocations, leaving the garbage > collection for later patches and more importantly, much less confusing > diff. Also given how much headaches this code has caused in the past, > letting this soak for a bit seems justified. > > Cc: Jon Bloomfield <[email protected]> > Signed-off-by: Daniel Vetter <[email protected]> > Cc: Chris Wilson <[email protected]> > Cc: Maarten Lankhorst <[email protected]> > Cc: Joonas Lahtinen <[email protected]> > Cc: Daniel Vetter <[email protected]> > Cc: "Thomas Hellström" <[email protected]> > Cc: Matthew Auld <[email protected]> > Cc: Lionel Landwerlin <[email protected]> > Cc: Dave Airlie <[email protected]>
Acked-by: Dave Airlie <[email protected]> Thanks for making this happen, hope the softpin world is a happier future. Dave.
