Bug 1781471 - systemtap tapscrip probes that use stack trace functionality fail to compile
Summary: systemtap tapscrip probes that use stack trace functionality fail to compile
Keywords:
Status: CLOSED EOL
Alias: None
Product: Fedora
Classification: Fedora
Component: systemtap
Version: 31
Hardware: All
OS: Linux
unspecified
low
Target Milestone: ---
Assignee: Frank Ch. Eigler
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2019-12-10 05:30 UTC by Craig Ringer
Modified: 2020-11-24 16:11 UTC (History)
10 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2020-11-24 16:11:56 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Sourceware 24923 0 P2 NEW perf_event_create_kernel_counter changed on kernel 5.2 2020-11-10 18:32:50 UTC
Sourceware 25265 0 P2 RESOLVED tapscripts using ustack, ubacktrace etc fail to compile on kernel 5.3 2020-11-10 18:33:09 UTC

Description Craig Ringer 2019-12-10 05:30:09 UTC
On Fedora 31 with the stock systemtap-4.2-1.fc31.x86_64 (stap 4.2/0.177) and kernel-5.3.14-300.fc31.x86_64 probes that attempt to use the bundled tapset functions `ustack`, `ubacktrace`, `print_ubacktrace` etc will fail to compile with:

```
[craig@linux pglogical3-master]$ sudo stap -v dev-tools/stap/backtrace_anywhere.stp $HOME/pg/pg11/ 'unrecognized conflict resolver'
Pass 1: parsed user script and 490 library scripts using 391688virt/165032res/9628shr/155604data kb, in 410usr/40sys/459real ms.
Pass 2: analyzed script: 2 probes, 7 functions, 1 embed, 0 globals using 407908virt/182836res/10772shr/171824data kb, in 160usr/20sys/217real ms.
Pass 3: translated to C into "/tmp/stapFp0X4O/stap_36181c9d60f549fc3d570f1980a16330_4085_src.c" using 407908virt/182968res/10900shr/171824data kb, in 100usr/290sys/562real ms.
In file included from /tmp/stapFp0X4O/stap_36181c9d60f549fc3d570f1980a16330_4085_src.c:159:
/usr/share/systemtap/runtime/stack.c:66:14: error: ‘struct stack_trace’ declared inside parameter list will not be visible outside of this definition or declaration [-Werror]
   66 |       struct stack_trace *trace);
      |              ^~~~~~~~~~~
/usr/share/systemtap/runtime/stack.c: In function ‘_stp_stack_print_fallback’:
/usr/share/systemtap/runtime/stack.c:202:21: error: storage size of ‘trace’ isn’t known
  202 |  struct stack_trace trace;
      |                     ^~~~~
In file included from ./include/asm-generic/bug.h:5,
                 from ./arch/x86/include/asm/bug.h:83,
                 from ./include/linux/bug.h:5,
                 from ./include/linux/mmdebug.h:5,
                 from ./include/linux/gfp.h:5,
                 from /usr/share/systemtap/runtime/linux/runtime_defines.h:20,
                 from /usr/share/systemtap/runtime/runtime_defines.h:8,
                 from /tmp/stapFp0X4O/stap_36181c9d60f549fc3d570f1980a16330_4085_src.c:11:
/usr/share/systemtap/runtime/unwind/unwind.h: In function ‘read_ptr_sect’:
./include/linux/compiler.h:328:5: error: this statement may fall through [-Werror=implicit-fallthrough=]
  328 |  do {        \
      |     ^
./include/linux/compiler.h:338:2: note: in expansion of macro ‘__compiletime_assert’
  338 |  __compiletime_assert(condition, msg, prefix, suffix)
      |  ^~~~~~~~~~~~~~~~~~~~
./include/linux/compiler.h:350:2: note: in expansion of macro ‘_compiletime_assert’
  350 |  _compiletime_assert(condition, msg, __compiletime_assert_, __LINE__)
      |  ^~~~~~~~~~~~~~~~~~~
./include/linux/build_bug.h:39:37: note: in expansion of macro ‘compiletime_assert’
   39 | #define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg)
      |                                     ^~~~~~~~~~~~~~~~~~
./include/linux/build_bug.h:50:2: note: in expansion of macro ‘BUILD_BUG_ON_MSG’
   50 |  BUILD_BUG_ON_MSG(condition, "BUILD_BUG_ON failed: " #condition)
      |  ^~~~~~~~~~~~~~~~
/usr/share/systemtap/runtime/unwind/unwind.h:159:3: note: in expansion of macro ‘BUILD_BUG_ON’
  159 |   BUILD_BUG_ON(sizeof(u64) != sizeof(value));
      |   ^~~~~~~~~~~~
In file included from /usr/share/systemtap/runtime/unwind.c:16,
                 from /usr/share/systemtap/runtime/linux/runtime.h:255,
                 from /usr/share/systemtap/runtime/runtime.h:26,
                 from /tmp/stapFp0X4O/stap_36181c9d60f549fc3d570f1980a16330_4085_src.c:27:
/usr/share/systemtap/runtime/unwind/unwind.h:164:2: note: here
  164 |  case DW_EH_PE_absptr:
      |  ^~~~
cc1: all warnings being treated as errors
```

e.g.

```
sudo stap -v -e 'probe process("/lib64/libc.so.6").function("fsync") { print_ubacktrace(); }'
```

See also https://sourceware.org/bugzilla/show_bug.cgi?id=24923

Comment 1 Craig Ringer 2019-12-10 05:43:24 UTC
Opened new upstream issue as https://sourceware.org/bugzilla/show_bug.cgi?id=25265

Comment 2 Craig Ringer 2019-12-11 03:24:40 UTC
I've supplied a patch upstream for one issue. I also found a second one https://sourceware.org/bugzilla/show_bug.cgi?id=25267 that arises from a similar cause.

Comment 3 Ben Cotton 2020-11-03 17:14:32 UTC
This message is a reminder that Fedora 31 is nearing its end of life.
Fedora will stop maintaining and issuing updates for Fedora 31 on 2020-11-24.
It is Fedora's policy to close all bug reports from releases that are no longer
maintained. At that time this bug will be closed as EOL if it remains open with a
Fedora 'version' of '31'.

Package Maintainer: If you wish for this bug to remain open because you
plan to fix it in a currently maintained version, simply change the 'version' 
to a later Fedora version.

Thank you for reporting this issue and we are sorry that we were not 
able to fix it before Fedora 31 is end of life. If you would still like 
to see this bug fixed and are able to reproduce it against a later version 
of Fedora, you are encouraged  change the 'version' to a later Fedora 
version prior this bug is closed as described in the policy above.

Although we aim to fix as many bugs as possible during every release's 
lifetime, sometimes those efforts are overtaken by events. Often a 
more recent Fedora release includes newer upstream software that fixes 
bugs or makes them obsolete.

Comment 4 Ben Cotton 2020-11-24 16:11:56 UTC
Fedora 31 changed to end-of-life (EOL) status on 2020-11-24. Fedora 31 is
no longer maintained, which means that it will not receive any further
security or bug fix updates. As a result we are closing this bug.

If you can reproduce this bug against a currently maintained version of
Fedora please feel free to reopen this bug against that version. If you
are unable to reopen this bug, please file a new report against the
current release. If you experience problems, please add a comment to this
bug.

Thank you for reporting this bug and we are sorry it could not be fixed.


Note You need to log in before you can comment on or make changes to this bug.