Bug 37354 - Make the heartbeat synchronise with the other applications
Summary: Make the heartbeat synchronise with the other applications
Status: RESOLVED FIXED
Alias: None
Product: Telepathy
Classification: Unclassified
Component: rakia (show other bugs)
Version: git master
Hardware: Other All
: medium normal
Assignee: Marco Barisione
QA Contact: Telepathy bugs list
URL: http://cgit.collabora.com/git/user/ba...
Whiteboard: review+
Keywords: patch
Depends on: 37115
Blocks:
  Show dependency treegraph
 
Reported: 2011-05-19 03:57 UTC by Marco Barisione
Modified: 2011-05-27 01:30 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:


Attachments

Description Marco Barisione 2011-05-19 03:57:26 UTC
+++ This bug was initially created as a clone of Bug #37115 +++

MeeGo has a new version of iphb that allows a different kind of interval to be passed to iphb_wait().
Currently we pass the minimum and maximum number of seconds we want to be waken up from now. With iphb 0.61.29 we can pass the same interval both as minimum and maximum value and that means we want to be waken up every x seconds (so the first time it could be much shorter).
If we use this feature with a set of recommended intervals (used also by other applications) we can get a better power usage as all the application will be waken up at the same time.
Comment 1 Marco Barisione 2011-05-19 03:58:29 UTC
Taking.
Comment 2 Marco Barisione 2011-05-19 06:31:03 UTC
http://cgit.collabora.com/git/user/bari/telepathy-rakia.git/log/?h=iphb-system-sync contains 3 patches to fix this. Warning: not tested :)
Comment 3 Mikhail Zabaluev 2011-05-19 09:08:04 UTC
The changes look good.
Didn't the iphb developers also provide a function that does what get_system_sync_interval() does in this branch?
Comment 4 Marco Barisione 2011-05-19 09:17:48 UTC
(In reply to comment #3)
> Didn't the iphb developers also provide a function that does what
> get_system_sync_interval() does in this branch?

No :( I even asked them if they wanted to copy my code directly, but they didn't see the point...
Comment 5 Marco Barisione 2011-05-23 06:26:22 UTC
Merged with the telepathy-sofiasip-0.6 branch.
Comment 6 Mikhail Zabaluev 2011-05-27 01:30:23 UTC
The release telepathy-sofiasip 0.6.7 has the fix.


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.