Skip to content

Commit

Permalink
[gdb/testsuite] Improve probe detection in gdb.base/break-probes.exp
Browse files Browse the repository at this point in the history
When running test-case gdb.base/break-probes.exp on ubuntu 18.04.5, we have:
...
 (gdb) run^M
 Starting program: break-probes^M
 Stopped due to shared library event (no libraries added or removed)^M
 (gdb) bt^M
 #0  0x00007ffff7dd6e12 in ?? () from /lib64/ld-linux-x86-64.so.2^M
 #1  0x00007ffff7dedf50 in ?? () from /lib64/ld-linux-x86-64.so.2^M
 #2  0x00007ffff7dd5128 in ?? () from /lib64/ld-linux-x86-64.so.2^M
 #3  0x00007ffff7dd4098 in ?? () from /lib64/ld-linux-x86-64.so.2^M
 #4  0x0000000000000001 in ?? ()^M
 #5  0x00007fffffffdaac in ?? ()^M
 #6  0x0000000000000000 in ?? ()^M
 (gdb) UNSUPPORTED: gdb.base/break-probes.exp: probes not present on this system
...

Using the backtrace, the test-case tries to establish that we're stopped in
dl_main, which is used as proof that we're using probes.

However, the backtrace only shows an address, because:
- the dynamic linker contains no minimal symbols and no debug info, and
- gdb is build without --with-separate-debug-dir so it can't find the
  corresponding .debug file, which does contain the mimimal symbols and
  debug info.

Fix this by instead printing the pc and grepping for the value in the
info probes output:
...
(gdb) p /x $pc^M
$1 = 0x7ffff7dd6e12^M
(gdb) info probes^M
Type Provider Name           Where              Object                      ^M
  ...
stap rtld     init_start     0x00007ffff7dd6e12 /lib64/ld-linux-x86-64.so.2 ^M
  ...
(gdb)
...

Tested on x86_64-linux.
  • Loading branch information
vries committed Sep 23, 2021
1 parent 108e608 commit d264c39
Showing 1 changed file with 19 additions and 13 deletions.
32 changes: 19 additions & 13 deletions gdb/testsuite/gdb.base/break-probes.exp
Original file line number Diff line number Diff line change
Expand Up @@ -23,13 +23,6 @@ set libname $testfile-solib
set srcfile_lib $srcdir/$subdir/$libname.c
set binfile_lib [standard_output_file $libname.so]

if { [istarget "*bsd*"] } {
set normal_bp "r_debug_state"
} else {
set normal_bp "_dl_debug_state"
}
set probes_bp "dl_main"

if { [gdb_compile_shlib $srcfile_lib $binfile_lib {}] != "" } {
untested "failed to compile shared library"
return -1
Expand All @@ -47,14 +40,27 @@ gdb_test_no_output "set stop-on-solib-events 1"
gdb_run_cmd
gdb_test "" ".*Stopped due to shared library event.*"

# XFAIL if we are not using probes
set test "ensure using probes"
# Check if we're using probes.
set using_probes 0
gdb_test_multiple "bt" $test {
-re "#0 +\[^\r\n\]*\\m(__GI_)?$normal_bp\\M.*$gdb_prompt $" {

# Get PC.
set pc ""
gdb_test_multiple "p /x \$pc" "" {
-re -wrap " = ($hex)" {
set pc $expect_out(1,string)
}
-re -wrap "" {
}
-re "#0 +\[^\r\n\]*\\m(__GI_)?$probes_bp\\M.*$gdb_prompt $" {
pass $test
}
if { $pc == "" } {
unsupported "Couldn't get $pc"
return -1
}
regsub "0x0*" $pc "" pc

# Verify that pc is at info_start probe address.
gdb_test_multiple "info probes stap rtld" "" {
-re -wrap "init_start +0x0*$pc .*" {
set using_probes 1
}
-re -wrap "" {
Expand Down

0 comments on commit d264c39

Please sign in to comment.