summaryrefslogtreecommitdiff
path: root/src/bus-proxyd/proxy.h
diff options
context:
space:
mode:
authorDavid Herrmann <dh.herrmann@gmail.com>2015-03-11 13:53:21 +0100
committerDavid Herrmann <dh.herrmann@gmail.com>2015-03-11 13:57:38 +0100
commitec2c7b56599981a7d9e76b15c75af3e1af3e6f81 (patch)
treeeebc77b642b1f8ae21ce2bdc1915ffa2badb79e9 /src/bus-proxyd/proxy.h
parentb2068772f5fd05173c4a42a907e42bb69160170c (diff)
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).
Diffstat (limited to 'src/bus-proxyd/proxy.h')
-rw-r--r--src/bus-proxyd/proxy.h1
1 files changed, 1 insertions, 0 deletions
diff --git a/src/bus-proxyd/proxy.h b/src/bus-proxyd/proxy.h
index 172bc7f569..ff278a2465 100644
--- a/src/bus-proxyd/proxy.h
+++ b/src/bus-proxyd/proxy.h
@@ -38,6 +38,7 @@ struct Proxy {
SharedPolicy *policy;
bool got_hello : 1;
+ bool queue_overflow : 1;
};
int proxy_new(Proxy **out, int in_fd, int out_fd, const char *dest);