diff options
author | Eric Wong <e@yhbt.net> | 2020-06-27 10:03:39 +0000 |
---|---|---|
committer | Eric Wong <e@yhbt.net> | 2020-06-28 22:27:16 +0000 |
commit | fac2f5b63f0595b67a99784c8a9f9801c1fc9f09 (patch) | |
tree | bb225dd4903a7078dbe1f4a271a2d59cc167777c /lib/PublicInbox/DS.pm | |
parent | 94096cab6cd5e00c8a36a4a2667bdb9acf43d01f (diff) | |
download | public-inbox-fac2f5b63f0595b67a99784c8a9f9801c1fc9f09.tar.gz |
We can avoid synchronous `waitpid(-1, 0)' and save a process when simultaneously watching Maildirs. One DS bug is fixed: ->Reset needs to clear the DS $in_loop flag in forked children so dwaitpid() fails and allows git processes to be reaped synchronously. TestCommon also calls DS->Reset when spawning new processes, since t/imapd.t uses DS->EventLoop while waiting on -watch to write.
Diffstat (limited to 'lib/PublicInbox/DS.pm')
-rw-r--r-- | lib/PublicInbox/DS.pm | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/PublicInbox/DS.pm b/lib/PublicInbox/DS.pm index da68802d..c46b20cb 100644 --- a/lib/PublicInbox/DS.pm +++ b/lib/PublicInbox/DS.pm @@ -68,7 +68,7 @@ Reset all state =cut sub Reset { %DescriptorMap = (); - $wait_pids = $later_queue = undef; + $in_loop = $wait_pids = $later_queue = undef; $EXPMAP = {}; $nextq = $ToClose = $reap_timer = $later_timer = $exp_timer = undef; $LoopTimeout = -1; # no timeout by default |