[29564] in CVS-changelog-for-Kerberos-V5

home help back first fref pref prev next nref lref last post

krb5 commit: Allow libev to use pthreads on old platforms

daemon@ATHENA.MIT.EDU (Greg Hudson)
Tue Aug 16 12:05:30 2016

Date: Tue, 16 Aug 2016 12:05:26 -0400
From: Greg Hudson <ghudson@mit.edu>
Message-Id: <201608161605.u7GG5Qo6029716@drugstore.mit.edu>
To: cvs-krb5@mit.edu
Reply-To: krbdev@mit.edu
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Errors-To: cvs-krb5-bounces@mit.edu

https://github.com/krb5/krb5/commit/615787708904fd3e472f3be4730cf17c8bba5891
commit 615787708904fd3e472f3be4730cf17c8bba5891
Author: Greg Hudson <ghudson@mit.edu>
Date:   Sun Aug 14 19:00:12 2016 -0400

    Allow libev to use pthreads on old platforms
    
    The upgrade to libev 4.22 introduced the use of "memory fences" for
    more reliable signal processing.  Memory fences are usually
    implemented using assembly or compiler primitives, but may be
    implemented using pthreads as a last resort.  The unmodified libev
    errors out at compile time if pthreads is used, but notes that this
    error can be removed if relying on pthreads is okay.  Because the
    project's nightly build infrastructure includes an old Solaris machine
    whose toolchain is too old for any of the non-pthreads memory fence
    implementations, remove the error to allow the build to succeed.  (A
    dependency on pthreads functions on Solaris does not require linking
    with libpthread.)

 src/util/verto/ev.c |    7 +++++++
 1 files changed, 7 insertions(+), 0 deletions(-)

diff --git a/src/util/verto/ev.c b/src/util/verto/ev.c
index cf55835..87f9067 100644
--- a/src/util/verto/ev.c
+++ b/src/util/verto/ev.c
@@ -1511,8 +1511,15 @@ ecb_binary32_to_binary16 (uint32_t x)
  * alternatively, you can remove this #error and link against libpthread,
  * which will then provide the memory fences.
  */
+/*
+ * krb5 change: per the comment below, we are allowing pthreads on platforms
+ * which are too old to have better memory thead support, as is the case on
+ * older Solaris versions.
+ */
+#if 0
 # error "memory fences not defined for your architecture, please report"
 #endif
+#endif
 
 #ifndef ECB_MEMORY_FENCE
 # define ECB_MEMORY_FENCE do { } while (0)
_______________________________________________
cvs-krb5 mailing list
cvs-krb5@mit.edu
https://mailman.mit.edu/mailman/listinfo/cvs-krb5

home help back first fref pref prev next nref lref last post