[Intel-gfx] [PATCH] Revert "store_dw_loop: make loops smaller"

Daniel Vetter daniel.vetter at ffwll.ch
Mon May 19 18:13:22 CEST 2014


This reverts commit f00efff326610fdba92dbc91d951790a3320052e.

This is a temporary revert since I want QA to first test with the
original testcase whether it got faster again. This is to test the
effects of

commit 227f782e4667fc622810bce8be8ccdeee45f89c2
Author: Chris Wilson <chris at chris-wilson.co.uk>
Date:   Thu May 15 10:41:42 2014 +0100

    drm/i915: Retire requests before creating a new one

We should revert this revert again as soon as QA has completed
testing.

References: https://bugs.freedesktop.org/show_bug.cgi?id=78024
Cc: Jesse Barnes <jbarnes at virtuousgeek.org>
Signed-off-by: Daniel Vetter <daniel.vetter at ffwll.ch>
---
 tests/gem_storedw_batches_loop.c | 2 +-
 tests/gem_storedw_loop_blt.c     | 2 +-
 tests/gem_storedw_loop_bsd.c     | 2 +-
 tests/gem_storedw_loop_render.c  | 2 +-
 tests/gem_storedw_loop_vebox.c   | 2 +-
 5 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/tests/gem_storedw_batches_loop.c b/tests/gem_storedw_batches_loop.c
index fef3718f95df..fe85387f47c6 100644
--- a/tests/gem_storedw_batches_loop.c
+++ b/tests/gem_storedw_batches_loop.c
@@ -62,7 +62,7 @@ store_dword_loop(int divider, unsigned flags)
 	if (!has_ppgtt)
 		cmd |= MI_MEM_VIRTUAL;
 
-	for (i = 0; i < SLOW_QUICK(0x2000, 4); i++) {
+	for (i = 0; i < SLOW_QUICK(0x80000, 4); i++) {
 		int j = 0;
 		int cmd_address_offset;
 		cmd_bo = drm_intel_bo_alloc(bufmgr, "cmd bo", 4096, 4096);
diff --git a/tests/gem_storedw_loop_blt.c b/tests/gem_storedw_loop_blt.c
index 8c4157d5d1c4..06afe14f210a 100644
--- a/tests/gem_storedw_loop_blt.c
+++ b/tests/gem_storedw_loop_blt.c
@@ -86,7 +86,7 @@ store_dword_loop(int devid, int divider)
 
 	igt_info("running storedw loop on render with stall every %i batch\n", divider);
 
-	for (i = 0; i < SLOW_QUICK(0x2000, 0x10); i++) {
+	for (i = 0; i < SLOW_QUICK(0x100000, 0x10); i++) {
 		emit_store_dword_imm(devid, target_buffer, val);
 		intel_batchbuffer_flush_on_ring(batch, I915_EXEC_BLT);
 
diff --git a/tests/gem_storedw_loop_bsd.c b/tests/gem_storedw_loop_bsd.c
index f89d522e2b41..14a56ba69205 100644
--- a/tests/gem_storedw_loop_bsd.c
+++ b/tests/gem_storedw_loop_bsd.c
@@ -86,7 +86,7 @@ store_dword_loop(int devid, int divider)
 
 	igt_info("running storedw loop on render with stall every %i batch\n", divider);
 
-	for (i = 0; i < SLOW_QUICK(0x2000, 0x10); i++) {
+	for (i = 0; i < SLOW_QUICK(0x100000, 0x10); i++) {
 		emit_store_dword_imm(devid, target_buffer, val);
 		intel_batchbuffer_flush_on_ring(batch, I915_EXEC_BSD);
 
diff --git a/tests/gem_storedw_loop_render.c b/tests/gem_storedw_loop_render.c
index 9defc6d6932e..fb1e6fdb0bf3 100644
--- a/tests/gem_storedw_loop_render.c
+++ b/tests/gem_storedw_loop_render.c
@@ -86,7 +86,7 @@ store_dword_loop(int devid, int divider)
 
 	igt_info("running storedw loop on render with stall every %i batch\n", divider);
 
-	for (i = 0; i < SLOW_QUICK(0x2000, 0x10); i++) {
+	for (i = 0; i < SLOW_QUICK(0x100000, 0x10); i++) {
 		emit_store_dword_imm(devid, target_buffer, val);
 		intel_batchbuffer_flush_on_ring(batch, 0);
 
diff --git a/tests/gem_storedw_loop_vebox.c b/tests/gem_storedw_loop_vebox.c
index c94a67707065..b8215fc4919c 100644
--- a/tests/gem_storedw_loop_vebox.c
+++ b/tests/gem_storedw_loop_vebox.c
@@ -61,7 +61,7 @@ store_dword_loop(int divider)
 
 	cmd = MI_STORE_DWORD_IMM;
 
-	for (i = 0; i < SLOW_QUICK(0x2000, 0x10); i++) {
+	for (i = 0; i < SLOW_QUICK(0x100000, 0x10); i++) {
 		BEGIN_BATCH(4);
 		OUT_BATCH(cmd);
 		if (intel_gen(batch->devid) < 8)
-- 
1.9.2




More information about the Intel-gfx mailing list