diff options
author | Chunming Zhou <david1.zhou@amd.com> | 2015-08-03 01:28:16 -0400 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2015-08-17 16:50:52 -0400 |
commit | ed40bfb81af54fcdc1b96ffdeb14d69458234642 (patch) | |
tree | 6bc587cdfa283a0d35c892da1b713a8bb729c629 | |
parent | 0e3f154a9eb9f7a5ec365c4586a606ba882a3f15 (diff) |
drm/amdgpu: use kernel fence for vce ib test
Signed-off-by: Chunming Zhou <david1.zhou@amd.com>
Reviewed-by: Christian K?nig <christian.koenig@amd.com>
Reviewed-by: Jammy Zhou <jammy.zhou@amd.com>
-rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_test.c | 5 | ||||
-rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_vce.c | 14 | ||||
-rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_vce.h | 4 |
3 files changed, 12 insertions, 11 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_test.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_test.c index 567c5b3c6f3a..251b1ebef722 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_test.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_test.c | |||
@@ -260,18 +260,19 @@ static int amdgpu_test_create_and_emit_fence(struct amdgpu_device *adev, | |||
260 | 260 | ||
261 | } else if (ring == &adev->vce.ring[0] || | 261 | } else if (ring == &adev->vce.ring[0] || |
262 | ring == &adev->vce.ring[1]) { | 262 | ring == &adev->vce.ring[1]) { |
263 | struct fence *f = NULL; | ||
263 | r = amdgpu_vce_get_create_msg(ring, handle, NULL); | 264 | r = amdgpu_vce_get_create_msg(ring, handle, NULL); |
264 | if (r) { | 265 | if (r) { |
265 | DRM_ERROR("Failed to get dummy create msg\n"); | 266 | DRM_ERROR("Failed to get dummy create msg\n"); |
266 | return r; | 267 | return r; |
267 | } | 268 | } |
268 | 269 | ||
269 | r = amdgpu_vce_get_destroy_msg(ring, handle, fence); | 270 | r = amdgpu_vce_get_destroy_msg(ring, handle, &f); |
270 | if (r) { | 271 | if (r) { |
271 | DRM_ERROR("Failed to get dummy destroy msg\n"); | 272 | DRM_ERROR("Failed to get dummy destroy msg\n"); |
272 | return r; | 273 | return r; |
273 | } | 274 | } |
274 | 275 | *fence = to_amdgpu_fence(f); | |
275 | } else { | 276 | } else { |
276 | r = amdgpu_ring_lock(ring, 64); | 277 | r = amdgpu_ring_lock(ring, 64); |
277 | if (r) { | 278 | if (r) { |
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vce.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vce.c index 59acb0b990e1..94c40ca5ba74 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vce.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vce.c | |||
@@ -358,7 +358,7 @@ static int amdgpu_vce_free_job( | |||
358 | * Open up a stream for HW test | 358 | * Open up a stream for HW test |
359 | */ | 359 | */ |
360 | int amdgpu_vce_get_create_msg(struct amdgpu_ring *ring, uint32_t handle, | 360 | int amdgpu_vce_get_create_msg(struct amdgpu_ring *ring, uint32_t handle, |
361 | struct amdgpu_fence **fence) | 361 | struct fence **fence) |
362 | { | 362 | { |
363 | const unsigned ib_size_dw = 1024; | 363 | const unsigned ib_size_dw = 1024; |
364 | struct amdgpu_ib *ib = NULL; | 364 | struct amdgpu_ib *ib = NULL; |
@@ -412,7 +412,7 @@ int amdgpu_vce_get_create_msg(struct amdgpu_ring *ring, uint32_t handle, | |||
412 | if (r) | 412 | if (r) |
413 | goto err; | 413 | goto err; |
414 | if (fence) | 414 | if (fence) |
415 | *fence = amdgpu_fence_ref(ib->fence); | 415 | *fence = fence_get(&ib->fence->base); |
416 | if (amdgpu_enable_scheduler) | 416 | if (amdgpu_enable_scheduler) |
417 | return 0; | 417 | return 0; |
418 | err: | 418 | err: |
@@ -432,7 +432,7 @@ err: | |||
432 | * Close up a stream for HW test or if userspace failed to do so | 432 | * Close up a stream for HW test or if userspace failed to do so |
433 | */ | 433 | */ |
434 | int amdgpu_vce_get_destroy_msg(struct amdgpu_ring *ring, uint32_t handle, | 434 | int amdgpu_vce_get_destroy_msg(struct amdgpu_ring *ring, uint32_t handle, |
435 | struct amdgpu_fence **fence) | 435 | struct fence **fence) |
436 | { | 436 | { |
437 | const unsigned ib_size_dw = 1024; | 437 | const unsigned ib_size_dw = 1024; |
438 | struct amdgpu_ib *ib = NULL; | 438 | struct amdgpu_ib *ib = NULL; |
@@ -476,7 +476,7 @@ int amdgpu_vce_get_destroy_msg(struct amdgpu_ring *ring, uint32_t handle, | |||
476 | if (r) | 476 | if (r) |
477 | goto err; | 477 | goto err; |
478 | if (fence) | 478 | if (fence) |
479 | *fence = amdgpu_fence_ref(ib->fence); | 479 | *fence = fence_get(&ib->fence->base); |
480 | if (amdgpu_enable_scheduler) | 480 | if (amdgpu_enable_scheduler) |
481 | return 0; | 481 | return 0; |
482 | err: | 482 | err: |
@@ -827,7 +827,7 @@ int amdgpu_vce_ring_test_ring(struct amdgpu_ring *ring) | |||
827 | */ | 827 | */ |
828 | int amdgpu_vce_ring_test_ib(struct amdgpu_ring *ring) | 828 | int amdgpu_vce_ring_test_ib(struct amdgpu_ring *ring) |
829 | { | 829 | { |
830 | struct amdgpu_fence *fence = NULL; | 830 | struct fence *fence = NULL; |
831 | int r; | 831 | int r; |
832 | 832 | ||
833 | r = amdgpu_vce_get_create_msg(ring, 1, NULL); | 833 | r = amdgpu_vce_get_create_msg(ring, 1, NULL); |
@@ -842,13 +842,13 @@ int amdgpu_vce_ring_test_ib(struct amdgpu_ring *ring) | |||
842 | goto error; | 842 | goto error; |
843 | } | 843 | } |
844 | 844 | ||
845 | r = amdgpu_fence_wait(fence, false); | 845 | r = fence_wait(fence, false); |
846 | if (r) { | 846 | if (r) { |
847 | DRM_ERROR("amdgpu: fence wait failed (%d).\n", r); | 847 | DRM_ERROR("amdgpu: fence wait failed (%d).\n", r); |
848 | } else { | 848 | } else { |
849 | DRM_INFO("ib test on ring %d succeeded\n", ring->idx); | 849 | DRM_INFO("ib test on ring %d succeeded\n", ring->idx); |
850 | } | 850 | } |
851 | error: | 851 | error: |
852 | amdgpu_fence_unref(&fence); | 852 | fence_put(fence); |
853 | return r; | 853 | return r; |
854 | } | 854 | } |
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vce.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_vce.h index 7ccdb5927da5..ba2da8ee5906 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vce.h +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vce.h | |||
@@ -29,9 +29,9 @@ int amdgpu_vce_sw_fini(struct amdgpu_device *adev); | |||
29 | int amdgpu_vce_suspend(struct amdgpu_device *adev); | 29 | int amdgpu_vce_suspend(struct amdgpu_device *adev); |
30 | int amdgpu_vce_resume(struct amdgpu_device *adev); | 30 | int amdgpu_vce_resume(struct amdgpu_device *adev); |
31 | int amdgpu_vce_get_create_msg(struct amdgpu_ring *ring, uint32_t handle, | 31 | int amdgpu_vce_get_create_msg(struct amdgpu_ring *ring, uint32_t handle, |
32 | struct amdgpu_fence **fence); | 32 | struct fence **fence); |
33 | int amdgpu_vce_get_destroy_msg(struct amdgpu_ring *ring, uint32_t handle, | 33 | int amdgpu_vce_get_destroy_msg(struct amdgpu_ring *ring, uint32_t handle, |
34 | struct amdgpu_fence **fence); | 34 | struct fence **fence); |
35 | void amdgpu_vce_free_handles(struct amdgpu_device *adev, struct drm_file *filp); | 35 | void amdgpu_vce_free_handles(struct amdgpu_device *adev, struct drm_file *filp); |
36 | int amdgpu_vce_ring_parse_cs(struct amdgpu_cs_parser *p, uint32_t ib_idx); | 36 | int amdgpu_vce_ring_parse_cs(struct amdgpu_cs_parser *p, uint32_t ib_idx); |
37 | bool amdgpu_vce_ring_emit_semaphore(struct amdgpu_ring *ring, | 37 | bool amdgpu_vce_ring_emit_semaphore(struct amdgpu_ring *ring, |