Description of problem: __sync and __atomic built-ins are not correctly detected by configure.ac. Version-Release number of selected component (if applicable): How reproducible: Always Steps to Reproduce: 1. ./autogen.sh && ./configure Actual results: Macros HAVE_ATOMIC_BUILTINS and HAVE_SYNC_BUILTINS in config.h are not defined, compiling all the code using mutexes instead of atomic operations. Expected results: Those macros should be defined if compiler supports them. Additional info:
REVIEW: https://review.gluster.org/18689 (libglusterfs/atomic: Improved atomic support) posted (#1) for review on master by Xavier Hernandez
COMMIT: https://review.gluster.org/18689 committed in master by \"Xavier Hernandez\" <jahernan> with a commit message- libglusterfs/atomic: Improved atomic support This patch solves a detection problem in configure.ac that prevented that compilation detects builtin __atomic or __sync functions. It also adds more atomic types and support for other atomic functions. An special case has been added to support 64-bit atomics on 32-bit systems. The solution is to fallback to the mutex solution only for 64-bit atomics, but smaller atomic types will still take advantage of builtins if available. Change-Id: I6b9afc7cd6e66b28a33278715583552872278801 BUG: 1510397 Signed-off-by: Xavier Hernandez <jahernan>
This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-4.0.0, please open a new bug report. glusterfs-4.0.0 has been announced on the Gluster mailinglists [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution. [1] http://lists.gluster.org/pipermail/announce/2018-March/000092.html [2] https://www.gluster.org/pipermail/gluster-users/