[PATCH 1/2] drm/ttm: set ttm_buffer_object pointer as null after it's freed

Christian König christian.koenig at amd.com
Mon Sep 10 09:00:04 UTC 2018


Hi Ray,

well those patches doesn't make sense, the pointer is only local to the 
function.

Regards,
Christian.

Am 10.09.2018 um 10:57 schrieb Huang Rui:
> It avoids to be refered again after freed.
>
> Signed-off-by: Huang Rui <ray.huang at amd.com>
> Cc: Christian König <christian.koenig at amd.com>
> Cc: Tom StDenis <Tom.StDenis at amd.com>
> ---
>   drivers/gpu/drm/ttm/ttm_bo.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/drivers/gpu/drm/ttm/ttm_bo.c b/drivers/gpu/drm/ttm/ttm_bo.c
> index 138c989..d3ef5f8 100644
> --- a/drivers/gpu/drm/ttm/ttm_bo.c
> +++ b/drivers/gpu/drm/ttm/ttm_bo.c
> @@ -54,6 +54,7 @@ static struct attribute ttm_bo_count = {
>   static void ttm_bo_default_destroy(struct ttm_buffer_object *bo)
>   {
>   	kfree(bo);
> +	bo = NULL;
>   }
>   
>   static inline int ttm_mem_type_from_place(const struct ttm_place *place,



More information about the amd-gfx mailing list