Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 15 Jun 2013 22:18:00 +0000 (UTC)
From:      Ed Schouten <ed@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r251796 - head/sbin/hastd
Message-ID:  <201306152218.r5FMI0uT047135@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: ed
Date: Sat Jun 15 22:17:59 2013
New Revision: 251796
URL: http://svnweb.freebsd.org/changeset/base/251796

Log:
  Let hastd use C11 atomics.
  
  C11 atomics now work on all the architectures. Have at least a single
  piece of software in our base system that uses C11 atomics. This
  somewhat makes it less likely that we break it because of LLVM imports,
  etc.

Modified:
  head/sbin/hastd/refcnt.h

Modified: head/sbin/hastd/refcnt.h
==============================================================================
--- head/sbin/hastd/refcnt.h	Sat Jun 15 21:29:47 2013	(r251795)
+++ head/sbin/hastd/refcnt.h	Sat Jun 15 22:17:59 2013	(r251796)
@@ -32,24 +32,24 @@
 #ifndef __REFCNT_H__
 #define __REFCNT_H__
 
-#include <machine/atomic.h>
+#include <stdatomic.h>
 
 #include "pjdlog.h"
 
-typedef unsigned int refcnt_t;
+typedef atomic_uint refcnt_t;
 
 static __inline void
 refcnt_init(refcnt_t *count, unsigned int v)
 {
 
-	*count = v;
+	atomic_init(count, v);
 }
 
 static __inline void
 refcnt_acquire(refcnt_t *count)
 {
 
-	atomic_add_acq_int(count, 1);
+	atomic_fetch_add_explicit(count, 1, memory_order_acquire);
 }
 
 static __inline unsigned int
@@ -58,7 +58,7 @@ refcnt_release(refcnt_t *count)
 	unsigned int old;
 
 	/* XXX: Should this have a rel membar? */
-	old = atomic_fetchadd_int(count, -1);
+	old = atomic_fetch_sub(count, 1);
 	PJDLOG_ASSERT(old > 0);
 	return (old - 1);
 }



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201306152218.r5FMI0uT047135>