[Mesa-dev] [PATCH] util/u_atomic: Add new macro p_atomic_add

[Aaron Watry]

On Fri, Feb 6, 2015 at 1:43 PM, Carl Worth <cworth at cworth.org> wrote:

> This provides for atomic addition, which will be used by an upcoming
> shader-cache patch. A simple test is added to "make check" as well.
>
> Note: The various O/S functions differ on whether they return the
> original value or the value after the addition, so I did not provide
> an add_return() macro which would be sensitive to that difference.
>
> ---
>
> Note: I referenced the MSDN web pages and some online Solaris man
> pages to update the relevant sections, (but I have neither compiled
> nor tested this new macro on those operating systems). I would
> appreciate any relvant review or reports from testing.
>
>  src/util/u_atomic.h      | 16 ++++++++++++++++
>  src/util/u_atomic_test.c |  5 +++++
>  2 files changed, 21 insertions(+)
>
> diff --git a/src/util/u_atomic.h b/src/util/u_atomic.h
> index cf7fff3..4eb0ec6 100644
> --- a/src/util/u_atomic.h
> +++ b/src/util/u_atomic.h
> @@ -39,6 +39,7 @@
>  #define p_atomic_dec_zero(v) (__sync_sub_and_fetch((v), 1) == 0)
>  #define p_atomic_inc(v) (void) __sync_add_and_fetch((v), 1)
>  #define p_atomic_dec(v) (void) __sync_sub_and_fetch((v), 1)
> +#define p_atomic_add(v, i) (void) __sync_add_and_fetch((v), (i))
>  #define p_atomic_inc_return(v) __sync_add_and_fetch((v), 1)
>  #define p_atomic_dec_return(v) __sync_sub_and_fetch((v), 1)
>  #define p_atomic_cmpxchg(v, old, _new) \
> @@ -60,6 +61,7 @@
>  #define p_atomic_dec_zero(_v) (p_atomic_dec_return(_v) == 0)
>  #define p_atomic_inc(_v) ((void) p_atomic_inc_return(_v))
>  #define p_atomic_dec(_v) ((void) p_atomic_dec_return(_v))
> +#define p_atomic_add(_v, _i) (*(_v) = *(_v) + (_i))
>  #define p_atomic_inc_return(_v) (++(*(_v)))
>  #define p_atomic_dec_return(_v) (--(*(_v)))
>  #define p_atomic_cmpxchg(_v, _old, _new) (*(_v) == (_old) ? (*(_v) =
> (_new), (_old)) : *(_v))
> @@ -144,6 +146,13 @@ char _InterlockedCompareExchange8(char volatile
> *Destination8, char Exchange8, c
>     sizeof *(_v) == sizeof(__int64) ? InterlockedDecrement64 ((__int64
> *)(_v)) : \
>                                       (assert(!"should not get here"), 0))
>
> +#define p_atomic_add(_v, _i) (\
> +   sizeof *(_v) == sizeof(short)   ? _InterlockedExchangeAdd8 ((char *)
>  (_v), (_i)) : \
>
+   sizeof *(_v) == sizeof(short)   ? _InterlockedExchangeAdd16((short *)
> (_v), (_i)) : \
>

Ignore me if this is a stupid question, but should those both be
sizeof(short)?  I'd expect the first to be sizeof(char).

--Aaron


> +   sizeof *(_v) == sizeof(long)    ? _InterlockedExchangeAdd  ((long *)
>  (_v), (_i)) : \
> +   sizeof *(_v) == sizeof(__int64) ? _InterlockedExchangeAdd64((__int64
> *)(_v), (_i)) : \
> +                                     (assert(!"should not get here"), 0))
> +
>  #define p_atomic_cmpxchg(_v, _old, _new) (\
>     sizeof *(_v) == sizeof(char)    ? _InterlockedCompareExchange8 ((char
> *)   (_v), (char)   (_new), (char)   (_old)) : \
>     sizeof *(_v) == sizeof(short)   ? _InterlockedCompareExchange16((short
> *)  (_v), (short)  (_new), (short)  (_old)) : \
> @@ -198,6 +207,13 @@ char _InterlockedCompareExchange8(char volatile
> *Destination8, char Exchange8, c
>     sizeof(*v) == sizeof(uint64_t) ? atomic_dec_64_nv((uint64_t *)(v)) : \
>                                      (assert(!"should not get here"), 0))
>
> +#define p_atomic_add(v, i) ((void)                                  \
> +   sizeof(*v) == sizeof(uint8_t)  ? atomic_add_8 ((uint8_t  *)(v), (i)) :
> \
> +   sizeof(*v) == sizeof(uint16_t) ? atomic_add_16((uint16_t *)(v), (i)) :
> \
> +   sizeof(*v) == sizeof(uint32_t) ? atomic_add_32((uint32_t *)(v), (i)) :
> \
> +   sizeof(*v) == sizeof(uint64_t) ? atomic_add_64((uint64_t *)(v), (i)) :
> \
> +                                    (assert(!"should not get here"), 0))
> +
>  #define p_atomic_cmpxchg(v, old, _new) ((typeof(*v)) \
>     sizeof(*v) == sizeof(uint8_t)  ? atomic_cas_8 ((uint8_t  *)(v),
> (uint8_t )(old), (uint8_t )(_new)) : \
>     sizeof(*v) == sizeof(uint16_t) ? atomic_cas_16((uint16_t *)(v),
> (uint16_t)(old), (uint16_t)(_new)) : \
> diff --git a/src/util/u_atomic_test.c b/src/util/u_atomic_test.c
> index 4845e75..c506275 100644
> --- a/src/util/u_atomic_test.c
> +++ b/src/util/u_atomic_test.c
> @@ -97,6 +97,11 @@
>        assert(v == ones && "p_atomic_dec_return"); \
>        assert(r == v && "p_atomic_dec_return"); \
>        \
> +      v = 23; \
> +      p_atomic_add(&v, 42); \
> +      r = p_atomic_read(&v); \
> +      assert(r == 65 && "p_atomic_add"); \
> +      \
>        (void) r; \
>        (void) b; \
>     }
> --
> 2.1.4
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20150206/8c19898b/attachment.html>