https://bugs.freedesktop.org/show_bug.cgi?id=107341

Martin Peres <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #3 from Martin Peres <[email protected]> ---
(In reply to Chris Wilson from comment #2)
> commit 6e11ea9de9576a644045ffdc2067c09bc2012eda
> Author: Chris Wilson <[email protected]>
> Date:   Wed Jan 30 10:55:17 2019 +0000
> 
>     drm/amdgpu: Transfer fences to dmabuf importer
>     
>     amdgpu only uses shared-fences internally, but dmabuf importers rely on
>     implicit write hazard tracking via the reservation_object.fence_excl.
>     For example, the importer use the write hazard for timing a page flip to
>     only occur after the exporter has finished flushing its write into the
>     surface. As such, on exporting a dmabuf, we must either flush all
>     outstanding fences (for we do not know which are writes and should have
>     been exclusive) or alternatively create a new exclusive fence that is
>     the composite of all the existing shared fences, and so will only be
>     signaled when all earlier fences are signaled (ensuring that we can not
>     be signaled before the completion of any earlier write).
>     
>     v2: reservation_object is already locked by amdgpu_bo_reserve()
>     v3: Replace looping with get_fences_rcu and special case the promotion
>     of a single shared fence directly to an exclusive fence, bypassing the
>     fence array.
>     v4: Drop the fence array ref after assigning to reservation_object
>     
>     Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=107341
>     Testcase: igt/amd_prime/amd-to-i915
>     References: 8e94a46c1770 ("drm/amdgpu: Attach exclusive fence to prime
> exported bo's. (v5)")
>     Signed-off-by: Chris Wilson <[email protected]>
>     Cc: Alex Deucher <[email protected]>
>     Cc: "Christian König" <[email protected]>
>     Reviewed-by: "Christian König" <[email protected]>
>     Signed-off-by: Alex Deucher <[email protected]>

This is indeed fixed! Thanks!

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to