From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH I-g-t V4 0/2] Tests: Add test cases based on multi drm_fd to test sync Date: Thu, 24 Apr 2014 11:54:51 +0200 Message-ID: <20140424093253.GA26374@phenom.ffwll.local> References: <1398305745-15846-1-git-send-email-yakui.zhao@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-ee0-f47.google.com (mail-ee0-f47.google.com [74.125.83.47]) by gabe.freedesktop.org (Postfix) with ESMTP id 9469D6E6D4 for ; Thu, 24 Apr 2014 02:55:04 -0700 (PDT) Received: by mail-ee0-f47.google.com with SMTP id b15so1640424eek.20 for ; Thu, 24 Apr 2014 02:55:03 -0700 (PDT) Content-Disposition: inline In-Reply-To: <1398305745-15846-1-git-send-email-yakui.zhao@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Zhao Yakui Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org 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