fix: log successful apis in the validation layer only if verbose is set

Signed-off-by: Neil R. Spruit <neil.r.spruit@intel.com>

Author: nrspruit

README.md

@@ -90,6 +90,10 @@ validation layer is enabled. Following variables need to be set to enable API lo
 
 `ZE_ENABLE_VALIDATION_LAYER=1`
 
+To print successful API call results, set
+`ZEL_LOADER_LOGGING_VERBOSE=1`
+Otherwise, only error results will be printed in the API trace output.
+
 By default logs will be written to the log file, as described above. To print the logs
 to stderr instead, `ZEL_LOADER_LOG_CONSOLE=1` needs to be set.
 

scripts/templates/validation/valddi.cpp.mako

@@ -65,6 +65,10 @@ namespace validation_layer
         %endfor
 %endif
 ) {
+        // Only log success results if verbose logging is enabled
+        if (result == ${X}_RESULT_SUCCESS && !context.verboseLogging) {
+            return result;
+        }
         std::string status = (result == ${X}_RESULT_SUCCESS) ? "SUCCESS" : "ERROR";
         %if is_void_params:
         context.logger->log_trace(status + " (" + loader::to_string(result) + ") in ${func_name}()");
@@ -117,6 +121,10 @@ namespace validation_layer
         const void* desc,
         ze_event_handle_t* phEvent
     ) {
+        // Only log success results if verbose logging is enabled
+        if (result == ${X}_RESULT_SUCCESS && !context.verboseLogging) {
+            return result;
+        }
         std::string status = (result == ${X}_RESULT_SUCCESS) ? "SUCCESS" : "ERROR";
         std::ostringstream oss;
         oss << status << " (" << loader::to_string(result) << ") in zexCounterBasedEventCreate2("

source/layers/validation/ze_valddi.cpp

@@ -23,6 +23,7 @@ namespace validation_layer
             handleLifetime = std::make_unique<HandleLifetimeValidation>();
         }
         enableThreadingValidation = getenv_tobool( "ZE_ENABLE_THREADING_VALIDATION" );
+        verboseLogging = getenv_tobool( "ZEL_LOADER_LOGGING_VERBOSE" );
 
         logger = loader::createLogger();
     }

source/layers/validation/ze_validation_layer.cpp

@@ -47,6 +47,7 @@ namespace validation_layer
 
         bool enableHandleLifetime = false;
         bool enableThreadingValidation = false;
+        bool verboseLogging = false;
 
         ze_dditable_t   zeDdiTable = {};
         zet_dditable_t  zetDdiTable = {};

source/layers/validation/ze_validation_layer.h

@@ -31,6 +31,10 @@ namespace validation_layer
         const char** ppString                           ///< [in,out] pointer to a null-terminated array of characters describing
                                                         ///< cause of error.
 ) {
+        // Only log success results if verbose logging is enabled
+        if (result == ZE_RESULT_SUCCESS && !context.verboseLogging) {
+            return result;
+        }
         std::string status = (result == ZE_RESULT_SUCCESS) ? "SUCCESS" : "ERROR";
         std::ostringstream oss;
         oss << status << " (" << loader::to_string(result) << ") in zerGetLastErrorDescription(";
@@ -46,6 +50,10 @@ namespace validation_layer
         ze_result_t result,
         ze_device_handle_t hDevice                      ///< [in] handle of the device
 ) {
+        // Only log success results if verbose logging is enabled
+        if (result == ZE_RESULT_SUCCESS && !context.verboseLogging) {
+            return result;
+        }
         std::string status = (result == ZE_RESULT_SUCCESS) ? "SUCCESS" : "ERROR";
         std::ostringstream oss;
         oss << status << " (" << loader::to_string(result) << ") in zerTranslateDeviceHandleToIdentifier(";
@@ -61,6 +69,10 @@ namespace validation_layer
         ze_result_t result,
         uint32_t identifier                             ///< [in] integer identifier of the device
 ) {
+        // Only log success results if verbose logging is enabled
+        if (result == ZE_RESULT_SUCCESS && !context.verboseLogging) {
+            return result;
+        }
         std::string status = (result == ZE_RESULT_SUCCESS) ? "SUCCESS" : "ERROR";
         std::ostringstream oss;
         oss << status << " (" << loader::to_string(result) << ") in zerTranslateIdentifierToDeviceHandle(";
@@ -74,6 +86,10 @@ namespace validation_layer
     }
         VALIDATION_MAYBE_UNUSED static ze_result_t logAndPropagateResult_zerGetDefaultContext(
         ze_result_t result) {
+        // Only log success results if verbose logging is enabled
+        if (result == ZE_RESULT_SUCCESS && !context.verboseLogging) {
+            return result;
+        }
         std::string status = (result == ZE_RESULT_SUCCESS) ? "SUCCESS" : "ERROR";
         context.logger->log_trace(status + " (" + loader::to_string(result) + ") in zerGetDefaultContext()");
         return result;