Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package opae for openSUSE:Factory checked in 
at 2023-03-29 23:28:00
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/opae (Old)
 and      /work/SRC/openSUSE:Factory/.opae.new.31432 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "opae"

Wed Mar 29 23:28:00 2023 rev:13 rq:1075166 version:2.0.0

Changes:
--------
--- /work/SRC/openSUSE:Factory/opae/opae.changes        2022-06-24 
08:45:36.319158988 +0200
+++ /work/SRC/openSUSE:Factory/.opae.new.31432/opae.changes     2023-03-29 
23:28:01.403721002 +0200
@@ -1,0 +2,5 @@
+Wed Mar 29 08:32:24 UTC 2023 - Nicolas Morey <[email protected]>
+
+- Add Gcc-13-2858.patch for GCC13 support
+
+-------------------------------------------------------------------

New:
----
  Gcc-13-2858.patch

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

Other differences:
------------------
++++++ opae.spec ++++++
--- /var/tmp/diff_new_pack.e1boqO/_old  2023-03-29 23:28:03.159730386 +0200
+++ /var/tmp/diff_new_pack.e1boqO/_new  2023-03-29 23:28:03.163730407 +0200
@@ -1,7 +1,7 @@
 #
 # spec file for package opae
 #
-# Copyright (c) 2022 SUSE LLC
+# Copyright (c) 2023 SUSE LLC
 #
 # All modifications and additions to the file contributed by third parties
 # remain the property of their copyright owners, unless otherwise agreed
@@ -43,6 +43,7 @@
 Patch4:         opae-libs-xfpga-fix-strnlen-argument.patch
 Patch5:         opae-fix-support-with-newer-spdlog.patch
 Patch6:         opae-disable-FORTIFY_SOURCE.patch
+Patch7:         Gcc-13-2858.patch
 BuildRoot:      %{_tmppath}/%{name}-%{version}-build
 BuildRequires:  boost-devel
 BuildRequires:  cli11-devel
@@ -153,6 +154,7 @@
 %patch4
 %patch5
 %patch6
+%patch7
 
 %build
 export RPM_OPT_FLAGS

++++++ Gcc-13-2858.patch ++++++
commit 207c9daa881a62bdade6e4c59d59c669e8c39de2
Author: trixirt <[email protected]>
Date:   Fri Mar 3 10:01:18 2023 -0800

    Gcc 13 (#2858)
    
    * include cstdint
    
    On fedora rawhide with gcc 13, there is this error
    
    In file included from tmp/opae-sdk/libraries/c++utils/cmd_handler.cpp:26:
    opae-sdk/libraries/c++utils/cmd_handler.h:62:27: error: ‘uint16_t’ has 
not been declared
       62 |                           uint16_t arg_count,
          |                           ^~~~~~~~
    opae-sdk/libraries/c++utils/cmd_handler.h:133:45: error: ‘uint16_t’ was 
not declared in this scope
      133 |     typedef std::map<std::string, std::pair<uint16_t, std::string>> 
cmd_help_map_t;
          |                                             ^~~~~~~~
    opae-sdk/libraries/c++utils/cmd_handler.h:33:1: note: ‘uint16_t’ is 
defined in header ‘<cstdint>’; did you forget to ‘#include <cstdint>’?
       32 | #include <fstream>
      +++ |+#include <cstdint>
    
    So include cstdint.
    
    Signed-off-by: Tom Rix <[email protected]>
    
    * silence warning on may uninitialized variable first_sw_desc
    
    On fedora rawhide with gcc 13, there is this error
    opae-sdk/binaries/fpgabist/dma/fpga_dma.cpp: In function ‘void* 
dispatcherWorker(void*)’:
    opae-sdk/binaries/fpgabist/dma/fpga_dma.cpp:507:48: error: 
‘first_sw_desc’ may be used uninitialized [-Werror=maybe-uninitialized]
      507 |                                 
first_sw_desc->hw_descp->hw_desc->owned_by_hw = 1;
          |                                 ~~~~~~~~~~~~~~~^~~~~~~~
    opae-sdk/binaries/fpgabist/dma/fpga_dma.cpp:440:27: note: 
‘first_sw_desc’ was declared here
      440 |         msgdma_sw_desc_t *first_sw_desc;
          |                           ^~~~~~~~~~~~~
    
    Simplify the flow so the compiler does not get confused.
    
    Signed-off-by: Tom Rix <[email protected]>
    
    * Update cmd_handler.h
    
    ---------
    
    Signed-off-by: Tom Rix <[email protected]>
    Co-authored-by: Tim Whisonant <[email protected]>

diff --git tools/extra/c++utils/cmd_handler.h tools/extra/c++utils/cmd_handler.h
index 7e7e166adbca..e616095e383a 100644
--- tools/extra/c++utils/cmd_handler.h
+++ tools/extra/c++utils/cmd_handler.h
@@ -1,4 +1,4 @@
-// Copyright(c) 2017, Intel Corporation
+// Copyright(c) 2017-2023, Intel Corporation
 //
 // Redistribution  and  use  in source  and  binary  forms,  with  or  without
 // modification, are permitted provided that the following conditions are met:
@@ -30,6 +30,7 @@
 #include <string>
 #include <iostream>
 #include <fstream>
+#include <cstdint>
 
 namespace intel
 {
diff --git tools/extra/fpgabist/dma/fpga_dma.cpp 
tools/extra/fpgabist/dma/fpga_dma.cpp
index c337efea1884..40b388a1c57b 100644
--- tools/extra/fpgabist/dma/fpga_dma.cpp
+++ tools/extra/fpgabist/dma/fpga_dma.cpp
@@ -475,8 +475,7 @@ static void *dispatcherWorker(void* dma_handle) {
 
                        // make a note of the first block descriptor
                        // mark it valid only after packing rest of the block
-                       if (desc_count == 1)
-                               first_sw_desc = sw_desc[desc_count];
+                       first_sw_desc = sw_desc[1];
                        is_owned_by_hw = (desc_count == 1)  ? false:true;
 
                        // refer prefetcher spec

Reply via email to