Re: [Qemu-devel] [PATCH] ui/cocoa.m: Fix recent compile breakage

["Andreas Färber" <andreas.faerber@web.de>]

Am 22.04.2013 14:32, schrieb Paolo Bonzini:
> Il 22/04/2013 13:51, Peter Maydell ha scritto:
>> On 22 April 2013 00:03, Andreas Färber <andreas.faerber@web.de> wrote:
>>> Even after this has been applied, things are still not building for me.
>>>
>>> I'm seeing CC trace/generated-events.o failing with "error: parameter
>>> name omitted" for trace_apic_local_deliver, trace_usb_xhci_queue_event,
>>> trace_megasas_msix_raise and trace_spapr_pci_msi_setup, plus a warning
>>> that trace_event_count is being declared twice.
>>
>> I suspect that whatever compiler you're using here treats 'vector'
>> as a magic word, so when it sees a prototype like
>>   static inline void trace_megasas_msix_raise(int vector)
>>
>> it's expecting that 'vector' is an attribute of the type and
>> it wants "int vector foo", hence the error message.
>>
>> You could test this theory by changing all the 'vector's in
>> trace-events to 'vec' instead and rebuilding.
> 
> The culprit is likely altivec.h.
> 
> Something like this should fix it:

That works fine, but one error remains:

  LINK  i386-softmmu/qemu-system-i386
Undefined symbols:
  "___sync_val_compare_and_swap", referenced from:
      _migrate_finish_set_state in migration.o
ld: symbol(s) not found
collect2: ld returned 1 exit status
make[1]: *** [qemu-system-i386] Error 1
make: *** [subdir-i386-softmmu] Error 2

That seems unrelated, so I'll send out this patch later.

Thanks,
Andreas

> 
> diff --git a/include/qemu-common.h b/include/qemu-common.h
> index 2cfb1f0..6c23b1b 100644
> --- a/include/qemu-common.h
> +++ b/include/qemu-common.h
> @@ -436,12 +436,18 @@
>  /* vector definitions */
>  #ifdef __ALTIVEC__
>  #include <altivec.h>
> -#define VECTYPE        vector unsigned char
> +/* The altivec.h header says we're allowed to undef these for
> + * C++ compatibility.  Here we don't care about C++, but we
> + * undef them anyway to avoid namespace pollution.
> + */
> +#undef vector
> +#undef pixel
> +#undef bool
> +#define VECTYPE        __vector unsigned char
>  #define SPLAT(p)       vec_splat(vec_ld(0, p), 0)
>  #define ALL_EQ(v1, v2) vec_all_eq(v1, v2)
>  /* altivec.h may redefine the bool macro as vector type.
>   * Reset it to POSIX semantics. */
> -#undef bool
>  #define bool _Bool
>  #elif defined __SSE2__
>  #include <emmintrin.h>
> 
> Paolo
>