Re: [Qemu-devel] build failure with coroutine-gthread

[Aneesh Kumar K.V]

On Tue, 9 Aug 2011 17:28:17 +0000, Blue Swirl <address@hidden> wrote:
> On Mon, Aug 8, 2011 at 9:29 AM, Stefan Hajnoczi <address@hidden> wrote:
> > On Mon, Aug 8, 2011 at 10:01 AM, Aneesh Kumar K.V
> > <address@hidden> wrote:
> >>
> >>
> >>  LINK  qemu-ga
> >> coroutine-gthread.o: In function `coroutine_init':
> >> /home/opensource/sources/qemu/qemu-upstream/coroutine-gthread.c:39: 
> >> undefined reference to `g_thread_init'
> >> collect2: ld returned 1 exit status
> >>
> >> The below patch fix the failure.  I also added the patch in the VirtFS
> >> pull request.
> >
> > It appears coroutine-core v7 was merged instead of v8.  The
> > differences are minor:
> >  * Bisectability: introduce gthread implementation before ucontext/fibers
> >  * Depend on gthread, not just glib, for multithreaded programming
> >
> > Here is the patchwork link to the gthread dependency patch in case a
> > committer wants to merge it immediately:
> > http://patchwork.ozlabs.org/patch/106810/
> 
> Coroutines only need gthreads when both ucontext and win32 versions
> are unavailable. This is handled better by my patch:
> http://lists.nongnu.org/archive/html/qemu-devel/2011-08/msg00998.html

We would soon need that for VirtFS. So that will be another check of 
if (linux && atttr )

-aneesh