mirror of
https://github.com/AuxXxilium/linux_dsm_epyc7002.git
synced 2024-12-15 21:26:40 +07:00
drm/i915: Use a dummy timeline name for a signaled fence
Michał Winiarski pointed out that the debugging infrastructure (such as trace_dma_fence_release) likes to pretty print the timeline name, long after we have freed the timeline. Our timelines currently live as part of the GTT (due to the strict ordering we currently use through each) which belong to the context. We aim to free the context and release its hardware resources as soon as we able to (i.e. when the last fence/request using it has been signaled and retired). As the .get_timeline_name is purely a debug feature, rather than extending the lifetime of the context, or splitting it into many different release phases just to keep the name around, replace the timeline name with a constant after the fence has been signaled. This avoids the potential use-after-free. Reported-by: Krzysztof Olinski <krzysztof.e.olinski@intel.com> Fixes:80b204bce8
("drm/i915: Enable multiple timelines") Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Michał Winiarski <michal.winiarski@intel.com> Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com> Cc: <stable@vger.kernel.org> # v4.10+ Link: http://patchwork.freedesktop.org/patch/msgid/20170330111614.29757-1-chris@chris-wilson.co.uk Reviewed-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com> Reviewed-by: Michał Winiarski <michal.winiarski@intel.com> (cherry picked from commit05506b5be0
) Signed-off-by: Jani Nikula <jani.nikula@intel.com>
This commit is contained in:
parent
1383aeca92
commit
ecf8e89917
@ -37,6 +37,17 @@ static const char *i915_fence_get_driver_name(struct dma_fence *fence)
|
|||||||
|
|
||||||
static const char *i915_fence_get_timeline_name(struct dma_fence *fence)
|
static const char *i915_fence_get_timeline_name(struct dma_fence *fence)
|
||||||
{
|
{
|
||||||
|
/* The timeline struct (as part of the ppgtt underneath a context)
|
||||||
|
* may be freed when the request is no longer in use by the GPU.
|
||||||
|
* We could extend the life of a context to beyond that of all
|
||||||
|
* fences, possibly keeping the hw resource around indefinitely,
|
||||||
|
* or we just give them a false name. Since
|
||||||
|
* dma_fence_ops.get_timeline_name is a debug feature, the occasional
|
||||||
|
* lie seems justifiable.
|
||||||
|
*/
|
||||||
|
if (test_bit(DMA_FENCE_FLAG_SIGNALED_BIT, &fence->flags))
|
||||||
|
return "signaled";
|
||||||
|
|
||||||
return to_request(fence)->timeline->common->name;
|
return to_request(fence)->timeline->common->name;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user