From 600d88ab3c11310b3d9c9dc1bf704282cb164ed3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jos=C3=A9=20Roberto=20de=20Souza?= Date: Fri, 9 Aug 2024 07:39:16 -0700 Subject: [PATCH] intel: Remove INTEL_ENGINE_CLASS_COMPUTE and INTEL_ENGINE_CLASS_COPY parameters MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit It has been a while that the GuC version with the compute engine fix was released, same for the KMD uAPI to query the GuC firmware version. So at this point this parameters do more harm than good. Also just setting those don't enable the async compute and copy engines this is not enabled by default on i915. If user wants to disable or enable usage of those engines a better approach would be use ANV_QUEUE_OVERRIDE. Reviewed-by: Paulo Zanoni Signed-off-by: José Roberto de Souza Part-of: --- src/intel/common/intel_engine.c | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/src/intel/common/intel_engine.c b/src/intel/common/intel_engine.c index b07c49331ef..882d26761a7 100644 --- a/src/intel/common/intel_engine.c +++ b/src/intel/common/intel_engine.c @@ -97,14 +97,9 @@ intel_engines_supported_count(int fd, const struct intel_device_info *info, { bool supported; - /* check if user set the force enabled engine with run-time parameter */ switch (engine_class) { case INTEL_ENGINE_CLASS_COMPUTE: - supported = debug_get_bool_option("INTEL_ENGINE_CLASS_COMPUTE", false); - supported |= is_guc_semaphore_functional(fd, info); - break; - case INTEL_ENGINE_CLASS_COPY: - supported = debug_get_bool_option("INTEL_ENGINE_CLASS_COPY", true); + supported = is_guc_semaphore_functional(fd, info); break; default: /* There is no restrictions or parameters for other engines */