diff options
author | Christian König <christian.koenig@amd.com> | 2017-08-07 17:32:21 -0400 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2017-08-09 15:01:32 -0400 |
commit | f3e31b73a54229a39db29345347b8f96d6cd312e (patch) | |
tree | 4a542d7be32ea4886a35ced54e483a102f319711 /drivers/dma-buf | |
parent | a5cb37d800a49922fdd80a20384b55b4a6b7bece (diff) |
dma-buf: dma_fence_put is NULL safe
No need to check.
Acked-by: Sumit Semwal <sumit.semwal@linaro.org>
Signed-off-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Link: https://patchwork.freedesktop.org/patch/msgid/1502141543-13455-2-git-send-email-alexander.deucher@amd.com
Diffstat (limited to 'drivers/dma-buf')
-rw-r--r-- | drivers/dma-buf/reservation.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/dma-buf/reservation.c b/drivers/dma-buf/reservation.c index 393817e849ed..87f8f5747500 100644 --- a/drivers/dma-buf/reservation.c +++ b/drivers/dma-buf/reservation.c | |||
@@ -195,8 +195,7 @@ done: | |||
195 | if (old) | 195 | if (old) |
196 | kfree_rcu(old, rcu); | 196 | kfree_rcu(old, rcu); |
197 | 197 | ||
198 | if (old_fence) | 198 | dma_fence_put(old_fence); |
199 | dma_fence_put(old_fence); | ||
200 | } | 199 | } |
201 | 200 | ||
202 | /** | 201 | /** |
@@ -258,8 +257,7 @@ void reservation_object_add_excl_fence(struct reservation_object *obj, | |||
258 | dma_fence_put(rcu_dereference_protected(old->shared[i], | 257 | dma_fence_put(rcu_dereference_protected(old->shared[i], |
259 | reservation_object_held(obj))); | 258 | reservation_object_held(obj))); |
260 | 259 | ||
261 | if (old_fence) | 260 | dma_fence_put(old_fence); |
262 | dma_fence_put(old_fence); | ||
263 | } | 261 | } |
264 | EXPORT_SYMBOL(reservation_object_add_excl_fence); | 262 | EXPORT_SYMBOL(reservation_object_add_excl_fence); |
265 | 263 | ||