Re: [PATCH i-g-t 1/3] igt/gem_cpu_reloc: Check HW exists before attempting to use it

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

 





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

Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
---
  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@xxxxxxxxxxxxxxxxxxxxx
https://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