You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by jp...@apache.org on 2013/03/21 18:36:20 UTC

git commit: TS-1746: disable 128bit compare and swap

Updated Branches:
  refs/heads/master 57ffdf5fc -> bd8fd4fbb


TS-1746: disable 128bit compare and swap

Disable the use of 128bit compare and swap that was introduced in
TS-1742. That change was causing crashes in the freelist.


Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/bd8fd4fb
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/bd8fd4fb
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/bd8fd4fb

Branch: refs/heads/master
Commit: bd8fd4fbb5ee4026f107365bcab6d546fc855f83
Parents: 57ffdf5
Author: James Peach <jp...@apache.org>
Authored: Thu Mar 21 10:34:07 2013 -0700
Committer: James Peach <jp...@apache.org>
Committed: Thu Mar 21 10:34:07 2013 -0700

----------------------------------------------------------------------
 configure.ac |   52 +++++++++++++++++++++++++++-------------------------
 1 files changed, 27 insertions(+), 25 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/trafficserver/blob/bd8fd4fb/configure.ac
----------------------------------------------------------------------
diff --git a/configure.ac b/configure.ac
index 652f327..19d2a39 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1108,31 +1108,33 @@ AC_SUBST(need_union_semun)
 AC_MSG_CHECKING(for 128bit CAS support)
 AC_LANG_PUSH([C++])
 
-TS_TRY_COMPILE_NO_WARNING([],[
-    __int128_t x = 0;
-    __sync_bool_compare_and_swap(&x,0,10);
-  ], [
-    AC_MSG_RESULT(yes)
-    has_128bit_cas=1
-  ], [
-      dnl If 128bit CAS fails, try again with the -mcx16 option. GCC needs this;
-      dnl clang doesn't; icc is unknown but presumed sane.
-    __saved_CXXFLAGS="${CXXFLAGS}"
-    TS_ADDTO(CXXFLAGS, [-mcx16])
-    TS_TRY_COMPILE_NO_WARNING([],[
-        __int128_t x = 0;
-        __sync_bool_compare_and_swap(&x,0,10);
-      ], [
-        AC_MSG_RESULT(yes)
-        has_128bit_cas=1
-        dnl Keep CFLAGS and CXXFLAGS in sync.
-        TS_ADDTO(CFLAGS, [-mcx16])
-      ], [
-        AC_MSG_RESULT(no)
-        has_128bit_cas=0
-        CXXFLAGS="${__saved_CXXFLAGS}"
-    ])
-])
+dnl TS_TRY_COMPILE_NO_WARNING([],[
+dnl     __int128_t x = 0;
+dnl     __sync_bool_compare_and_swap(&x,0,10);
+dnl   ], [
+dnl     AC_MSG_RESULT(yes)
+dnl     has_128bit_cas=1
+dnl   ], [
+dnl       dnl If 128bit CAS fails, try again with the -mcx16 option. GCC needs this;
+dnl       dnl clang doesn't; icc is unknown but presumed sane.
+dnl     __saved_CXXFLAGS="${CXXFLAGS}"
+dnl     TS_ADDTO(CXXFLAGS, [-mcx16])
+dnl     TS_TRY_COMPILE_NO_WARNING([],[
+dnl         __int128_t x = 0;
+dnl         __sync_bool_compare_and_swap(&x,0,10);
+dnl       ], [
+dnl         AC_MSG_RESULT(yes)
+dnl         has_128bit_cas=1
+dnl         dnl Keep CFLAGS and CXXFLAGS in sync.
+dnl         TS_ADDTO(CFLAGS, [-mcx16])
+dnl       ], [
+dnl         AC_MSG_RESULT(no)
+dnl         has_128bit_cas=0
+dnl         CXXFLAGS="${__saved_CXXFLAGS}"
+dnl     ])
+dnl ])
+
+has_128bit_cas=0
 
 AC_LANG_POP
 AC_SUBST(has_128bit_cas)