Re: [PATCH I-g-t V4 0/2] Tests: Add test cases based on multi drm_fd to test sync

[Daniel Vetter <daniel@ffwll.ch>]

On Thu, Apr 24, 2014 at 10:15:43AM +0800, Zhao Yakui wrote:
> This follows Daniel's advice to add the two test cases based on multi drm_fd to 
> test the ring sync and CPU<->GPU sync.
> The Broadwell GT3 machine has two independent BSD rings that can be used
> to process the video commands. This is implemented in kernel driver and transparent
> to the user-space. But we still need to check the ring sync and CPU<->GPU sync for
> the second BSD ring. Two tests are created based on the multi drm_fds to
> test the sync. Multi drm_fd can assure that the second BSD ring has the opportunity
> to dispatch the GPU command. 
> 
> V1->V2: Follow Daniel's comment to add one subtext instead of one individual
> test case, which is used to test the CPU<->GPU sync under multi BSD rings/
> 
> V2->V3: Follow Imre's comment to remove the unnecessary initialization and
> use igt_assert_f instead of igt_assert
> 
> V3->V4: Add gem_multi_bsd_sync_loop.c into the tests/.gitignore
> 
> Zhao Yakui (2):
>   tests: Add one ring sync case based on multi drm_fd to test ring
>     semaphore sync under multi BSD rings
>   tests/gem_dummy_reloc_loop: Add one subtest based on multi drm_fd to
>     test CPU<->GPU sync under multi BSD rings

Merged, thanks for the patches.
-Daniel

> 
>  tests/.gitignore                |    1 +
>  tests/Makefile.sources          |    1 +
>  tests/gem_dummy_reloc_loop.c    |  107 +++++++++++++++++++++++-
>  tests/gem_multi_bsd_sync_loop.c |  175 +++++++++++++++++++++++++++++++++++++++
>  4 files changed, 283 insertions(+), 1 deletion(-)
>  create mode 100644 tests/gem_multi_bsd_sync_loop.c
> 
> -- 
> 1.7.10.1
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch