[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 14/42] migration: prepare to access s->state out
From: |
Stefan Weil |
Subject: |
Re: [Qemu-devel] [PATCH 14/42] migration: prepare to access s->state outside critical sections |
Date: |
Fri, 29 Mar 2013 18:49:37 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux i686; rv:17.0) Gecko/20130308 Thunderbird/17.0.4 |
Am 25.03.2013 12:02, schrieb Paolo Bonzini:
>
> ----- Messaggio originale -----
>> Da: "Gerd Hoffmann" <address@hidden>
>> A: "Stefan Hajnoczi" <address@hidden>
>> Cc: "Paolo Bonzini" <address@hidden>, "qemu-devel" <address@hidden>, "Orit
>> Wasserman"
>> <address@hidden>, "Juan Quintela" <address@hidden>
>> Inviato: Lunedì, 25 marzo 2013 10:52:36
>> Oggetto: Re: [Qemu-devel] [PATCH 14/42] migration: prepare to access
>> s->state outside critical sections
>>
>> Hi,
>>
>>> http://buildbot.b1-systems.de/qemu/builders/default_mingw32/builds/566/steps/compile/logs/stdio
>>>
>>> The latest RHEL6 mingw gcc is version 4.4 but this buildslave is
>>> RHEL6.1 so perhaps it's an older version that is missing the
>>> atomics builtins?
>> No, it's RHEL-6.4 actually, even though the name suggests otherwise
>> (was a bad idea to include the minor rev in the buildslave name ...).
> Can you check if this fixes it?
>
> diff --git a/configure b/configure
> index 46a7594..a324ca5 100755
> --- a/configure
> +++ b/configure
> @@ -931,9 +931,9 @@ case "$cpu" in
> LDFLAGS="-m64 $LDFLAGS"
> ;;
> i386)
> - QEMU_CFLAGS="-m32 $QEMU_CFLAGS"
> - LDFLAGS="-m32 $LDFLAGS"
> - cc_i386='$(CC) -m32'
> + QEMU_CFLAGS="-m32 -mcpu=i486 $QEMU_CFLAGS"
> + LDFLAGS="-m32 -mcpu=i486 $LDFLAGS"
> + cc_i386='$(CC) -m32 -mcpu=i486'
> ;;
> x86_64)
> QEMU_CFLAGS="-m64 $QEMU_CFLAGS"
>
> Paolo
Hi Stefan, hi Paolo,
cross compilation on Debian Squeeze shows the same error.
It is fixed by setting the correct cpu architecture (-march=i686).
As far as I know, gcc 4.4 defaults to -march=i386 which does
not support the atomic operations.
I have sent a patch today - please try it.
Regards,
Stefan