]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commit
Avoid constant stream of TARGET_WAITKIND_NO_RESUMED
authorPedro Alves <pedro@palves.net>
Sat, 4 Jul 2020 18:31:21 +0000 (19:31 +0100)
committerPedro Alves <palves@redhat.com>
Mon, 6 Jul 2020 18:57:34 +0000 (19:57 +0100)
commitf7bf4732bb5a8d2b27c3ab7428f359481cffe970
treeabd9636baf72e208b7a761fd7aaccb92c3b66bfd
parentce502e47c0b5333703dfa4b6d1268ca65a0b2926
Avoid constant stream of TARGET_WAITKIND_NO_RESUMED

If we hit the synchronous execution command case described by
handle_no_resumed, and handle_no_resumed determines that the event
should be ignored, because it found a thread that is executing, we end
up in prepare_to_wait.

There, if the current target is not registered in the event loop right
now, we call mark_infrun_async_event_handler.  With that event handler
marked, the event loop calls again into fetch_inferior_event, which
calls target_wait, which returns TARGET_WAITKIND_NO_RESUMED, and we
end up in handle_no_resumed, again ignoring the event and marking
infrun_async_event_handler.  The result is that GDB is now always
keeping the CPU 100% busy in this loop, even though it continues to be
able to react to input and to real target events, because we still go
through the event-loop.

The problem is that marking of the infrun_async_event_handler in
prepare_to_wait.  That is there to handle targets that don't support
asynchronous execution.  So the correct predicate is whether async
execution is supported, not whether the target is async right now.

gdb/ChangeLog:

PR gdb/26199
* infrun.c (prepare_to_wait): Check target_can_async_p instead of
target_is_async_p.
gdb/infrun.c