trace: Workaround GCC bug compiling with old systemtap #33310

pull luke-jr wants to merge 1 commits into bitcoin:master from luke-jr:wrkarnd_gcc_systemtap_ice changing 2 files +11 −0
  1. luke-jr commented at 7:26 PM on September 4, 2025: contributor

    No description provided.

  2. trace: Workaround GCC bug compiling with old systemtap 93a29ff283
  3. DrahtBot commented at 7:27 PM on September 4, 2025: contributor

    <!--e57a25ab6845829454e8d69fc972939a-->

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    <!--006a51241073e994b41acfe9ec718e94-->

    Code Coverage & Benchmarks

    For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/33310.

    <!--021abf342d371248e50ceaed478a90ca-->

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK 0xB10C

    If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

    <!--5faf32d7da4f0f540f40219e4f7537a3-->

  4. fanquake commented at 11:18 AM on September 5, 2025: member

    Recreated the issue using 4.6:

            Run Build Command(s): /usr/bin/cmake -E env VERBOSE=1 /bin/gmake -f Makefile cmTC_5ba56/fast
            /bin/gmake  -f CMakeFiles/cmTC_5ba56.dir/build.make CMakeFiles/cmTC_5ba56.dir/build
            gmake[1]: Entering directory '/bitcoin/build/CMakeFiles/CMakeScratch/TryCompile-goOPiQ'
            Building CXX object CMakeFiles/cmTC_5ba56.dir/src.cxx.o
            /bin/arm-linux-gnueabihf-g++ -DHAVE_USDT_H -I/bitcoin/depends/arm-linux-gnueabihf/include -O2 -O2 -g -std=c++20 -o CMakeFiles/cmTC_5ba56.dir/src.cxx.o -c /bitcoin/build/CMakeFiles/CMakeScratch/TryCompile-goOPiQ/src.cxx
            during RTL pass: mach
            /bitcoin/build/CMakeFiles/CMakeScratch/TryCompile-goOPiQ/src.cxx: In function 'int main()':
            /bitcoin/build/CMakeFiles/CMakeScratch/TryCompile-goOPiQ/src.cxx:12:5: internal compiler error: in create_fix_barrier, at config/arm/arm.cc:18266
               12 |     }
                  |     ^
            0x1a0339f internal_error(char const*, ...)
            	???:0
            0x677e6b fancy_abort(char const*, int, char const*)
            	???:0
            Please submit a full bug report, with preprocessed source (by using -freport-bug).
            Please include the complete backtrace with any bug report.
            See <file:///usr/share/doc/gcc-14/README.Bugs> for instructions.
            gmake[1]: *** [CMakeFiles/cmTC_5ba56.dir/build.make:81: CMakeFiles/cmTC_5ba56.dir/src.cxx.o] Error 1
            gmake[1]: Leaving directory '/bitcoin/build/CMakeFiles/CMakeScratch/TryCompile-goOPiQ'
            gmake: *** [Makefile:134: cmTC_5ba56/fast] Error 2
            
          exitCode: 2
    
  5. 0xB10C commented at 7:13 AM on September 12, 2025: contributor

    lgtm ACK 93a29ff2830162c8129d35c7b9beb43fab984503 - I did not test this.

    The change in Systemtap can be found here: https://sourceware.org/git/?p=systemtap.git;a=blobdiff;f=includes/sys/sdt.h;h=28d236d91cd8e4d3a5db8b13cac90fb678c629cf;hp=9ecb1cb6fed2bbb9f58aaca640c5840e471b67ac;hb=34facf7ee6b43dae66cc109973a4eda42e439163;hpb=2c11a115bdbe41a7f064ccd32b41c76c5b7c516d

    Including this makes sense as there are likely some OSs (e.g. Ubuntu 22.04) still shipping <systemtap-4.7.

    Depends builds use systemtap 4.8 at the moment: https://github.com/bitcoin/bitcoin/blob/b81445333a10c50e2f74b6629a7b41542fcbd4a5/depends/packages/systemtap.mk#L2

  6. fanquake merged this on Sep 12, 2025
  7. fanquake closed this on Sep 12, 2025

  8. fanquake referenced this in commit 35038b03c9 on Sep 12, 2025
  9. fanquake commented at 10:51 AM on September 12, 2025: member

    Backported to 30.x in #33356.

  10. fanquake referenced this in commit e97588fc3d on Sep 12, 2025
  11. fanquake commented at 10:53 AM on September 12, 2025: member

    Backported to 29.x in #33344.

  12. alexanderwiederin referenced this in commit 49e068b15b on Sep 16, 2025
  13. alexanderwiederin referenced this in commit 4b0c2f2a8f on Sep 17, 2025
  14. alexanderwiederin referenced this in commit 2edb618ffe on Sep 17, 2025
  15. glozow referenced this in commit 7e1eca4882 on Sep 17, 2025
  16. glozow referenced this in commit b7a722724d on Sep 17, 2025
  17. stringintech referenced this in commit fb8510ba20 on Sep 17, 2025
  18. bug-castercv502 referenced this in commit 9f5d12d027 on Sep 28, 2025
  19. yuvicc referenced this in commit d89c6d0002 on Sep 28, 2025
  20. stickies-v referenced this in commit a785fe6759 on Nov 5, 2025
  21. morozow referenced this in commit 6b1d3da174 on May 8, 2026
  22. morozow referenced this in commit 48f5d00b2b on May 8, 2026
  23. morozow referenced this in commit d5f9992f15 on May 8, 2026
  24. morozow referenced this in commit 3848de1a09 on May 8, 2026
  25. Kino1994 referenced this in commit aa0a609348 on Jun 28, 2026
  26. BigcoinBGC referenced this in commit 3b8d4ce8ea on Jun 30, 2026
  27. Kino1994 referenced this in commit 2218aa847c on Aug 19, 2026
  28. bitcoin locked this on Sep 12, 2026

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-09-25 01:51 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me