mbox series

[v7,0/6] gdb: refine scheduler locking settings

Message ID 20260325134328.550990-1-klaus.gerlicher@intel.com
Headers
Series gdb: refine scheduler locking settings |

Message

Klaus Gerlicher March 25, 2026, 1:43 p.m. UTC 
  Hi,

This is version 7 of the patch series to refine scheduler locking
settings in GDB.

Previous versions:
  V6: https://inbox.sourceware.org/gdb-patches/20260204133753.547281-1-klaus.gerlicher@intel.com/
  V5: https://inbox.sourceware.org/gdb-patches/20260129095918.2216661-1-klaus.gerlicher@intel.com/
  V4: https://inbox.sourceware.org/gdb-patches/20251205135851.2581702-1-klaus.gerlicher@intel.com/
  V3: https://inbox.sourceware.org/gdb-patches/20250811132321.486911-1-klaus.gerlicher@intel.com/

Changes since V6:
-----------------

  * Fixed commit message of patch 1/6:
    - Removed accidentally included leftover text at end of message.

  * Fixed commit message of patch 2/6:
    - Removed accidentally included "Approved-By: Tom Tromey" trailer.
      This patch was not reviewed by Tom; only patches 1/6 and 3/6 were.

  * Fixed code in infrun.c (patch 6/6):
    - show_schedlock_option: Changed type string for "eval" from
      "expression evaluation" to "function calls", consistent with
      the documentation and NEWS entries.
    - show_schedlock_option: Changed format string from "during the %s"
      to "during %s", matching the output shown in the commit message
      and documentation (e.g., "during normal execution").

Approval status:
  Patch 1/6: Approved-By: Tom Tromey <tom@tromey.com>
             https://inbox.sourceware.org/gdb-patches/87a4xu9733.fsf@tromey.com/

  Patch 2/6: (unchanged from V5, no review comments)
             https://inbox.sourceware.org/gdb-patches/20260129095918.2216661-3-klaus.gerlicher@intel.com/

  Patch 3/6: Approved-By: Tom Tromey <tom@tromey.com>
             https://inbox.sourceware.org/gdb-patches/875x8i970d.fsf@tromey.com/

  Patch 4/6: (unchanged from V5, no review comments)
             https://inbox.sourceware.org/gdb-patches/20260129095918.2216661-5-klaus.gerlicher@intel.com/

  Patch 5/6: Reviewed-By: Eli Zaretskii <eliz@gnu.org> (documentation parts)
             https://inbox.sourceware.org/gdb-patches/86ttb4t6kz.fsf@gnu.org/

  Patch 6/6: Reviewed-By: Eli Zaretskii <eliz@gnu.org> (documentation parts)
             https://inbox.sourceware.org/gdb-patches/86ldx8t6ex.fsf@gnu.org/
             Changes: code fixes in show_schedlock_option

  Overall series: Reviewed-By: Guinevere Larsen <guinevere@redhat.com>
                  https://inbox.sourceware.org/gdb-patches/bc6fd851-29c1-436e-a833-a53c0cf5a18d@redhat.com/

Thanks to all reviewers for their thorough feedback throughout the
v3, v4, v5, and v6 iterations.

Best regards,
Klaus
 
Natalia Saiapova (6):
  gdb: use schedlock_applies in user_visible_resume_ptid.
  gdb, cli: remove left-over code from "set_logging_on".
  gdb, cli: pass the argument of a set command to its callback.
  gdb: change the internal representation of scheduler locking.
  gdb: refine commands to control scheduler locking.
  gdb: add eval option to lock the scheduler during infcalls.

 gdb/NEWS                                      |  26 ++
 gdb/cli/cli-logging.c                         |   5 -
 gdb/cli/cli-setshow.c                         |   2 +-
 gdb/doc/gdb.texinfo                           |  77 +++-
 gdb/infrun.c                                  | 414 +++++++++++++++---
 .../gdb.mi/user-selected-context-sync.exp     |  23 +-
 .../gdb.threads/hand-call-in-threads.exp      |  12 +-
 .../multiple-successive-infcall.exp           |   7 +-
 gdb/testsuite/gdb.threads/schedlock.exp       | 105 ++++-
 gdb/testsuite/lib/gdb.exp                     |  71 ++-
 10 files changed, 621 insertions(+), 121 deletions(-)