[ltt-dev] [PATCH 03/11] add urcu/arch_defaults.h

Paolo Bonzini pbonzini at redhat.com
Mon Feb 15 02:53:34 EST 2010


On 02/14/2010 03:34 PM, Mathieu Desnoyers wrote:
> +#if !defined(mc)&&  !defined(rmc)&&  !defined(wmc)
>
> instead ?

Ok.

>> >  +/*
>> >  + * Architectures without cache coherency need something like the following:
>> >  + *
>> >  + * #define mb()	mc()
>> >  + * #define rmb()	rmc()
>> >  + * #define wmb()	wmc()
>> >  + * #define mc()	arch_cache_flush()        // mandatory
>
> The // comment style should be avoided. (Following Linux kernel
> guide-lines)

Note that this is within a comment; I used it reluctantly because C 
comments do not nest.  I could replace it with "<-- mandatory" but I 
think // is better in this case.

Paolo




More information about the lttng-dev mailing list