diff options
| author | Tvrtko Ursulin <tvrtko.ursulin@intel.com> | 2019-06-27 13:25:52 +0100 |
|---|---|---|
| committer | Tvrtko Ursulin <tvrtko.ursulin@intel.com> | 2019-06-28 14:08:40 +0100 |
| commit | 03779dd3de8a57544f124d9952a6d2b3e34e34ca (patch) | |
| tree | 68075c745171cfcde06bf54f1a2f3a865618fba6 /lib | |
| parent | ad129d2a583689765eaef31ff57e8cdd219f1d05 (diff) | |
lib/i915: Use engine discovery in gem_test_engines
If request is to verify all engines work, use the new engine discovery API
so we can actually test all.
There is a sporadic and mysterious, possibly memory corruption issue
surronding this area so lets see if this changes the pattern.
It is easy to do in this function due it re-opening the driver and
throwing it away. So there are no concerns that the modified default
context can affect any other test.
Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
References: https://bugzilla.freedesktop.org/show_bug.cgi?id=110667
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/i915/gem_submission.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/lib/i915/gem_submission.c b/lib/i915/gem_submission.c index 2fd460d5..a8bb45c6 100644 --- a/lib/i915/gem_submission.c +++ b/lib/i915/gem_submission.c @@ -28,6 +28,8 @@ #include <i915_drm.h> +#include "i915/gem_engine_topology.h" + #include "igt_core.h" #include "igt_gt.h" #include "igt_sysfs.h" @@ -215,8 +217,10 @@ void gem_test_engine(int i915, unsigned int engine) gem_write(i915, obj.handle, 0, &bbe, sizeof(bbe)); if (engine == ALL_ENGINES) { - for_each_physical_engine(i915, engine) { - execbuf.flags = engine; + const struct intel_execution_engine2 *e2; + + __for_each_physical_engine(i915, e2) { + execbuf.flags = e2->flags; gem_execbuf(i915, &execbuf); } } else { |