Danilo Krummrich
2025-Mar-28 11:01 UTC
[PATCH v2] drm/prime: fix drm_prime_gem_destroy comment
On Wed, Mar 26, 2025 at 01:10:58PM +0000, Chris Bainbridge wrote:> Edit the comments on correct usage of drm_prime_gem_destroy to note > that, if using TTM, drm_prime_gem_destroy must be called in the > ttm_buffer_object.destroy hook, to avoid the dma_buf being freed leaving > a dangling pointer which will be later dereferenced by > ttm_bo_delayed_delete. > > Signed-off-by: Chris Bainbridge <chris.bainbridge at gmail.com> > Suggested-by: Christian K?nig <christian.koenig at amd.com>Can you please send new version of patches as a new mail thread (not in reply to previous versions) please? Otherwise, Reviewed-by: Danilo Krummrich <dakr at kernel.org> @Christian, I assume you will pick this one up?
Christian König
2025-Mar-28 12:46 UTC
[PATCH v2] drm/prime: fix drm_prime_gem_destroy comment
Am 28.03.25 um 12:01 schrieb Danilo Krummrich:> On Wed, Mar 26, 2025 at 01:10:58PM +0000, Chris Bainbridge wrote: >> Edit the comments on correct usage of drm_prime_gem_destroy to note >> that, if using TTM, drm_prime_gem_destroy must be called in the >> ttm_buffer_object.destroy hook, to avoid the dma_buf being freed leaving >> a dangling pointer which will be later dereferenced by >> ttm_bo_delayed_delete. >> >> Signed-off-by: Chris Bainbridge <chris.bainbridge at gmail.com> >> Suggested-by: Christian K?nig <christian.koenig at amd.com> > Can you please send new version of patches as a new mail thread (not in reply to > previous versions) please? > > Otherwise, > > Reviewed-by: Danilo Krummrich <dakr at kernel.org> > > @Christian, I assume you will pick this one up?Sure, I can take care of that. Regards, Christian.