Bug 14184 - TpDynamicHandleRepo: listen for name owner changes more efficiently
Summary: TpDynamicHandleRepo: listen for name owner changes more efficiently
Status: RESOLVED FIXED
Alias: None
Product: Telepathy
Classification: Unclassified
Component: tp-glib (show other bugs)
Version: unspecified
Hardware: Other All
: medium enhancement
Assignee: Telepathy bugs list
QA Contact: Telepathy bugs list
URL:
Whiteboard:
Keywords:
Depends on: 14183
Blocks:
  Show dependency treegraph
 
Reported: 2008-01-21 11:14 UTC by Simon McVittie
Modified: 2009-09-09 10:43 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments

Description Simon McVittie 2008-01-21 11:14:27 UTC
If dbus-glib didn't indiscriminately connect to NameOwnerChanged, we could avoid waking up every time a process connects to or disconnects from the session bus, by using a less brute-force approach to monitoring the death of client processes in TpDynamicHandleRepo. Until #14183 is fixed, there's no point, though.
Comment 1 Mikhail Zabaluev 2008-12-30 10:20:23 UTC
Note that installing a name-specific watch upon learning about the client connection has a race condition with the client falling off before the watch is effective.
You may want to Ping the client after installing the watch, but that brings another bit of extra traffic in between useful messages. Sending the ping from an idle handler could be an optimization.
Comment 2 Mikhail Zabaluev 2009-01-12 05:19:05 UTC
(In reply to comment #1)
> You may want to Ping the client after installing the watch, but that brings
> another bit of extra traffic in between useful messages. Sending the ping from
> an idle handler could be an optimization.

And of course, there is a method to get the current name owner.
Comment 3 Simon McVittie 2009-09-09 10:43:12 UTC
Fixed in 0.7.35


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.