[roll] Roll fuchsia [kernel][dlog] Use a Timer to allow printing from within the scheduler
This change resolves an issue that prevented printf from working
within the scheduler, and more generally, whenever there was an active
chainlock transaction.
printf bottoms out in DLog::Write, which places a record in a buffer
and then calls Event::Signal to unblock the notifier thread. Prior to
this change, if DLog::Write was called from a context with an existing
active chainlock transaction (e.g. from within the scheduler), calling
Event::Signal would trigger an assert (can't start a new transaction
when you've already got one in progress).
To resolve the issue, we update DLog::Write to check for an active
chainlock transaction. If there is none, it calls Event::Signal as
usual. If there one, DLog::Write will instead set a Timer that will
immediately fire once the current CPU has re-enabled interrupts after
finishing the current transaction (or during the period in which
interrupts are re-enabled during backoff). The Timer callback simply
calls Event::Signal on Write's behalf.
~DLog now cancels the Timer to ensure there is no use-after-free.
As a small optimization, DLog tracks whether there is a pending Timer
(pending_deferred_signal_). Of course, this tracking is not 100%
accurate as the Timer callback could be racing with the write.
However, because the observation is made in a critical section guarded
by the same lock that the dumper thread must ultimately acquire, there
is no risk of a "lost wakeup".
Original-Fixed: 339440805
Original-Fixed: 331847876
Original-Reviewed-on: https://fuchsia-review.googlesource.com/c/fuchsia/+/1045567
Original-Revision: 0bc3bcab25532535b793e767a763b3e73e249d79
GitOrigin-RevId: 0562996fa0f27a6a8332e9e4711baff04877c8db
Change-Id: I7c74be9358dd6cf33498beb8fb31944bec9542b9
diff --git a/stem b/stem
index 2ea9f93..9ed5f79 100644
--- a/stem
+++ b/stem
@@ -11,7 +11,7 @@
githooks="integration/git-hooks"
remote="https://fuchsia.googlesource.com/fuchsia"
gerrithost="https://fuchsia-review.googlesource.com"
- revision="48691ba26761e2b8bf1f0318853133219e0dd366"/>
+ revision="0bc3bcab25532535b793e767a763b3e73e249d79"/>
</projects>
<hooks>
<hook name="install-environment"
