Fix handle_no_resumed w/ multiple targets
authorPedro Alves <pedro@palves.net>
Sat, 4 Jul 2020 18:26:59 +0000 (19:26 +0100)
committerPedro Alves <palves@redhat.com>
Fri, 10 Jul 2020 22:50:11 +0000 (23:50 +0100)
commit7d3badc6a88b510f96c9aa0bab8f3375292d23bf
tree2bbeaa4b4c51763ef1139c841f08a9467b7355b3
parent42bd97a6b1e5fa845af116ce52af1a8a3a58be7c
Fix handle_no_resumed w/ multiple targets

handle_no_resumed is currently not considering multiple targets.

Say you have two inferiors 1 and 2, each connected to a different
target, A and B.

Now say you set inferior 2 running, with "continue &".

Now you select a thread of inferior 1, say thread 1.2, and continue in
the foreground.  All other threads of inferior 1 are left stopped.
Thread 1.2 exits, and thus target A has no other resumed thread, so it
reports TARGET_WAITKIND_NO_RESUMED.

At this point, if both inferiors were running in the same target,
handle_no_resumed would realize that threads of inferior 2 are still
executing, so the TARGET_WAITKIND_NO_RESUMED event should be ignored.
But because handle_no_resumed only walks the threads of the current
target, it misses noticing that threads of inferior 2 are still
executing.  The fix is just to walk over all threads of all targets.

A testcase covering the use case above will be added in a following
patch.  It can't be added yet because it depends on yet another fix to
handle_no_resumed not included here.

gdb/ChangeLog:

PR gdb/26199
* infrun.c (handle_no_resumed): Handle multiple targets.
gdb/ChangeLog
gdb/infrun.c