diff options
| author | Matthew Wild <mwild1@gmail.com> | 2023-11-21 18:12:49 +0000 |
|---|---|---|
| committer | Matthew Wild <mwild1@gmail.com> | 2023-11-21 18:12:49 +0000 |
| commit | 9e43a7742ab4f6e44c76158284f95f8b3ce520c4 (patch) | |
| tree | a42ab9d0eb07d834c209ca8b9aa5cda93806c2f6 | |
| parent | ab26786924b3b5871cba11f5f2fe265a95480881 (diff) | |
| download | prosody-9e43a7742ab4f6e44c76158284f95f8b3ce520c4.tar.gz prosody-9e43a7742ab4f6e44c76158284f95f8b3ce520c4.zip | |
util.async: Improve debug logging in a few places
Knowing the state of the coroutine as well as the runner state can be helpful.
| -rw-r--r-- | util/async.lua | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/util/async.lua b/util/async.lua index ff88ad17..b225cc72 100644 --- a/util/async.lua +++ b/util/async.lua @@ -46,7 +46,9 @@ end local function runner_continue(thread) -- ASSUMPTION: runner is in 'waiting' state (but we don't have the runner to know for sure) if coroutine.status(thread) ~= "suspended" then -- This should suffice - log("error", "unexpected async state: thread not suspended"); + log("error", "unexpected async state: thread not suspended (%s, %s)", thread, coroutine.status(thread)); + -- Fetching the traceback is likely to *crash* if a C library is calling us while suspended + --log("error", "coroutine stack: %s", debug.traceback()); return false; end local ok, state, runner = coroutine.resume(thread); @@ -198,6 +200,7 @@ function runner_mt:run(input) -- Loop through queue items, and attempt to run them for i = 1,n do local queued_input = q[i]; + self:log("Resuming thread with new item [%s]", thread); local ok, new_state = coroutine.resume(thread, queued_input); if not ok then -- There was an error running the coroutine, save the error, mark runner as ready to begin again @@ -225,7 +228,7 @@ function runner_mt:run(input) -- Runner processed all items it can, so save current runner state self.state = state; if err or state ~= self.notified_state then - self:log("debug", "changed state from %s to %s", self.notified_state, err and ("error ("..state..")") or state); + self:log("debug", "changed state from %s to %s [%s %s]", self.notified_state, err and ("error ("..state..")") or state, self.thread, self.thread and coroutine.status(self.thread)); if err then state = "error" else |