Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package gpuvis for openSUSE:Factory checked 
in at 2021-12-10 21:52:56
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/gpuvis (Old)
 and      /work/SRC/openSUSE:Factory/.gpuvis.new.2520 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "gpuvis"

Fri Dec 10 21:52:56 2021 rev:2 rq:939150 version:20211124

Changes:
--------
--- /work/SRC/openSUSE:Factory/gpuvis/gpuvis.changes    2021-12-08 
22:09:23.802874993 +0100
+++ /work/SRC/openSUSE:Factory/.gpuvis.new.2520/gpuvis.changes  2021-12-10 
21:53:30.066922180 +0100
@@ -1,0 +2,12 @@
+Fri Dec 10 10:41:06 UTC 2021 - Patrik Jakobsson <patrik.jakobs...@suse.com>
+
+- Update to version 20211204 (7f47419470687c7ecbdf086b81f5bafdb05d1bef) 
+  * Look at all buffers when calculating min_file_ts
+
+-------------------------------------------------------------------
+Fri Dec 10 09:48:27 UTC 2021 - Patrik Jakobsson <patrik.jakobs...@suse.com>
+
+- Only build with intel-gpu-tools-devel on suse_version > 1530
+- Only build with i915_perf=true on x86*
+
+-------------------------------------------------------------------

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ gpuvis.spec ++++++
--- /var/tmp/diff_new_pack.HatYX9/_old  2021-12-10 21:53:30.658922443 +0100
+++ /var/tmp/diff_new_pack.HatYX9/_new  2021-12-10 21:53:30.662922444 +0100
@@ -1,5 +1,5 @@
 #
-# spec file for package renderdoc
+# spec file for package gpuvis
 #
 # Copyright (c) 2021 SUSE LLC
 #
@@ -15,7 +15,8 @@
 # Please submit bugfixes or comments via https://bugs.opensuse.org/
 #
 
-%define last_commit ff96f19529021991e6cbcc81f026bca658897bd8
+
+%define last_commit 7f47419470687c7ecbdf086b81f5bafdb05d1bef
 %define rapidjson_commit 1c2c8e085a8b2561dff17bedb689d2eb0609b689
 
 Name:           gpuvis
@@ -27,13 +28,17 @@
 URL:            https://github.com/mikesart/gpuvis
 Source0:        
https://github.com/mikesart/gpuvis/archive/%{last_commit}/gpuvis.tar.gz
 Source1:        
https://github.com/Tencent/rapidjson/archive/%{rapidjson_commit}/rapidjson.tar.gz
-BuildRequires:  meson
-BuildRequires:  ninja
-BuildRequires:  gcc-c++
-BuildRequires:  libSDL2-devel
 BuildRequires:  freetype2-devel
+BuildRequires:  gcc-c++
 BuildRequires:  gtk3-devel
+BuildRequires:  libSDL2-devel
+BuildRequires:  meson
+BuildRequires:  ninja
+%ifarch %{ix86} x86_64
+%if 0%{?suse_version} > 1530
 BuildRequires:  intel-gpu-tools-devel
+%endif
+%endif
 
 %description
 Gpuvis is a Linux GPU profiler similar to GPUView on Windows. It is designed 
to work with trace-cmd captures and help track down Linux gpu and application 
performance issues.
@@ -45,7 +50,12 @@
 
 %build
 %meson \
+%ifarch %{ix86} x86_64
+%if 0%{?suse_version} > 1530
   -Duse_i915_perf=true
+%endif
+%endif
+
 %meson_build
 
 %install

++++++ gpuvis.tar.gz ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/gpuvis-ff96f19529021991e6cbcc81f026bca658897bd8/src/trace-cmd/trace-read.cpp
 
new/gpuvis-7f47419470687c7ecbdf086b81f5bafdb05d1bef/src/trace-cmd/trace-read.cpp
--- 
old/gpuvis-ff96f19529021991e6cbcc81f026bca658897bd8/src/trace-cmd/trace-read.cpp
    2021-11-16 21:15:11.000000000 +0100
+++ 
new/gpuvis-7f47419470687c7ecbdf086b81f5bafdb05d1bef/src/trace-cmd/trace-read.cpp
    2021-12-04 00:56:38.000000000 +0100
@@ -1961,17 +1961,23 @@
     // Find the lowest ts value in the trace file
     for ( size_t cpu = 0; cpu < ( size_t )handle->cpus; cpu++ )
     {
-        pevent_record_t *record = tracecmd_peek_data( handle, cpu );
+        for ( file_info_t *file_info : file_list )
+        {
+            pevent_record_t *record = tracecmd_peek_data( file_info->handle, 
cpu );
 
-        if ( record )
-            trace_info.min_file_ts = std::min< int64_t >( 
trace_info.min_file_ts, record->ts );
+            if ( record )
+                trace_info.min_file_ts = std::min< int64_t >( 
trace_info.min_file_ts, record->ts );
+        }
     }
 
+    // Should never happen, but sure is bad if it ever does as our min_ts is 
set to INT64 MAX
+    if (trace_info.min_file_ts == INT64_MAX)
+        trace_info.min_file_ts = 0;
+
     trace_info.cpu_info.resize( handle->cpus );
     for ( size_t cpu = 0; cpu < ( size_t )handle->cpus; cpu++ )
     {
         cpu_info_t &cpu_info = trace_info.cpu_info[ cpu ];
-        pevent_record_t *record = tracecmd_peek_data( handle, cpu );
 
         cpu_info.file_offset = handle->cpu_data[ cpu ].file_offset;
         cpu_info.file_size = handle->cpu_data[ cpu ].file_size;
@@ -1995,12 +2001,17 @@
                 cpu_info.now_ts -= trace_info.min_file_ts;
         }
 
-        if ( record )
+        for ( file_info_t *file_info : file_list )
         {
-            cpu_info.min_ts = record->ts - trace_info.min_file_ts;
+            pevent_record_t *record = tracecmd_peek_data( file_info->handle, 
cpu );
 
-            if ( cpu_info.overrun && trace_info.trim_trace )
-                trim_ts = std::max< unsigned long long >( trim_ts, record->ts 
);
+            if ( record )
+            {
+                cpu_info.min_ts = record->ts - trace_info.min_file_ts;
+
+                if ( cpu_info.overrun && trace_info.trim_trace )
+                    trim_ts = std::max< unsigned long long >( trim_ts, 
record->ts );
+            }
         }
     }
 

Reply via email to