Quoting Daniel Vetter (2019-06-14 21:36:04) > It looks like this was done purely to get a consistent place to look > up the reservation object pointer. With the drm_prime.c helper code > now also setting gem_object->resv for imported objects we can just use > that pointer directly, instead of first ensuring a dma-buf exists. Oh. commit 1ba627148ef5 ("drm: Add reservation_object to drm_gem_object") embedded a reservation_object into the struct. I was wondering what on earth I was doing if the code should have been so simple. References: 1ba627148ef5 ("drm: Add reservation_object to drm_gem_object") > Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxxx> Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> Quick leave before I start ranting about the horrors of reservation_object. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx