[PATCH 4/4] reservation: revert "wait only with non-zero timeout specified (v3)" v2

Gustavo Padovan gustavo at padovan.org
Tue Nov 8 00:50:23 UTC 2016


2016-11-07 Alex Deucher <alexdeucher at gmail.com>:

> From: Christian König <christian.koenig at amd.com>
> 
> This reverts commit fb8b7d2b9d80e1e71f379e57355936bd2b024be9.
> 
> Otherwise signaling might never be activated on the fences. This can
> result in infinite waiting with hardware which has unreliable interrupts.
> 
> v2: still return one when the timeout is zero and we don't have any fences.
> 
> Reviewed-by: Alex Deucher <alexander.deucher at amd.com>
> Signed-off-by: Christian König <christian.koenig at amd.com>
> Reviewed-by: Chunming Zhou <david1.zhou at amd.com> (v1)
> Signed-off-by: Alex Deucher <alexander.deucher at amd.com>
> ---
>  drivers/dma-buf/reservation.c | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)

You left all my r-b out. For the whole series:

Reviewed-by: Gustavo Padovan <gustavo.padovan at collabora.co.uk> 

Gustavo


More information about the dri-devel mailing list