[PATCH 1/4] tests: Fix "-thrashing" and "-thrash-inactive" distinction

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Oscar Mateo <oscar.mateo@xxxxxxxxx>

A typo in the relocation tests made both sub-tests perform the
same action: drop *all* caches.

Signed-off-by: Oscar Mateo <oscar.mateo@xxxxxxxxx>
---
 tests/gem_persistent_relocs.c |    2 +-
 tests/gem_reloc_vs_gpu.c      |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/gem_persistent_relocs.c b/tests/gem_persistent_relocs.c
index 29b19ed..34d8492 100644
--- a/tests/gem_persistent_relocs.c
+++ b/tests/gem_persistent_relocs.c
@@ -289,7 +289,7 @@ static void do_forked_test(int fd, unsigned flags)
 	if (flags & (THRASH | THRASH_INACTIVE)) {
 		char fname[FILENAME_MAX];
 		int drop_caches_fd;
-		const char *data = THRASH_INACTIVE ? "0xf" : "0x7";
+		const char *data = (flags & THRASH_INACTIVE) ? "0x7" : "0xf";
 
 		snprintf(fname, FILENAME_MAX, "%s/%i/%s",
 			 "/sys/kernel/debug/dri", drm_get_card(),
diff --git a/tests/gem_reloc_vs_gpu.c b/tests/gem_reloc_vs_gpu.c
index ae7b446..bbfdc3a 100644
--- a/tests/gem_reloc_vs_gpu.c
+++ b/tests/gem_reloc_vs_gpu.c
@@ -284,7 +284,7 @@ static void do_forked_test(int fd, unsigned flags)
 	if (flags & (THRASH | THRASH_INACTIVE)) {
 		char fname[FILENAME_MAX];
 		int drop_caches_fd;
-		const char *data = THRASH_INACTIVE ? "0xf" : "0x7";
+		const char *data = (flags & THRASH_INACTIVE) ? "0x7" : "0xf";
 
 		snprintf(fname, FILENAME_MAX, "%s/%i/%s",
 			 "/sys/kernel/debug/dri", drm_get_card(),
-- 
1.7.9.5

_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux