qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] osdep: initialize glib threads in all QEMU tool


From: Stefan Hajnoczi
Subject: Re: [Qemu-devel] [PATCH] osdep: initialize glib threads in all QEMU tools
Date: Wed, 9 Oct 2013 10:31:41 +0200
User-agent: Mutt/1.5.21 (2010-09-15)

On Tue, Oct 08, 2013 at 03:58:24PM +0200, Kevin Wolf wrote:
> Am 08.10.2013 um 15:08 hat Paolo Bonzini geschrieben:
> > Il 08/10/2013 14:25, Kevin Wolf ha scritto:
> > > The glib documentation says:
> > > 
> > >     Since version 2.24, calling g_thread_init() multiple times is
> > >     allowed, but nothing happens except for the first call.
> > > 
> > > I take that this means previously it wasn't allowed. qemu's configure
> > > checks for a minimum version of 2.12, so we seems to support glib
> > > versions that don't allow g_thread_init() to be called multiple times.
> > > 
> > > Do we need to protect against this?
> > 
> > I think that's the point of the "if (!g_thread_supported ())" tests.
> 
> Ah yes, I think you're right. Not the best function name I've ever seen
> that glib uses there, but okay.

That's correct, g_thread_support() is there to initialize once only.

Stefan



reply via email to

[Prev in Thread] Current Thread [Next in Thread]