Re: [Rhythmbox-devel] idle rhythmbox

2007-05-21 Thread Jonathan Matthew
On Mon, May 21, 2007 at 12:32:52PM -0700, Adam Zimmerman wrote: > On Mon, 2007-21-05 at 16:30 +0100, Peter Robinson wrote: > > > > > Any chance of a 0.10.1 release with the current work before it gets > > > > > the dependency on glib 2.14? > > > > > > > > We won't be adding a dependency on glib 2.1

Re: [Rhythmbox-devel] idle rhythmbox

2007-05-21 Thread Adam Zimmerman
On Mon, 2007-21-05 at 16:30 +0100, Peter Robinson wrote: > > > > Any chance of a 0.10.1 release with the current work before it gets > > > > the dependency on glib 2.14? > > > > > > We won't be adding a dependency on glib 2.14. We'll just use the new > > > timeout function if glib 2.14 is availabl

Re: [Rhythmbox-devel] idle rhythmbox

2007-05-21 Thread Peter Robinson
> > > Any chance of a 0.10.1 release with the current work before it gets > > > the dependency on glib 2.14? > > > > We won't be adding a dependency on glib 2.14. We'll just use the new > > timeout function if glib 2.14 is available at ./configure time. > > How would that work for python plugins l

Re: [Rhythmbox-devel] idle rhythmbox

2007-05-21 Thread Adam Zimmerman
On Mon, 2007-21-05 at 19:21 +1000, Jonathan Matthew wrote: > On Mon, May 21, 2007 at 09:21:31AM +0100, Peter Robinson wrote: > > Any chance of a 0.10.1 release with the current work before it gets > > the dependency on glib 2.14? > > We won't be adding a dependency on glib 2.14. We'll just use th

Re: [Rhythmbox-devel] idle rhythmbox

2007-05-21 Thread Jonathan Matthew
On Mon, May 21, 2007 at 09:21:31AM +0100, Peter Robinson wrote: > > > [g_timeout_add_seconds stuff] > > Hi All, > > This is nice work :-) Bug here > http://bugzilla.gnome.org/show_bug.cgi?id=399012 for those who haven't > found it. > > Any chance of a 0.10.1 release with the current work before

Re: [Rhythmbox-devel] idle rhythmbox

2007-05-21 Thread Peter Robinson
> > > Second one is in rhythmdb/rhythmdb.c in the rhythmdb_idle_poll_events, > > > line 2003. When idle it runs the else branch every second. Not sure > > > what it does every second but being the DB it may not be fixable. > > > > This is the hard one. While it is certainly fixable, it's somewhat >

Re: [Rhythmbox-devel] idle rhythmbox

2007-05-20 Thread Jonathan Matthew
On Mon, May 14, 2007 at 08:39:19AM +1000, James Doc Livingston wrote: > > Second one is in rhythmdb/rhythmdb.c in the rhythmdb_idle_poll_events, > > line 2003. When idle it runs the else branch every second. Not sure > > what it does every second but being the DB it may not be fixable. > > This is

Re: [Rhythmbox-devel] idle rhythmbox

2007-05-14 Thread Peter Robinson
Hi James, On 5/13/07, James Doc Livingston <[EMAIL PROTECTED]> wrote: > Hi Peter, > > On Sun, 2007-05-13 at 21:19 +0100, Peter Robinson wrote: > > I've played with this a bit more this afternoon using the blog posts > > that you mentioned. I have no idea whether what I've found is an issue > > but

Re: [Rhythmbox-devel] idle rhythmbox

2007-05-13 Thread James \"Doc\" Livingston
Hi Peter, On Sun, 2007-05-13 at 21:19 +0100, Peter Robinson wrote: > I've played with this a bit more this afternoon using the blog posts > that you mentioned. I have no idea whether what I've found is an issue > but from Federico's blog post the g_timeout_add is seems to be a > problem. From wha

Re: [Rhythmbox-devel] idle rhythmbox

2007-05-13 Thread Jonathan Matthew
On Sun, May 13, 2007 at 09:19:38PM +0100, Peter Robinson wrote: > Hi Bastien & all, > > I've played with this a bit more this afternoon using the blog posts > that you mentioned. I have no idea whether what I've found is an issue > but from Federico's blog post the g_timeout_add is seems to be a

Re: [Rhythmbox-devel] idle rhythmbox

2007-05-13 Thread Peter Robinson
> > The guys at intel have just released a tool called PowerTop > > http://www.linuxpowertop.org (announcement to fedora-devel - > > https://www.redhat.com/archives/fedora-devel-list/2007-May/msg00796.html > > ) > > > > Anyway I installed it to play around with and noticed that rhythmbox > > was ap

Re: [Rhythmbox-devel] idle rhythmbox

2007-05-13 Thread Peter Robinson
> > AFAICT from binding a strace to RB when its doing nothing there's lots > > of gettimeofday, poll and ioctl running every second. Sample attached. > > Not sure what else needs to be done from here. > > The strace is just a way to see if there are unnecessary wakeups. > There's a gdb trick mentio

Re: [Rhythmbox-devel] idle rhythmbox

2007-05-13 Thread Bastien Nocera
On Sun, 2007-05-13 at 14:29 +0100, Peter Robinson wrote: > Thanks Bastien, > > AFAICT from binding a strace to RB when its doing nothing there's lots > of gettimeofday, poll and ioctl running every second. Sample attached. > Not sure what else needs to be done from here. The strace is just a way

Re: [Rhythmbox-devel] idle rhythmbox

2007-05-13 Thread Peter Robinson
> > > Anyway I installed it to play around with and noticed that rhythmbox > > > was appearing fairly frequently even though it was idle and not > > > playing (I forgot I even had it running). Something like 6.2% of the > > > nearly 2500 wakeups in a second (output below). > > > > >3.6%

Re: [Rhythmbox-devel] idle rhythmbox

2007-05-13 Thread Bastien Nocera
On Sun, 2007-05-13 at 13:33 +0100, Peter Robinson wrote: > > > The guys at intel have just released a tool called PowerTop > > > http://www.linuxpowertop.org (announcement to fedora-devel - > > > https://www.redhat.com/archives/fedora-devel-list/2007-May/msg00796.html > > > ) > > > > > > Anyway I i

Re: [Rhythmbox-devel] idle rhythmbox

2007-05-13 Thread Peter Robinson
> > The guys at intel have just released a tool called PowerTop > > http://www.linuxpowertop.org (announcement to fedora-devel - > > https://www.redhat.com/archives/fedora-devel-list/2007-May/msg00796.html > > ) > > > > Anyway I installed it to play around with and noticed that rhythmbox > > was ap

Re: [Rhythmbox-devel] idle rhythmbox

2007-05-13 Thread Bastien Nocera
On Sun, 2007-05-13 at 10:40 +0100, Peter Robinson wrote: > Hi All, > > The guys at intel have just released a tool called PowerTop > http://www.linuxpowertop.org (announcement to fedora-devel - > https://www.redhat.com/archives/fedora-devel-list/2007-May/msg00796.html > ) > > Anyway I installed i

[Rhythmbox-devel] idle rhythmbox

2007-05-13 Thread Peter Robinson
Hi All, The guys at intel have just released a tool called PowerTop http://www.linuxpowertop.org (announcement to fedora-devel - https://www.redhat.com/archives/fedora-devel-list/2007-May/msg00796.html ) Anyway I installed it to play around with and noticed that rhythmbox was appearing fairly fre