On 17/05/18 01:23, Chris Wilson wrote:
Confirm we have the available HW before asserting it succeeds.

Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>
---
  tests/gem_cpu_reloc.c | 1 +
  1 file changed, 1 insertion(+)

diff --git a/tests/gem_cpu_reloc.c b/tests/gem_cpu_reloc.c
index 882c312d4..e3bbcd239 100644
--- a/tests/gem_cpu_reloc.c
+++ b/tests/gem_cpu_reloc.c
@@ -167,6 +167,7 @@ static void run_test(int fd, int count)
        use_blt = 0;

Is this^ meant to be EXEC_DEFAULT?

        if (intel_gen(noop) >= 6)
                use_blt = I915_EXEC_BLT;
+       gem_require_ring(fd, use_blt);

Are any gens 6+ that do not have a BLT ring? if that is the case shouldn't we use '0' like we do for 5- gens?

Thanks,
Antonio

if (intel_gen(noop) >= 8) {
                batch = gen8_batch;

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

Reply via email to