chiark / gitweb /
bus-proxy: complain only once about queue overflows
authorDavid Herrmann <dh.herrmann@gmail.com>
Wed, 11 Mar 2015 12:53:21 +0000 (13:53 +0100)
committerDavid Herrmann <dh.herrmann@gmail.com>
Wed, 11 Mar 2015 12:57:38 +0000 (13:57 +0100)
commitec2c7b56599981a7d9e76b15c75af3e1af3e6f81
treeeebc77b642b1f8ae21ce2bdc1915ffa2badb79e9
parentb2068772f5fd05173c4a42a907e42bb69160170c
bus-proxy: complain only once about queue overflows

If the local peer does not dispatch its incoming queue, the bus-proxy will
slowly fill its outgoing queue. Once its full, it will continously
complain that it cannot forward its messages.

As it turns out, pulseaudio does have an idle background dbus connection
that is not integrated into any mainloop (and given that gdbus and
libdbus1 both support background shared connections, PA is probably not
the only example), therefore, the bus-proxy will loudly complain if it
cannot forward NameOwnerChanged events once the queue is full.

This commit makes the proxy track queue-state and complain only once the
queue runs full, not if it is already full.

A PA bug-report (and patch) has been filed, and other applications should
be fixed similarly. Hence, lets keep the error message, instead of
dropping it. It's unused resources we really want to get rid of, so
silencing the message does not really help (which is actually what
dbus-daemon does).
src/bus-proxyd/proxy.c
src/bus-proxyd/proxy.h