diff options
author | Michel Dänzer <michel.daenzer@amd.com> | 2015-01-22 16:00:17 +0900 |
---|---|---|
committer | Sasha Levin <sasha.levin@oracle.com> | 2015-03-23 21:02:50 -0400 |
commit | 6a6eed5961f0c3181524fe6d425eeeaa8c9707bd (patch) | |
tree | eb052e5591208062068307610382f5aed01df7e3 | |
parent | eb67532e61435486a7bc747e36f0dda697d7817e (diff) |
reservation: Remove shadowing local variable 'ret'
commit 4eb2440ed60fb5793f7aa6da89b3d517cc59de43 upstream.
It was causing the return value of fence_is_signaled to be ignored, making
reservation objects signal too early.
Reviewed-by: Maarten Lankhorst <maarten.lankhorst@canonical.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Michel Dänzer <michel.daenzer@amd.com>
Signed-off-by: Sumit Semwal <sumit.semwal@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r-- | drivers/dma-buf/reservation.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/dma-buf/reservation.c b/drivers/dma-buf/reservation.c index 3c97c8fa8d02..8a37af923094 100644 --- a/drivers/dma-buf/reservation.c +++ b/drivers/dma-buf/reservation.c @@ -402,8 +402,6 @@ reservation_object_test_signaled_single(struct fence *passed_fence) int ret = 1; if (!test_bit(FENCE_FLAG_SIGNALED_BIT, &lfence->flags)) { - int ret; - fence = fence_get_rcu(lfence); if (!fence) return -1; |