Re: [PATCH 0/3] Fix handling of pending fork events

[Simon Marchi <simark@simark.ca>]

Aaaaaand I forgot to use v2 in the subject :(.

On 2021-11-24 3:04 p.m., Simon Marchi via Gdb-patches wrote:
> This is v2 of:
> 
>   https://sourceware.org/pipermail/gdb-patches/2021-October/182922.html
> 
> Patch 2/3 is new, it factors out some code in the function, to keep the
> following patch simple.
> 
> Changes in patch 3/3 (patch 2/2 in v1) are:
> 
>  - The test did fail on some machines (not my development machine
>    apparently).  It is because I only handled pending statuses in
>    lwp_info::status, in the linux-nat target.  There may also be some
>    pending statuses in lwp_info::waitstatus, handle them too.
>  - Rather than adding a target method to allow the core to detach
>    pending fork children, make the targets (linux-nat and remote) peek
>    in thread_info::pending_waitstatus to see if there is one there.
> 
> Simon Marchi (3):
>   gdbserver: hide fork child threads from GDB
>   gdb/linux-nat: factor ptrace-detach code to new detach_one_pid
>     function
>   gdb, gdbserver: detach fork child when detaching from fork parent
> 
>  gdb/linux-nat.c                               | 124 ++++++++++-----
>  gdb/remote.c                                  |  14 ++
>  .../pending-fork-event-touch-file.c           |  26 +++
>  .../gdb.threads/pending-fork-event.c          |  82 ++++++++++
>  .../gdb.threads/pending-fork-event.exp        | 150 ++++++++++++++++++
>  gdbserver/gdbthread.h                         |   9 ++
>  gdbserver/linux-low.cc                        |  36 +++--
>  gdbserver/linux-low.h                         |   6 -
>  gdbserver/server.cc                           |  35 ++++
>  9 files changed, 422 insertions(+), 60 deletions(-)
>  create mode 100644 gdb/testsuite/gdb.threads/pending-fork-event-touch-file.c
>  create mode 100644 gdb/testsuite/gdb.threads/pending-fork-event.c
>  create mode 100644 gdb/testsuite/gdb.threads/pending-fork-event.exp
> 
> -- 
> 2.26.2
>