diff options
author | Rainer Gerhards <rgerhards@adiscon.com> | 2009-12-02 17:04:04 +0100 |
---|---|---|
committer | Rainer Gerhards <rgerhards@adiscon.com> | 2009-12-02 17:04:04 +0100 |
commit | 113915eb9dfddf5a04f8116b01b71e591e75f193 (patch) | |
tree | 2787b4bcbbe904a80c0e1849762d5ac8071b18bf | |
parent | 8c20add150be92822d2484822b18ead9fd29ac1d (diff) | |
download | rsyslog-113915eb9dfddf5a04f8116b01b71e591e75f193.tar.gz rsyslog-113915eb9dfddf5a04f8116b01b71e591e75f193.tar.xz rsyslog-113915eb9dfddf5a04f8116b01b71e591e75f193.zip |
worked around an issue where omfile failed to compile on32 bit platforms
under some circumstances (this smells like a gcc problem, but a simple
solution was available). Thanks to Kenneth Marshall for some advice.
-rw-r--r-- | ChangeLog | 3 | ||||
-rw-r--r-- | configure.ac | 1 | ||||
-rw-r--r-- | m4/atomic_operations_64bit.m4 | 53 | ||||
-rw-r--r-- | tools/omfile.c | 4 |
4 files changed, 61 insertions, 0 deletions
@@ -1,5 +1,8 @@ --------------------------------------------------------------------------- Version 5.5.2 [DEVEL] (rgerhards), 2009-11-?? +- worked around an issue where omfile failed to compile on32 bit platforms + under some circumstances (this smells like a gcc problem, but a simple + solution was available). Thanks to Kenneth Marshall for some advice. - extended testbench - bugfix: hostname accidently set to IP address for some message sources, for example imudp. Thanks to Anton for reporting this bug. [imported v4] diff --git a/configure.ac b/configure.ac index 08eb24f3..ea3fb1c8 100644 --- a/configure.ac +++ b/configure.ac @@ -128,6 +128,7 @@ AC_TRY_COMPILE([ # check for availability of atomic operations RS_ATOMIC_OPERATIONS +RS_ATOMIC_OPERATIONS_64BIT # Additional module directories diff --git a/m4/atomic_operations_64bit.m4 b/m4/atomic_operations_64bit.m4 new file mode 100644 index 00000000..3121cf15 --- /dev/null +++ b/m4/atomic_operations_64bit.m4 @@ -0,0 +1,53 @@ +# rsyslog +# +# atomic_operations.m4 - autoconf macro to check if compiler supports atomic +# operations +# +# rgerhards, 2008-09-18, added based on +# http://svn.apache.org/repos/asf/apr/apr/trunk/configure.in +# +# +AC_DEFUN([RS_ATOMIC_OPERATIONS_64BIT], +[AC_CACHE_CHECK([whether the compiler provides atomic builtins for 64 bit data types], [ap_cv_atomic_builtins_64], +[AC_TRY_RUN([ +int main() +{ + unsigned long long val = 1010, tmp, *mem = &val; + + if (__sync_fetch_and_add(&val, 1010) != 1010 || val != 2020) + return 1; + + tmp = val; + + if (__sync_fetch_and_sub(mem, 1010) != tmp || val != 1010) + return 1; + + if (__sync_sub_and_fetch(&val, 1010) != 0 || val != 0) + return 1; + + tmp = 3030; + + if (__sync_val_compare_and_swap(mem, 0, tmp) != 0 || val != tmp) + return 1; + + if (__sync_lock_test_and_set(&val, 4040) != 3030) + return 1; + + mem = &tmp; + + if (__sync_val_compare_and_swap(&mem, &tmp, &val) != &tmp) + return 1; + + __sync_synchronize(); + + if (mem != &val) + return 1; + + return 0; +}], [ap_cv_atomic_builtins_64=yes], [ap_cv_atomic_builtins_64=no], [ap_cv_atomic_builtins_64=no])]) + +if test "$ap_cv_atomic_builtins_64" = "yes"; then + AC_DEFINE(HAVE_ATOMIC_BUILTINS_64BIT, 1, [Define if compiler provides 64 bit atomic builtins]) +fi + +]) diff --git a/tools/omfile.c b/tools/omfile.c index 13bb74ad..3a206ebf 100644 --- a/tools/omfile.c +++ b/tools/omfile.c @@ -83,7 +83,11 @@ DEFobjCurrIf(strm) * That should be sufficient (and even than, there would no really bad effect ;)). * The variable below is the global counter/clock. */ +#if HAVE_ATOMIC_BUILTINS_64BIT static uint64 clockFileAccess = 0; +#else +static unsigned clockFileAccess = 0; +#endif /* and the "tick" function */ static inline uint64 getClockFileAccess(void) |