Re: [LAD] Atomic Operations

From: Tim Goetze <tim@email-addr-hidden>
Date: Thu Dec 17 2009 - 17:00:17 EET

[Olivier Guilyardi]

>On 12/17/2009 01:03 PM, Tim Blechmann wrote:
>>> +#if defined(__APPLE__)
>>> +#include <libkern/OSAtomic.h>
>>> +#define MEMORY_BARRIER() OSMemoryBarrier()
>>> +#elif (__GNUC__ > 4) || (__GNUC__ == 4 && __GNUC_MINOR__ >= 1)
>>> +#define MEMORY_BARRIER() __sync_synchronize()
>>> +#else
>>> +#warning SMP Danger: memory barriers are not supported on this system
>>> +#endif

I may be wrong but if neither __APPLE__ nor __GNUC__ >= 4.1 are true
shouldn't MEMORY_BARRIER() at least be an empty define to keep cpp
happy?

After all, your #warning statement seems to indicate you don't want
compilation to fail.

Cheers, Tim
_______________________________________________
Linux-audio-dev mailing list
Linux-audio-dev@email-addr-hidden
http://lists.linuxaudio.org/mailman/listinfo/linux-audio-dev
Received on Thu Dec 17 20:15:02 2009

This archive was generated by hypermail 2.1.8 : Thu Dec 17 2009 - 20:15:02 EET