If we can't find the enable_execlists parameter, presume that the switch is forced by the kernel and enabled for all hw supporting execlists. We don't have a GETPARAM or ENGINE_INFO to query the internal details. Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> --- lib/i915/gem_submission.c | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/lib/i915/gem_submission.c b/lib/i915/gem_submission.c index 882d4f41..8bff4844 100644 --- a/lib/i915/gem_submission.c +++ b/lib/i915/gem_submission.c @@ -28,6 +28,7 @@ #include "igt_core.h" #include "igt_sysfs.h" +#include "intel_chipset.h" #include "i915/gem_submission.h" @@ -64,30 +65,30 @@ static bool has_semaphores(int fd, int dir) */ unsigned gem_submission_method(int fd) { + const int gen = intel_gen(intel_get_drm_devid(fd)); unsigned flags = 0; - bool active; + int result; + int dir; dir = igt_sysfs_open_parameters(fd); if (dir < 0) return 0; - active = igt_sysfs_get_boolean(dir, "enable_guc_submission"); - if (active) { + if (igt_sysfs_get_boolean(dir, "enable_guc_submission")) { flags |= GEM_SUBMISSION_GUC | GEM_SUBMISSION_EXECLISTS; goto out; } - active = igt_sysfs_get_boolean(dir, "enable_execlists"); - if (active) { + if (igt_sysfs_scanf(dir, "enable_execlists", "%d", &result) != 1) + result = gen >= 8; + if (result) { flags |= GEM_SUBMISSION_EXECLISTS; goto out; } - active = has_semaphores(fd, dir); - if (active) { + if (has_semaphores(fd, dir)) flags |= GEM_SUBMISSION_SEMAPHORES; - } out: close(dir); -- 2.15.0 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx