Re: [PATCH 2/2] dma-buf: fix reservation_object_wait_timeout_rcu to wait correctly v2

2017-08-10 Thread Christian König
Am 10.08.2017 um 20:50 schrieb Chris Wilson: Quoting Christian König (2017-08-10 19:19:52) Am 10.08.2017 um 19:11 schrieb Chris Wilson: Quoting Alex Deucher (2017-08-10 18:01:49) From: Christian König With hardware resets in mind it is possible that all shared fences are signaled, but the ex

Re: [PATCH 2/2] dma-buf: fix reservation_object_wait_timeout_rcu to wait correctly v2

2017-08-10 Thread Chris Wilson
Quoting Christian König (2017-08-10 19:19:52) > Am 10.08.2017 um 19:11 schrieb Chris Wilson: > > Quoting Alex Deucher (2017-08-10 18:01:49) > >> From: Christian König > >> > >> With hardware resets in mind it is possible that all shared fences are > >> signaled, but the exlusive isn't. Fix waiting

Re: [PATCH 2/2] dma-buf: fix reservation_object_wait_timeout_rcu to wait correctly v2

2017-08-10 Thread Christian König
Am 10.08.2017 um 19:11 schrieb Chris Wilson: Quoting Alex Deucher (2017-08-10 18:01:49) From: Christian König With hardware resets in mind it is possible that all shared fences are signaled, but the exlusive isn't. Fix waiting for everything in this situation. I'm still puzzling over this one

Re: [PATCH 2/2] dma-buf: fix reservation_object_wait_timeout_rcu to wait correctly v2

2017-08-10 Thread Chris Wilson
Quoting Alex Deucher (2017-08-10 18:01:49) > From: Christian König > > With hardware resets in mind it is possible that all shared fences are > signaled, but the exlusive isn't. Fix waiting for everything in this > situation. I'm still puzzling over this one. Setting an exclusive fence will cl

[PATCH 2/2] dma-buf: fix reservation_object_wait_timeout_rcu to wait correctly v2

2017-08-10 Thread Alex Deucher
From: Christian König With hardware resets in mind it is possible that all shared fences are signaled, but the exlusive isn't. Fix waiting for everything in this situation. v2: make sure we always wait for the exclusive fence Signed-off-by: Christian König Reviewed-by: Alex Deucher Reviewed-b