Re: [PATCH 0/6] Eliminate infrun_thread_thread_exit observer

[Pedro Alves <pedro@palves.net>]

Hi!

On 2022-12-03 9:13 p.m., Pedro Alves wrote:
> This series started out as me realizing that I had one change in the
> pending step-over-thread-exit series that was in conflict with
> infrun_thread_thread_exit.  infrun_thread_thread_exit wants to clear
> the last target waitstatus when that status is for the thread that
> exited, while I'll need to preserve the thread exit waitstatus for
> normal stop.  In the step-over-thread-exit series, I was calling
> set_last_target_status again when handling a
> TARGET_WAITKING_THREAD_EXIT, and that was undoing the clear that
> infrun_thread_thread_exit does.
> 
> So I looked at eliminating infrun_thread_thread_exit, see if we could
> make whatever is relying on it no longer rely on it.
> 
> The end result is this thread.  It ends up fixing some bugs and
> improving GDB behavior such that I think it stands on its own, even
> without the last patch, the one that finally eliminates
> infrun_thread_thread_exit.  I kept the thread title about
> infrun_thread_thread_exit because that's my end goal, but I could as
> well remove the last patch and call this thread "improve 'info
> program' and 'follow-fork'".
> 
> Tested on x86-64 GNU/Linux, native and extended-gdbserver.
> 
> Pedro Alves (6):
>   Tweak "Using the running image of ..." output
>   Convert previous_inferior_ptid to strong reference to thread_info
>   Improve "info program"
>   Make follow_fork not rely on get_last_target_status
>   all-stop "follow-fork parent" and selecting another thread
>   Remove infrun_thread_thread_exit observer

FYI, I'm putting this in.