On Thu Aug 14, 2025 at 5:01 PM CEST, Alice Ryhl wrote: > I had wanted to move it below free(), but we can't do that since > free() also performs the kfree() call.
Actually, I think it belongs in drm_gem_private_object_fini(), where also call dma_resv_fini() for the GEM owned dma-resv lock. Any cleanup should be done from where a driver calls drm_gem_object_release().