mbox series

[0/1] update btrace tests to test all recording methods

Message ID 20240304082354.3336-1-abdul.b.ijaz@intel.com
Headers
Series update btrace tests to test all recording methods |

Message

Ijaz, Abdul B March 4, 2024, 8:23 a.m. UTC 
  From: "Ijaz, Abdul B" <abdul.b.ijaz@intel.com>

Fixes the feedback from Larsen and Markus in V2 patch.

Changes since V1:

1) Keep the function name allow_btrace_tests for checking if btrace tests
are allowed.
2) Update the allow_btrace_tests function to test both btrace recording
methods support and return 0 if both of them are not supported.
3) Remove unnecessary changes  to add "allow_btrace_pt_tests" calls for
btrace related tests in gdb.python folder.
4) Since the function body of "allow_btrace_pt_tests" and
"allow_btrace_pt_tests" is similar so caching functions are still kept
but the function body is copied to common function as per the feedback.

The feedback regaring gdb.btrace/tsx.exp test is not updated because of
the reason mentioned here:
https://sourceware.org/pipermail/gdb-patches/2024-February/206617.html

V1 patch can be found here:
https://sourceware.org/pipermail/gdb-patches/2024-February/206529.html

Thanks & Best Regards,
Abdul Basit

Ijaz, Abdul B (1):
  testsuite, btrace: update btrace testsuite to test all btrace
    recording methods

 gdb/testsuite/gdb.btrace/buffer-size.exp      |  37 +-
 gdb/testsuite/gdb.btrace/data.exp             |  64 +--
 gdb/testsuite/gdb.btrace/delta.exp            | 102 +++--
 gdb/testsuite/gdb.btrace/dlopen.exp           |  37 +-
 .../gdb.btrace/enable-new-thread.exp          |  35 +-
 gdb/testsuite/gdb.btrace/enable-running.exp   |  59 +--
 gdb/testsuite/gdb.btrace/enable.exp           | 153 ++++---
 gdb/testsuite/gdb.btrace/exception.exp        | 103 +++--
 .../gdb.btrace/function_call_history.exp      | 414 +++++++++---------
 gdb/testsuite/gdb.btrace/gcore.exp            |  34 +-
 .../gdb.btrace/instruction_history.exp        | 303 ++++++-------
 gdb/testsuite/gdb.btrace/multi-inferior.exp   |  62 +--
 .../gdb.btrace/multi-thread-step.exp          | 106 ++---
 gdb/testsuite/gdb.btrace/nohist.exp           |  30 +-
 gdb/testsuite/gdb.btrace/non-stop.exp         | 305 ++++++-------
 gdb/testsuite/gdb.btrace/reconnect.exp        |  94 ++--
 gdb/testsuite/gdb.btrace/record_goto-step.exp |  39 +-
 gdb/testsuite/gdb.btrace/record_goto.exp      | 310 ++++++-------
 gdb/testsuite/gdb.btrace/rn-dl-bind.exp       |  61 +--
 gdb/testsuite/gdb.btrace/segv.exp             |  40 +-
 gdb/testsuite/gdb.btrace/step.exp             |  48 +-
 gdb/testsuite/gdb.btrace/stepi.exp            | 205 ++++-----
 gdb/testsuite/gdb.btrace/tailcall-only.exp    |  92 ++--
 gdb/testsuite/gdb.btrace/tailcall.exp         | 133 +++---
 gdb/testsuite/gdb.btrace/tsx.exp              |   2 +-
 .../gdb.btrace/unknown_functions.exp          |  66 +--
 gdb/testsuite/gdb.btrace/vdso.exp             |  34 +-
 .../gdb.python/py-record-btrace-threads.exp   |  75 ++--
 gdb/testsuite/lib/gdb.exp                     | 124 +++---
 29 files changed, 1689 insertions(+), 1478 deletions(-)