summaryrefslogtreecommitdiffstats
path: root/drivers/dma-buf
diff options
context:
space:
mode:
authorDaniel Vetter <daniel.vetter@ffwll.ch>2017-07-20 08:51:07 -0400
committerDaniel Vetter <daniel.vetter@ffwll.ch>2017-07-26 07:45:07 -0400
commit6ce31263c9758c85e7d326cdc65c31fc521dc63d (patch)
tree2a814f9691364add420e75709d285ac944c16887 /drivers/dma-buf
parent4bf99144d2b407bf393e5a7663fe1ed69c646269 (diff)
dma-fence: Don't BUG_ON when not absolutely needed
It makes debugging a massive pain. Signed-off-by: Daniel Vetter <daniel.vetter@intel.com> Cc: Sumit Semwal <sumit.semwal@linaro.org> Cc: Gustavo Padovan <gustavo@padovan.org> Cc: linux-media@vger.kernel.org Cc: linaro-mm-sig@lists.linaro.org Reviewed-by: Lucas Stach <l.stach@pengutronix.de> Acked-by: Sumit Semwal <sumit.semwal@linaro.org> Acked-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> Link: https://patchwork.freedesktop.org/patch/msgid/20170720125107.26693-1-daniel.vetter@ffwll.ch Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Diffstat (limited to 'drivers/dma-buf')
-rw-r--r--drivers/dma-buf/dma-fence.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/dma-buf/dma-fence.c b/drivers/dma-buf/dma-fence.c
index 57da14c15987..eee8377b8b3a 100644
--- a/drivers/dma-buf/dma-fence.c
+++ b/drivers/dma-buf/dma-fence.c
@@ -48,7 +48,7 @@ static atomic64_t dma_fence_context_counter = ATOMIC64_INIT(0);
48 */ 48 */
49u64 dma_fence_context_alloc(unsigned num) 49u64 dma_fence_context_alloc(unsigned num)
50{ 50{
51 BUG_ON(!num); 51 WARN_ON(!num);
52 return atomic64_add_return(num, &dma_fence_context_counter) - num; 52 return atomic64_add_return(num, &dma_fence_context_counter) - num;
53} 53}
54EXPORT_SYMBOL(dma_fence_context_alloc); 54EXPORT_SYMBOL(dma_fence_context_alloc);
@@ -177,7 +177,7 @@ void dma_fence_release(struct kref *kref)
177 177
178 trace_dma_fence_destroy(fence); 178 trace_dma_fence_destroy(fence);
179 179
180 BUG_ON(!list_empty(&fence->cb_list)); 180 WARN_ON(!list_empty(&fence->cb_list));
181 181
182 if (fence->ops->release) 182 if (fence->ops->release)
183 fence->ops->release(fence); 183 fence->ops->release(fence);