[igt-dev] [PATCH i-g-t 2/5] tests/gem_exec_big: Move check of pressumed offset out of no-reloc scope

Zbigniew Kempczyński zbigniew.kempczynski at intel.com
Tue Aug 17 06:31:15 UTC 2021


Missed during addressing last review - we don't want to process checking
of pressumed offset on no-reloc path. Move this out of no-reloc scope.

Signed-off-by: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>
Cc: Petri Latvala <petri.latvala at intel.com>
Cc: Ashutosh Dixit <ashutosh.dixit at intel.com>
Reviewed-by: Ashutosh Dixit <ashutosh.dixit at intel.com>
---
 tests/i915/gem_exec_big.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/tests/i915/gem_exec_big.c b/tests/i915/gem_exec_big.c
index 90230fc33..2f47de398 100644
--- a/tests/i915/gem_exec_big.c
+++ b/tests/i915/gem_exec_big.c
@@ -98,12 +98,13 @@ static void exec1(int fd, uint32_t handle, uint64_t reloc_ofs, unsigned flags, c
 
 	gem_execbuf(fd, &execbuf);
 
-	igt_warn_on(gem_reloc[0].presumed_offset == -1);
 	gem_set_domain(fd, gem_exec[0].handle, I915_GEM_DOMAIN_WC, 0);
 
 	if (!has_relocs)
 		return;
 
+	igt_warn_on(gem_reloc[0].presumed_offset == -1);
+
 	if (use_64bit_relocs) {
 		uint64_t tmp;
 		if (ptr)
-- 
2.26.0



More information about the igt-dev mailing list