chiark / gitweb /
core: don't process dbus unit and job queue when there are already too many messages...
authorLennart Poettering <lennart@poettering.net>
Tue, 13 Feb 2018 17:30:34 +0000 (18:30 +0100)
committerSven Eden <yamakuzure@gmx.net>
Wed, 30 May 2018 05:59:08 +0000 (07:59 +0200)
commit459590e77817fd0b3d14168642b96b65a4f829e7
tree317e645e1a34995d9fc854349bbab16a06277a45
parent1a07732212ebf11d6f517efb3721dfc00aef6fe0
core: don't process dbus unit and job queue when there are already too many messages pending

We maintain a queue of units and jobs that we are supposed to generate
change/new notifications for because they were either just created or
some of their property has changed. Let's throttle processing of this
queue a bit: as soon as > 1K of bus messages are queued for writing
let's skip processing the queue, and then recheck on the next
iteration again.

Moreover, never process more than 100 units in one go, return to the
event loop after that. Both limits together should put effective limits
on both space and time usage of the function, delaying further
operations until a later moment, when the queue is empty or the the
event loop is sufficiently idle again.

This should keep the number of generated messages much lower than
before on busy systems or where some client is hanging.

Note that this also means a bad client can slow down message dispatching
substantially for up to 90s if it likes to, for all clients. But that
should be acceptable as we only allow trusted bus clients, anyway.

Fixes: #8166
src/systemd/sd-bus.h