diff options
author | Christian König <christian.koenig@amd.com> | 2016-03-14 10:23:11 -0400 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2016-03-16 18:00:01 -0400 |
commit | 480d0bf07ecf27da0a5f12774d9642072c364fa9 (patch) | |
tree | ef580c534a93d08f96b845bd300756a3582de0f0 /drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c | |
parent | 91cc6418a0ea14633bdc1d2ea91d174fc1e9187d (diff) |
drm/amdgpu: remove amdgpu_fence_is_signaled
It's just overhead to check the fence value
when we signal them directly anyway.
Signed-off-by: Christian König <christian.koenig@amd.com>
Acked-by: Alex Deucher <alexander.deucher@amd.com>
Reviewed-by: Chunming Zhou <david1.zhou@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c')
-rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c | 25 |
1 files changed, 0 insertions, 25 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c index fa4eabeee71d..da9a155a622c 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c | |||
@@ -547,30 +547,6 @@ static const char *amdgpu_fence_get_timeline_name(struct fence *f) | |||
547 | } | 547 | } |
548 | 548 | ||
549 | /** | 549 | /** |
550 | * amdgpu_fence_is_signaled - test if fence is signaled | ||
551 | * | ||
552 | * @f: fence to test | ||
553 | * | ||
554 | * Test the fence sequence number if it is already signaled. If it isn't | ||
555 | * signaled start fence processing. Returns True if the fence is signaled. | ||
556 | */ | ||
557 | static bool amdgpu_fence_is_signaled(struct fence *f) | ||
558 | { | ||
559 | struct amdgpu_fence *fence = to_amdgpu_fence(f); | ||
560 | struct amdgpu_ring *ring = fence->ring; | ||
561 | |||
562 | if (atomic64_read(&ring->fence_drv.last_seq) >= fence->seq) | ||
563 | return true; | ||
564 | |||
565 | amdgpu_fence_process(ring); | ||
566 | |||
567 | if (atomic64_read(&ring->fence_drv.last_seq) >= fence->seq) | ||
568 | return true; | ||
569 | |||
570 | return false; | ||
571 | } | ||
572 | |||
573 | /** | ||
574 | * amdgpu_fence_enable_signaling - enable signalling on fence | 550 | * amdgpu_fence_enable_signaling - enable signalling on fence |
575 | * @fence: fence | 551 | * @fence: fence |
576 | * | 552 | * |
@@ -622,7 +598,6 @@ static const struct fence_ops amdgpu_fence_ops = { | |||
622 | .get_driver_name = amdgpu_fence_get_driver_name, | 598 | .get_driver_name = amdgpu_fence_get_driver_name, |
623 | .get_timeline_name = amdgpu_fence_get_timeline_name, | 599 | .get_timeline_name = amdgpu_fence_get_timeline_name, |
624 | .enable_signaling = amdgpu_fence_enable_signaling, | 600 | .enable_signaling = amdgpu_fence_enable_signaling, |
625 | .signaled = amdgpu_fence_is_signaled, | ||
626 | .wait = fence_default_wait, | 601 | .wait = fence_default_wait, |
627 | .release = amdgpu_fence_release, | 602 | .release = amdgpu_fence_release, |
628 | }; | 603 | }; |