[Gc] Re[2]: Fwd: atomic_ops for ppc64 testing of CAS

[Ivan Maidanski]

Hi Andreas,

Thanks. I've checked in your patch.

Regards.

18 11 2011, 09:44 Andreas Tobler <andreastt at gmail.com>:
> Hi Ivan,
> 
> in case you didn't get it. I had some mailer issues the last two days.....
> 
> Gruss,
> Andreas
> 
> -------- Original Message --------
> Subject: Re: atomic_ops for ppc64 testing of CAS
> Date: Thu, 17 Nov 2011 20:51:58 +0100
> From: Andreas Tobler <andreast-list at fgznet.ch>
> CC: IvanMaidanski <ivmai at mail.ru>
> 
> Hi Ivan,
> 
> On 17.11.11 13:19, Andreas Tobler wrote:
> 
> > I have. I'll give you a feedback for ppc64 linux and freebsd later tonight.
> 
> Tested on powerpc-freebsd, powerpc64-freebsd and Fedora F16 powerpc64.
> 
> All mention this:
> ==================
> All 4 tests passed
> ==================
> 
> I had to apply the below patch to be able to compile.
> 
> So far I did not do deeper testing. If you need to know something
> special feel free to ask.
> 
> Gruss,
> Andreas
> 
> diff --git a/src/atomic_ops/sysdeps/gcc/powerpc.h
> b/src/atomic_ops/sysdeps/gcc/powerpc.h
> index cc1aaa6..b843692 100644
> --- a/src/atomic_ops/sysdeps/gcc/powerpc.h
> +++ b/src/atomic_ops/sysdeps/gcc/powerpc.h
> @@ -246,7 +246,7 @@ AO_fetch_compare_and_swap(volatile AO_t *addr, AO_t
> old_val, AO_t new_val)
>          "stdcx. %2,0,%1\n"        /* else store conditional       */
>          "bne- 1b\n"               /* retry if lost reservation    */
>          "2:\n"
> -      : "=&r"(fetched_val),
> +      : "=&r"(fetched_val)
>          : "r"(addr), "r"(new_val), "r"(old_val)
>          : "memory", "cr0");
>    # else
> @@ -257,7 +257,7 @@ AO_fetch_compare_and_swap(volatile AO_t *addr, AO_t
> old_val, AO_t new_val)
>          "stwcx. %2,0,%1\n"        /* else store conditional       */
>          "bne- 1b\n"               /* retry if lost reservation    */
>          "2:\n"
> -      : "=&r"(fetched_val),
> +      : "=&r"(fetched_val)
>          : "r"(addr), "r"(new_val), "r"(old_val)
>