diff options
author | Christian König <christian.koenig@amd.com> | 2016-02-29 16:53:11 +0100 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2016-03-21 09:26:50 +0100 |
commit | b47bcb93bbf201e9c5af698945755efeb60c0bc8 (patch) | |
tree | 896718f2139a30d5b0bcfbf1d185e1fd474c0bd1 /include | |
parent | dma-buf: Update docs for SYNC ioctl (diff) | |
download | linux-b47bcb93bbf201e9c5af698945755efeb60c0bc8.tar.xz linux-b47bcb93bbf201e9c5af698945755efeb60c0bc8.zip |
dma-buf/fence: fix fence_is_later v2
A fence is never later than itself. This caused a bunch of overhead for AMDGPU.
v2: simplify check as suggested by Michel.
Signed-off-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Michel Dänzer <michel.daenzer@amd.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Reviewed-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Sumit Semwal <sumit.semwal@linaro.org>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Diffstat (limited to 'include')
-rw-r--r-- | include/linux/fence.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/fence.h b/include/linux/fence.h index bb522011383b..5aa95eb886f7 100644 --- a/include/linux/fence.h +++ b/include/linux/fence.h @@ -292,7 +292,7 @@ static inline bool fence_is_later(struct fence *f1, struct fence *f2) if (WARN_ON(f1->context != f2->context)) return false; - return f1->seqno - f2->seqno < INT_MAX; + return (int)(f1->seqno - f2->seqno) > 0; } /** |