fix failures

Author: alexbatashev

sycl/source/detail/xpti_registry.hpp

@@ -34,10 +34,7 @@ class XPTIRegistry {
   void initializeFrameworkOnce() {
 #ifdef XPTI_ENABLE_INSTRUMENTATION
     if (!MInitialized) {
-      xpti::result_t result = xptiFrameworkInitialize();
-      if (result != xpti::result_t::XPTI_RESULT_SUCCESS) {
-        throw std::runtime_error("Failed to initialize XPTI");
-      }
+      xptiFrameworkInitialize();
       MInitialized = true;
     }
 #endif

xpti/include/xpti_trace_framework.h

@@ -34,8 +34,19 @@
 
 extern "C" {
 
-XPTI_EXPORT_API xpti::result_t xptiFrameworkInitialize();
-XPTI_EXPORT_API xpti::result_t xptiFrameworkFinalize();
+/// @brief Initializes XPTI framework.
+/// @details Initialize XPTI framework resources. Each user of XPTI must call
+/// this function prior to any other XPTI API call. It is framework's
+/// responsibility to ensure that resources are initialized once. Each call to
+/// this function must have corresponding call to xptiFrameworkFinalize() to
+/// ensure resources are freed.
+XPTI_EXPORT_API void xptiFrameworkInitialize();
+
+/// @brief Deinitializes XPTI framework.
+/// @details Call to this function decrements framework's internal reference
+/// counter. Once its value is equal to zero, XPTI framework can release
+/// resources and unload subscribers.
+XPTI_EXPORT_API void xptiFrameworkFinalize();
 
 /// @brief Initialization function that is called when a new stream is generated
 /// @details When a runtime or application that uses XPTI instrumentation API

xpti/src/xpti_proxy.cpp

@@ -140,32 +140,26 @@ class ProxyLoader {
 };
 } // namespace xpti
 
-XPTI_EXPORT_API xpti::result_t xptiFrameworkInitialize() {
+XPTI_EXPORT_API void xptiFrameworkInitialize() {
   if (xpti::ProxyLoader::instance().noErrors()) {
     void *f = xpti::ProxyLoader::instance().functionByIndex(
         XPTI_FRAMEWORK_INITIALIZE);
     if (f) {
-      return (*reinterpret_cast<xpti_framework_initialize_t>(f))();
+      (*reinterpret_cast<xpti_framework_initialize_t>(f))();
     }
   }
-
-  return xpti::result_t::XPTI_RESULT_FAIL;
 }
 
-XPTI_EXPORT_API xpti::result_t xptiFrameworkFinalize() {
-  xpti::result_t result = xpti::result_t::XPTI_RESULT_FAIL;
-
+XPTI_EXPORT_API void xptiFrameworkFinalize() {
   if (xpti::ProxyLoader::instance().noErrors()) {
     void *f = xpti::ProxyLoader::instance().functionByIndex(
         XPTI_FRAMEWORK_INITIALIZE);
     if (f) {
-      result = (*reinterpret_cast<xpti_framework_initialize_t>(f))();
+      (*reinterpret_cast<xpti_framework_initialize_t>(f))();
     }
   }
 
   delete &xpti::ProxyLoader::instance();
-
-  return result;
 }
 
 XPTI_EXPORT_API uint16_t xptiRegisterUserDefinedTracePoint(

xptifw/src/xpti_trace_framework.cpp

@@ -1004,21 +1004,18 @@ static int GFrameworkReferenceCounter = 0;
 
 extern "C" {
 
-XPTI_EXPORT_API xpti::result_t xptiFrameworkInitialize() {
+XPTI_EXPORT_API void xptiFrameworkInitialize() {
   std::lock_guard<xpti::utils::SpinLock> guard{xpti::g_framework_mutex};
   xpti::GFrameworkReferenceCounter++;
-  return xpti::result_t::XPTI_RESULT_SUCCESS;
 }
 
-XPTI_EXPORT_API xpti::result_t xptiFrameworkFinalize() {
+XPTI_EXPORT_API void xptiFrameworkFinalize() {
   std::lock_guard<xpti::utils::SpinLock> guard{xpti::g_framework_mutex};
 
   xpti::GFrameworkReferenceCounter--;
   if (xpti::GFrameworkReferenceCounter == 0) {
     delete &xpti::Framework::instance();
   }
-
-  return xpti::result_t::XPTI_RESULT_SUCCESS;
 }
 
 XPTI_EXPORT_API uint16_t