mirror of
https://github.com/torvalds/linux.git
synced 2026-09-26 10:02:02 +02:00
d2da6696e0
i915_gem_busy_ioctl uses dma_resv_for_each_fence_unlocked() to iterate
over the fences in an GEM object without holding a reference but only
the RCU read side lock.
What can happen here is that the GEM object is destroyed concurrently
while i915_gem_busy_ioctl is still running. This won't free the GEM
objects memory, but still drops all the dma_fence references.
Now when dma_resv_for_each_fence_unlocked() sees a destroyed dma_fence it
assumes that a new fence list was installed and re-starts the loop.
But in the case of a destroyed GEM object a new fence list is never
installed, only the old one freed and therefore the iteration never
finishes resulting in an endless loop.
The solution is to drop the fence references only after the RCU grace
period.
The fixes tag is not necessary the patch introducing the problem, but the
one making it so worse that we need to address it.
This problem was pointed out by Sashiko-bot.
Signed-off-by: Christian König <christian.koenig@amd.com>
Fixes:
|
||
|---|---|---|
| .. | ||
| selftests | ||
| i915_gem_busy.c | ||
| i915_gem_clflush.c | ||
| i915_gem_clflush.h | ||
| i915_gem_context_types.h | ||
| i915_gem_context.c | ||
| i915_gem_context.h | ||
| i915_gem_create.c | ||
| i915_gem_create.h | ||
| i915_gem_dmabuf.c | ||
| i915_gem_dmabuf.h | ||
| i915_gem_domain.c | ||
| i915_gem_domain.h | ||
| i915_gem_execbuffer.c | ||
| i915_gem_internal.c | ||
| i915_gem_internal.h | ||
| i915_gem_ioctls.h | ||
| i915_gem_lmem.c | ||
| i915_gem_lmem.h | ||
| i915_gem_mman.c | ||
| i915_gem_mman.h | ||
| i915_gem_object_frontbuffer.c | ||
| i915_gem_object_frontbuffer.h | ||
| i915_gem_object_types.h | ||
| i915_gem_object.c | ||
| i915_gem_object.h | ||
| i915_gem_pages.c | ||
| i915_gem_panic.c | ||
| i915_gem_panic.h | ||
| i915_gem_phys.c | ||
| i915_gem_pm.c | ||
| i915_gem_pm.h | ||
| i915_gem_region.c | ||
| i915_gem_region.h | ||
| i915_gem_shmem.c | ||
| i915_gem_shrinker.c | ||
| i915_gem_shrinker.h | ||
| i915_gem_stolen.c | ||
| i915_gem_stolen.h | ||
| i915_gem_throttle.c | ||
| i915_gem_tiling.c | ||
| i915_gem_tiling.h | ||
| i915_gem_ttm_move.c | ||
| i915_gem_ttm_move.h | ||
| i915_gem_ttm_pm.c | ||
| i915_gem_ttm_pm.h | ||
| i915_gem_ttm.c | ||
| i915_gem_ttm.h | ||
| i915_gem_userptr.c | ||
| i915_gem_wait.c | ||