priority of gnonlin and clutter-toolkit

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

priority of gnonlin and clutter-toolkit

Aapo Rantalainen
Hi, I'm using gnonlin and clutter together and there are problems with
priorities. Gnonlin should have better priority than redraw of
Clutter.

gnl/gnlcomposition.c:633
/* FIXME : This should be switched to using a g_thread_create() instead
       * of a g_idle_add(). EXTENSIVE TESTING AND ANALYSIS REQUIRED BEFORE
       * DOING THE SWITCH !!! */
      comp->priv->pending_idle =

-  g_idle_add ((GSourceFunc) eos_main_thread, (gpointer) comp);
+ g_idle_add_full (G_PRIORITY_HIGH_IDLE, (GSourceFunc)
eos_main_thread, (gpointer) comp, NULL);


http://www.clutter-project.org/docs/clutter/stable/clutter-General.html#CLUTTER-PRIORITY-REDRAW--CAPS

-Aapo Rantalainen

------------------------------------------------------------------------------
Return on Information:
Google Enterprise Search pays you back
Get the facts.
http://p.sf.net/sfu/google-dev2dev
_______________________________________________
gstreamer-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/gstreamer-devel
Reply | Threaded
Open this post in threaded view
|

Re: priority of gnonlin and clutter-toolkit

Tim-Philipp Müller-2
On Tue, 2009-12-15 at 14:35 +0200, Aapo Rantalainen wrote:

Hi,

> Hi, I'm using gnonlin and clutter together and there are problems with
> priorities. Gnonlin should have better priority than redraw of
> Clutter.
>
> gnl/gnlcomposition.c:633
> /* FIXME : This should be switched to using a g_thread_create() instead
>        * of a g_idle_add(). EXTENSIVE TESTING AND ANALYSIS REQUIRED BEFORE
>        * DOING THE SWITCH !!! */
>       comp->priv->pending_idle =
>
> -  g_idle_add ((GSourceFunc) eos_main_thread, (gpointer) comp);
> + g_idle_add_full (G_PRIORITY_HIGH_IDLE, (GSourceFunc)
> eos_main_thread, (gpointer) comp, NULL);
>
>
> http://www.clutter-project.org/docs/clutter/stable/clutter-General.html#CLUTTER-PRIORITY-REDRAW--CAPS

Could you file a bug in bugzilla about this please, so it's not
forgotten about and we can be sure the gnonlin maintainers actually
notice it. Thanks!

Cheers
 -Tim



------------------------------------------------------------------------------
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
_______________________________________________
gstreamer-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/gstreamer-devel
Reply | Threaded
Open this post in threaded view
|

Re: priority of gnonlin and clutter-toolkit

Edward Hervey
Administrator
On Sat, 2009-12-19 at 15:34 +0000, Tim-Philipp Müller wrote:

> On Tue, 2009-12-15 at 14:35 +0200, Aapo Rantalainen wrote:
>
> Hi,
>
> > Hi, I'm using gnonlin and clutter together and there are problems with
> > priorities. Gnonlin should have better priority than redraw of
> > Clutter.
> >
> > gnl/gnlcomposition.c:633
> > /* FIXME : This should be switched to using a g_thread_create() instead
> >        * of a g_idle_add(). EXTENSIVE TESTING AND ANALYSIS REQUIRED BEFORE
> >        * DOING THE SWITCH !!! */
> >       comp->priv->pending_idle =
> >
> > -  g_idle_add ((GSourceFunc) eos_main_thread, (gpointer) comp);
> > + g_idle_add_full (G_PRIORITY_HIGH_IDLE, (GSourceFunc)
> > eos_main_thread, (gpointer) comp, NULL);
> >
> >
> > http://www.clutter-project.org/docs/clutter/stable/clutter-General.html#CLUTTER-PRIORITY-REDRAW--CAPS
>
> Could you file a bug in bugzilla about this please, so it's not
> forgotten about and we can be sure the gnonlin maintainers actually
> notice it. Thanks!

  That would be https://bugzilla.gnome.org/show_bug.cgi?id=595187

  Since, as written in the big comment just above, the proper fix isn't
to change idle_add priorities... but to use a *real* separate thread.

   Edward

>
> Cheers
>  -Tim
>
>
>
> ------------------------------------------------------------------------------
> This SF.Net email is sponsored by the Verizon Developer Community
> Take advantage of Verizon's best-in-class app development support
> A streamlined, 14 day to market process makes app distribution fast and easy
> Join now and get one step closer to millions of Verizon customers
> http://p.sf.net/sfu/verizon-dev2dev 
> _______________________________________________
> gstreamer-devel mailing list
> [hidden email]
> https://lists.sourceforge.net/lists/listinfo/gstreamer-devel



------------------------------------------------------------------------------
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
_______________________________________________
gstreamer-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/gstreamer-devel