Bug 78024

Summary: [BDW]Subcases normal and secure-dispatch of gem_storedw_batches_loop are slow
Product: DRI Reporter: Guo Jinxian <jinxianx.guo>
Component: DRM/IntelAssignee: Mika Kuoppala <mika.kuoppala>
Status: CLOSED FIXED QA Contact: Intel GFX Bugs mailing list <intel-gfx-bugs>
Severity: normal    
Priority: medium CC: intel-gfx-bugs
Version: unspecified   
Hardware: Other   
OS: All   
Whiteboard:
i915 platform: i915 features:
Bug Depends on: 78724    
Bug Blocks:    
Attachments:
Description Flags
dmesg none

Description Guo Jinxian 2014-04-28 07:50:36 UTC
Created attachment 98106 [details]
dmesg

system Environment:
--------------------------
Platform: BDW
Kernel:(drm-intel-nightly)11ddb598492d1f97f894495eea398febb41a9eb0

Bug detailed description:
-----------------------------
Subcases normal and secure-dispatch of gem_storedw_batches_loop are slow on -nightly(11ddb598492d1f97f894495eea398febb41a9eb0) and -next-queued(b7c0d9df97c10ec5693a838df2fd53058f8e9e96). it's passed on -fixes(7f1950fbb989e8fc5463b307e062b4529d51c862).

it spends about 30 minutes.

output:
 date;./gem_storedw_batches_loop --run-subtest normal;date
Mon Apr 28 14:57:15 EDT 2014
IGT-Version: 1.6-ga595a40 (x86_64) (Linux: 3.15.0-rc2_drm-intel-nightly_11ddb5_20140428+ x86_64)
running storedw loop with stall every 1 batch
completed 524288 writes successfully
running storedw loop with stall every 2 batch
completed 524288 writes successfully
running storedw loop with stall every 3 batch
completed 524288 writes successfully
running storedw loop with stall every 5 batch
completed 524288 writes successfully
Subtest normal: SUCCESS
Mon Apr 28 15:27:27 EDT 2014

Reproduce steps:
----------------------------
1. ./gem_storedw_batches_loop --run-subtest normal
Comment 1 Daniel Vetter 2014-05-19 10:31:18 UTC
Can you please retest with latest -nightly?

Also does the test complete after 30minutes or just time out?
Comment 2 Jesse Barnes 2014-05-19 15:42:45 UTC
The loops are way too big in these tests.  I think they only need to write say a couple of pages worth of data, rather than the megabytes they do now.
Comment 3 Jesse Barnes 2014-05-19 15:45:26 UTC
commit f00efff326610fdba92dbc91d951790a3320052e
Author: Jesse Barnes <jbarnes@virtuousgeek.org>
Date:   Mon May 19 08:44:12 2014 -0700

    store_dw_loop: make loops smaller
Comment 4 Daniel Vetter 2014-05-19 16:14:14 UTC
commit 3005ac3ee8d7aede73a3e63d2068a6074156ebe5
Author: Daniel Vetter <daniel.vetter@ffwll.ch>
Date:   Mon May 19 18:11:34 2014 +0200

    Revert "store_dw_loop: make loops smaller"

Temporary revert until we have the test results. Needs to be re-reverted again after QA has run a test round and timed the tests again.
Comment 5 Guo Jinxian 2014-05-21 06:20:47 UTC
(In reply to comment #1)
> Can you please retest with latest -nightly?
> 
> Also does the test complete after 30minutes or just time out?

The test is able to finish on latest -nightly(36765340cb068dec1216342bfcdbf2678ec29860) less than 1 minute.

root@x-bdw05:/GFX/Test/Intel_gpu_tools/intel-gpu-tools/tests# time ./gem_storedw_batches_loop --run-subtest normal
IGT-Version: 1.6-g737d248 (x86_64) (Linux: 3.15.0-rc5_drm-intel-nightly_367653_20140521+ x86_64)
running storedw loop with stall every 1 batch
completed 524288 writes successfully
running storedw loop with stall every 2 batch
completed 524288 writes successfully
running storedw loop with stall every 3 batch
completed 524288 writes successfully
running storedw loop with stall every 5 batch
completed 524288 writes successfully
Subtest normal: SUCCESS

real    0m30.233s
user    0m4.197s
sys     0m25.706s
Comment 6 Guo Jinxian 2014-05-23 00:56:34 UTC
Verified on latest -next-queued(e6a7e5d0fd778bc6cf295414b9937723091c9bb6)

root@x-bdw05:/GFX/Test/Intel_gpu_tools/intel-gpu-tools/tests#  time ./gem_storedw_batches_loop --run-subtest normal
IGT-Version: 1.6-gc75dcbd (x86_64) (Linux: 3.15.0-rc3_drm-intel-next-queued_e6a7e5_20140522+ x86_64)
running storedw loop with stall every 1 batch
completed 524288 writes successfully
running storedw loop with stall every 2 batch
completed 524288 writes successfully
running storedw loop with stall every 3 batch
completed 524288 writes successfully
running storedw loop with stall every 5 batch
completed 524288 writes successfully
Subtest normal: SUCCESS

real    0m30.221s
user    0m3.965s
sys     0m25.908s
root@x-bdw05:/GFX/Test/Intel_gpu_tools/intel-gpu-tools/tests#  time ./gem_storedw_batches_loop --run-subtest secure-dispatch
IGT-Version: 1.6-gc75dcbd (x86_64) (Linux: 3.15.0-rc3_drm-intel-next-queued_e6a7e5_20140522+ x86_64)
running storedw loop with stall every 1 batch
completed 524288 writes successfully
running storedw loop with stall every 2 batch
completed 524288 writes successfully
running storedw loop with stall every 3 batch
completed 524288 writes successfully
running storedw loop with stall every 5 batch
completed 524288 writes successfully
Subtest secure-dispatch: SUCCESS

real    0m31.823s
user    0m4.262s
sys     0m27.230s
Comment 7 Elizabeth 2017-10-06 14:38:29 UTC
Closing old verified.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.