[SYCL][UR] Fully remove enabling the validation layer thru the SYCL init

Signed-off-by: Neil R. Spruit <neil.r.spruit@intel.com>
nrspruit · Aug 26, 2024 · 0cb0e5f · 0cb0e5f
1 parent 3311870
commit 0cb0e5f
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 10 deletions.
diff --git a/sycl/cmake/modules/FetchUnifiedRuntime.cmake b/sycl/cmake/modules/FetchUnifiedRuntime.cmake
@@ -141,8 +141,8 @@ if(SYCL_UR_USE_FETCH_CONTENT)
   )
 
   fetch_adapter_source(opencl
-    ${UNIFIED_RUNTIME_REPO}
-    ${UNIFIED_RUNTIME_TAG}
+    https://github.com/nrspruit/unified-runtime.git
+    be9094363c4f8ddd94a41d8b001b7701deffb48b
   )
 
   fetch_adapter_source(cuda

diff --git a/sycl/source/detail/ur.cpp b/sycl/source/detail/ur.cpp
@@ -109,14 +109,9 @@ static void initializePlugins(std::vector<PluginPtr> &Plugins,
   __SYCL_CHECK_OCL_CODE_NO_EXC(Call)
 
   bool OwnLoaderConfig = false;
-  // If we weren't provided with a custom config handle create our own and
-  // enable full validation by default.
+  // If we weren't provided with a custom config handle create our own.
   if(!LoaderConfig) {
     CHECK_UR_SUCCESS(urLoaderConfigCreate(&LoaderConfig))
-    if (std::getenv("UR_LOG_VALIDATION")) {
-      CHECK_UR_SUCCESS(urLoaderConfigEnableLayer(
-          LoaderConfig, "UR_LAYER_PARAMETER_VALIDATION"))
-    }
     OwnLoaderConfig = true;
   }
 

diff --git a/sycl/test-e2e/format.py b/sycl/test-e2e/format.py
@@ -201,8 +201,6 @@ def get_extra_env(sycl_devices):
             if "level_zero:gpu" in sycl_devices and litConfig.params.get("ur_l0_debug"):
                 extra_env.append("UR_L0_DEBUG={}".format(test.config.ur_l0_debug))
 
-            extra_env.append('UR_LOG_VALIDATION="level:info;output:stdout;flush:info"')
-
             if "level_zero:gpu" in sycl_devices and litConfig.params.get(
                 "ur_l0_leaks_debug"
             ):