Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* [PATCH 0/1] update btrace tests to test all recording methods
@ 2024-03-04  8:23 Abdul Basit Ijaz
  2024-03-04  8:23 ` [PATCH 1/1] testsuite, btrace: update btrace testsuite to test all btrace " Abdul Basit Ijaz
  0 siblings, 1 reply; 5+ messages in thread
From: Abdul Basit Ijaz @ 2024-03-04  8:23 UTC (permalink / raw)
  To: gdb-patches; +Cc: abdul.b.ijaz, markus.t.metzger, blarsen

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(-)

-- 
2.34.1

Intel Deutschland GmbH
Registered Address: Am Campeon 10, 85579 Neubiberg, Germany
Tel: +49 89 99 8853-0, www.intel.de <http://www.intel.de>
Managing Directors: Christin Eisenschmid, Sharon Heck, Tiffany Doon Silva  
Chairperson of the Supervisory Board: Nicole Lau
Registered Office: Munich
Commercial Register: Amtsgericht Muenchen HRB 186928


^ permalink raw reply	[flat|nested] 5+ messages in thread
* [PATCH 0/1] update btrace tests to test all recording methods
@ 2024-02-12 16:26 Abdul Basit Ijaz
  0 siblings, 0 replies; 5+ messages in thread
From: Abdul Basit Ijaz @ 2024-02-12 16:26 UTC (permalink / raw)
  To: gdb-patches; +Cc: abdul.b.ijaz

From: "Ijaz, Abdul B" <abdul.b.ijaz@intel.com>

Hi All,

This change updates all btrace related tests in gdb.btrace and
gdb.python/py-record-btrace*.exp.  After this change, gdb.btrace
will, instead of selecting the default recording method, run all tests
for all available and applicable methods. This increases testing
coverage.

Tests ran fine using gcc64/native-gdbserver configurations on Ubuntu22
and Fedora38.

Thanks & Best Regard,
Abdul Basit

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

 gdb/testsuite/gdb.btrace/buffer-size.exp      |  39 +-
 gdb/testsuite/gdb.btrace/data.exp             |  66 +--
 gdb/testsuite/gdb.btrace/delta.exp            | 104 +++--
 gdb/testsuite/gdb.btrace/dlopen.exp           |  39 +-
 .../gdb.btrace/enable-new-thread.exp          |  37 +-
 gdb/testsuite/gdb.btrace/enable-running.exp   |  61 +--
 gdb/testsuite/gdb.btrace/enable.exp           | 155 ++++---
 gdb/testsuite/gdb.btrace/exception.exp        | 105 +++--
 .../gdb.btrace/function_call_history.exp      | 416 +++++++++---------
 gdb/testsuite/gdb.btrace/gcore.exp            |  36 +-
 .../gdb.btrace/instruction_history.exp        | 305 ++++++-------
 gdb/testsuite/gdb.btrace/multi-inferior.exp   |  64 +--
 .../gdb.btrace/multi-thread-step.exp          | 108 ++---
 gdb/testsuite/gdb.btrace/nohist.exp           |  32 +-
 gdb/testsuite/gdb.btrace/non-stop.exp         | 307 ++++++-------
 gdb/testsuite/gdb.btrace/reconnect.exp        |  96 ++--
 gdb/testsuite/gdb.btrace/record_goto-step.exp |  42 +-
 gdb/testsuite/gdb.btrace/record_goto.exp      | 312 ++++++-------
 gdb/testsuite/gdb.btrace/rn-dl-bind.exp       |  63 +--
 gdb/testsuite/gdb.btrace/segv.exp             |  42 +-
 gdb/testsuite/gdb.btrace/step.exp             |  50 ++-
 gdb/testsuite/gdb.btrace/stepi.exp            | 207 ++++-----
 gdb/testsuite/gdb.btrace/tailcall-only.exp    |  94 ++--
 gdb/testsuite/gdb.btrace/tailcall.exp         | 138 +++---
 gdb/testsuite/gdb.btrace/tsx.exp              |   2 +-
 .../gdb.btrace/unknown_functions.exp          |  68 +--
 gdb/testsuite/gdb.btrace/vdso.exp             |  36 +-
 .../gdb.python/py-record-btrace-threads.exp   |  77 ++--
 gdb/testsuite/gdb.python/py-record-btrace.exp |   3 +-
 gdb/testsuite/lib/gdb.exp                     |  60 ++-
 30 files changed, 1710 insertions(+), 1454 deletions(-)

-- 
2.34.1

Intel Deutschland GmbH
Registered Address: Am Campeon 10, 85579 Neubiberg, Germany
Tel: +49 89 99 8853-0, www.intel.de <http://www.intel.de>
Managing Directors: Christin Eisenschmid, Sharon Heck, Tiffany Doon Silva  
Chairperson of the Supervisory Board: Nicole Lau
Registered Office: Munich
Commercial Register: Amtsgericht Muenchen HRB 186928


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2024-04-09  9:55 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-03-04  8:23 [PATCH 0/1] update btrace tests to test all recording methods Abdul Basit Ijaz
2024-03-04  8:23 ` [PATCH 1/1] testsuite, btrace: update btrace testsuite to test all btrace " Abdul Basit Ijaz
2024-03-26  9:16   ` Metzger, Markus T
2024-04-09  9:54     ` Metzger, Markus T
  -- strict thread matches above, loose matches on Subject: below --
2024-02-12 16:26 [PATCH 0/1] update btrace tests to test all " Abdul Basit Ijaz

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox