Skip to content

manager: Add temporary code to debug issue #7150 - #7204

Draft
CharlieFenton wants to merge 5 commits into
client_release/8/8.2from
client_8.2.15_test
Draft

CharlieFenton wants to merge 5 commits into
client_release/8/8.2from
client_8.2.15_test

Conversation

@CharlieFenton

@CharlieFenton CharlieFenton commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Add temporary debugging code to Manager to:

  • show time each RPC request is added to asynchronous RPC queue
  • show time each RPC request completes
  • RPC causing "Communicating with client" dialog to appear
  • all RPCs in queue when the dialog appears
  • whether the dialog was dismissed automatically because RPC completed (wxID_OK), user clicked Cancel (wxID_CANCEL) or use clicked "Exit BOINC Manager (wxID_EXIT).

Built on BOINC 8.2.15 rather than master to avoid the possibility that later changes to master might affect the problems reported in issue #7150


Summary by cubic

Adds compile-time gated debug logging in the Manager to trace async RPCs and diagnose “Communicating with client” stalls. Logs each RPC enqueue/completion with microsecond timestamps and RPC names, prints when the dialog opens/closes with the queue and close reason, logs RPC errors, and adds PRINT_ONLY_DEMAND_RPC_INFO to limit the enqueue/completion logging to demand RPCs.

Written for commit d14ce1a. Summary will update on new commits.

Review in cubic

@CharlieFenton
CharlieFenton marked this pull request as draft August 5, 2026 13:10
@CharlieFenton CharlieFenton changed the title manager: Print all RPC requests and completions plus Communicating di… manager: Add temporary code to debug issue #7150 Aug 5, 2026

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 1 file

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread clientgui/AsyncRPC.cpp Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant