On Thu, Dec 21, 2017 at 01:27:33AM +0000, Marius Strobl wrote:
> Author: marius
> Date: Thu Dec 21 01:27:32 2017
> New Revision: 327053
> URL: https://svnweb.freebsd.org/changeset/base/327053
> 
> Log:
>   Remove MD atomic_load_{32,64,int,long,ptr}(9) obsolete since the addition
>   of (conflicting) MI ones in r326971.
I am sorry. Initially I wrote the patch with only support for required
MI types int long ptr and run the tinderbox. Then I decided that there
is no reason to not provide the rest, but did not realized that tb is
needed again.

> 
> Modified:
>   head/sys/sparc64/include/atomic.h
> 
> Modified: head/sys/sparc64/include/atomic.h
> ==============================================================================
> --- head/sys/sparc64/include/atomic.h Thu Dec 21 01:22:36 2017        
> (r327052)
> +++ head/sys/sparc64/include/atomic.h Thu Dec 21 01:27:32 2017        
> (r327053)
> @@ -258,11 +258,6 @@ atomic_fcmpset_rel_ ## name(volatile ptype p, vtype *e
>  }                                                                    \
>                                                                       \
>  static __inline vtype                                                        
> \
> -atomic_load_ ## name(volatile ptype p)                                       
> \
> -{                                                                    \
> -     return ((vtype)atomic_cas((p), 0, 0, sz));                      \
atomic_cas() does not have any barrier behavior, am I right ?
I.e., it does not prevent a reordering in RMO and PSO models.

> -}                                                                    \
> -static __inline vtype                                                        
> \
>  atomic_load_acq_ ## name(volatile ptype p)                           \
>  {                                                                    \
>       return ((vtype)atomic_cas_acq((p), 0, 0, sz));                  \
_______________________________________________
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to