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

--- Comment #31 from Suzuki, Shinji <shinji.suz...@gmail.com> ---
(In reply to Michel Dänzer from comment #30)
> (In reply to Suzuki, Shinji from comment #29)
> Yes, that'll be patch 2, à la
> https://patchwork.freedesktop.org/patch/115220/ . :)
Why is this patch rejected?

Now I'm revisiting your patch. Do we need to have mutual exclusion on
   screen->fence_reference(screen, &fence, so->fence);
and
      screen->fence_reference(screen, &so->fence, NULL);
?
Concurrent calls to screen->fence_reference(screen, &so->fence, NULL) must be
arbitrated as they race on dereference/resetting of the lhs value. But I
believe, though not too confident, that reference counting itself is
implemented in thread-safe manner therefore no arbitration on
screen->fence_reference(screen, &fence, so->fence); is needed.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are the QA Contact for the bug.
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to