tree 043d571f69c08cc1211dbd9d85821ca488160a38
parent 553cc842a3fea3594158c8314a577c2b916bf45d
author Josh Bleecher Snyder <josharian@gmail.com> 1751940773 -0700
committer Josh Bleecher Snyder <josharian@gmail.com> 1751940823 -0700

sketch/loop: fix concurrency handling in port monitor shutdown

The graceful shutdown code assumes it can take the pm.mu lock.
Make the Stop code oblige.

This fixes the issue at hand, but the remaining code still gives
me the heebie jeebies--for example, after calling Stop, there could
still be notifications that come in.

Sketch really shouldn't write concurrent code.

This will work well enough, I guess, until it doesn't,
at which point we'll have to carefully fix up the rats nest.


Before:

2m10s: 1157 runs so far, 36 failures (3.11%)

After:

16m45s: 12484 runs so far, 0 failures


Fixes boldsoftware/bold#446 enough for now