[PATCH v7 3/3] Revert "drm/i915: Wait for active retire before i915_active_fini()"

Nirmoy Das nirmoy.das at linux.intel.com
Tue Mar 5 14:59:02 UTC 2024


On 3/5/2024 3:35 PM, Janusz Krzysztofik wrote:
> This reverts commit 7a2280e8dcd2f1f436db9631287c0b21cf6a92b0, obsoleted
> by "drm/i915/vma: Fix UAF on destroy against retire race".
>
> Signed-off-by: Janusz Krzysztofik<janusz.krzysztofik at linux.intel.com>
> Cc: Nirmoy Das<nirmoy.das at intel.com>

Reviewed-by: Nirmoy Das <nirmoy.das at intel.com>

> ---
>   drivers/gpu/drm/i915/i915_vma.c | 2 --
>   1 file changed, 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_vma.c b/drivers/gpu/drm/i915/i915_vma.c
> index b70715b1411d6..d2f064d2525cc 100644
> --- a/drivers/gpu/drm/i915/i915_vma.c
> +++ b/drivers/gpu/drm/i915/i915_vma.c
> @@ -1776,8 +1776,6 @@ static void release_references(struct i915_vma *vma, struct intel_gt *gt,
>   	if (vm_ddestroy)
>   		i915_vm_resv_put(vma->vm);
>   
> -	/* Wait for async active retire */
> -	i915_active_wait(&vma->active);
>   	i915_active_fini(&vma->active);
>   	GEM_WARN_ON(vma->resource);
>   	i915_vma_free(vma);
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20240305/d5a09dfc/attachment-0001.htm>


More information about the dri-devel mailing list