[PTISDK] fixes for iso metrics test failure with oneapi_2025_2 (#582)

Author: maaswani

sdk/samples/metrics_iso3dfd_dpcpp/src/iso3dfd_kernels.cpp

@@ -365,12 +365,12 @@ bool Iso3dfdDevice(sycl::queue &q, float *ptr_next, float *ptr_prev,
       if (i == 0 || i == (nIterations / 2)) {
         if (i == 0) { // Collect a different group for the first half of the iterations
           // TIME metric groups
-          group_name = /*"GpuOffload"*/ "ComputeBasic" /* "MemProfile" "DataportProfile" "L1ProfileReads" "L1ProfileSlmBankConflicts" "L1ProfileWrites" */;
+          group_name = "ComputeBasic"; //"GpuOffload" "ComputeBasic" "MemProfile" "DataportProfile" "L1ProfileReads"  "L1ProfileSlmBankConflicts" "L1ProfileWrites"
 
           // TRACE metric groups
           //group_name = "tpcs_utilization_and_bw" /* "nic_stms" "dcore0_bmons_bw"*/;
           group_type = /*PTI_METRIC_GROUP_TYPE_TRACE_BASED*/ PTI_METRIC_GROUP_TYPE_TIME_BASED;
-        } else { // collect a different metric group for the second hald of the iterations
+        } else { // collect a different metric group for the second half of the iterations
           // Stop the collection for the first half of the iterations
           if (MetricsProfiler::MetricsProfilerInstance().StopCollection() != true) {
             exit(-1);
@@ -388,8 +388,10 @@ bool Iso3dfdDevice(sycl::queue &q, float *ptr_next, float *ptr_prev,
           DeleteFile(lib_filename);
           DeleteFile(sample_filename);
 
-          // TIME metric groups
-          group_name = /*"GpuOffload"*/ /*"ComputeBasic"  "MemProfile" "DataportProfile"*/ "L1ProfileReads" /* "L1ProfileSlmBankConflicts" "L1ProfileWrites"*/ ;
+          // TIME metric groups for second half -- this needs to be *named* same across other architectures!
+          // Choose same group to ensure we do not fail on other architectures.
+          //group_name = "TestOa";
+          group_name = "ComputeBasic"; //"GpuOffload" "ComputeBasic" "MemProfile" "DataportProfile" "L1ProfileReads"  "L1ProfileSlmBankConflicts" "L1ProfileWrites"
 
           // TRACE metric groups
           //group_name = "tpcs_utilization_and_bw" /* "nic_stms" "dcore0_bmons_bw"*/;

sdk/src/metrics_handler.h

@@ -587,7 +587,10 @@ class PtiStreamMetricsProfiler : public PtiMetricsProfiler {
           std::ifstream(it->second->metric_file_name_, std::ios::in | std::ios::binary);
       PTI_ASSERT(inf.is_open());
       inf.seekg(0, inf.end);
-      uint32_t file_size = inf.tellg();
+      std::streamsize stream_file_size = inf.tellg();
+      PTI_ASSERT(stream_file_size >= 0);
+      size_t file_size = static_cast<size_t>(stream_file_size);
+
       inf.seekg(0, inf.beg);  // rewind
       std::vector<uint8_t> raw_metrics(file_size);
 
@@ -611,7 +614,6 @@ class PtiStreamMetricsProfiler : public PtiMetricsProfiler {
     }
 
     // Option 2: user wants the buffer filled.
-    std::vector<uint8_t> raw_metrics(PtiMetricsProfiler::GetMaxMetricBufferSize());
 
     *metrics_values_count = 0;
 
@@ -661,9 +663,14 @@ class PtiStreamMetricsProfiler : public PtiMetricsProfiler {
         // open input file stream where metrics data is saved
         std::ifstream inf =
             std::ifstream(it->second->metric_file_name_, std::ios::in | std::ios::binary);
-        if (!inf.is_open()) {
-          continue;
-        }
+        PTI_ASSERT(inf.is_open());
+        inf.seekg(0, inf.end);
+        std::streamsize stream_file_size = inf.tellg();
+        PTI_ASSERT(stream_file_size >= 0);
+        size_t file_size = static_cast<size_t>(stream_file_size);
+
+        inf.seekg(0, inf.beg);  // rewind
+        std::vector<uint8_t> raw_metrics(file_size);
 
         user_logger_->info("{\n\t\"displayTimeUnit\": \"us\",\n\t\"traceEvents\": [");
 
@@ -1162,7 +1169,10 @@ class PtiTraceMetricsProfiler : public PtiMetricsProfiler {
     // Option 1: user wants metrics values count
     if (metrics_values_buffer == nullptr) {
       inf.seekg(0, inf.end);
-      uint32_t file_size = inf.tellg();
+      std::streamsize stream_file_size = inf.tellg();
+      PTI_ASSERT(stream_file_size >= 0);
+      size_t file_size = static_cast<size_t>(stream_file_size);
+
       inf.seekg(0, inf.beg);  // rewind
       std::vector<uint8_t> raw_metrics(file_size);