Re: [PATCHv4 0/2] Make thread_info executing and resumed state more consistent

[Craig Blackmore <craig.blackmore@embecosm.com>]

On 21/04/2022 17:45, Andrew Burgess via Gdb-patches wrote:
> And I'm back on this patch again!
>
> I've rebased this onto current master and done some testing.  I
> haven't found any regressions, but as I've said before, this change
> touches every target, and my testing on anything other than GNU/Linux
> (x86-64) is pretty thin, so I'm fully expecting this to introduce some
> issues - these issues _should_ be limited to assertion failures when
> threads are not in the expected states, I would hope there's no
> crashes being introduced.
>
> There's been no significant changes since v3.  I have split out a
> little bit of the debug printing code (the new #1 patch), but the
> second patch is pretty much the same as v3.
>
> V3 was given a cautious OK, but I wanted to advertise this patch again
> in case anyone wants to object before I merge this.

Hi Andrew,

Do you still intend to merge the second patch in this series?

Thanks,

Craig

> All feedback is welcome.
>
> Thanks,
> Andrew
>
> ---
>
> Andrew Burgess (2):
>    gdb: add some additional thread status debug output
>    gdb: make thread_info executing and resumed state more consistent
>
>   gdb/aix-thread.c             |  4 +-
>   gdb/bsd-kvm.c                |  2 +-
>   gdb/bsd-uthread.c            |  4 +-
>   gdb/corelow.c                |  9 +++-
>   gdb/darwin-nat.c             |  2 +-
>   gdb/fbsd-nat.c               |  4 +-
>   gdb/fork-child.c             |  3 ++
>   gdb/gdbthread.h              | 24 ++++++---
>   gdb/gnu-nat.c                | 13 +++--
>   gdb/go32-nat.c               |  2 +-
>   gdb/inf-ptrace.c             |  4 +-
>   gdb/infcmd.c                 | 84 +++++++++++++++++++++----------
>   gdb/infrun.c                 | 96 +++++++++++++++++++++++++++++++-----
>   gdb/infrun.h                 | 26 ++++++++++
>   gdb/linux-nat.c              |  8 +--
>   gdb/linux-thread-db.c        |  2 +-
>   gdb/netbsd-nat.c             |  4 +-
>   gdb/nto-procfs.c             |  2 +-
>   gdb/obsd-nat.c               |  4 +-
>   gdb/process-stratum-target.c |  4 +-
>   gdb/procfs.c                 | 12 ++---
>   gdb/ravenscar-thread.c       |  4 +-
>   gdb/remote-sim.c             |  2 +-
>   gdb/remote.c                 | 21 +++++---
>   gdb/sol-thread.c             |  4 +-
>   gdb/thread.c                 | 40 +++++++++++----
>   gdb/tracectf.c               |  3 +-
>   gdb/tracefile-tfile.c        |  2 +-
>   gdb/windows-nat.c            | 12 ++++-
>   29 files changed, 295 insertions(+), 106 deletions(-)
>