Ticket History #3084: need more thorough testing for conditional pthread support

# Tue May 31 20:36:02 2005 raeburn (Ken Raeburn) - Ticket created

Subject:

need more thorough testing for conditional pthread support

Download (untitled) / with headers
text/plain 486B

When the pthread support may or may not be linked in, either because
we're using weak symbols or because the system C library has stubs for
all the functions we use, we need to do a better job of testing for
*real* pthread support (i.e., not non-functional stubs).

Recommended approach: Test all the symbols we use (and maybe some others
we know of) for null addresses. Try using pthread_once and see if it
works. Only if all tests pass should we assume thread support is available.

# Tue May 31 20:38:33 2005 raeburn (Ken Raeburn) - Status changed from 'new' to 'open'

# Tue May 31 20:38:34 2005 raeburn (Ken Raeburn) - Correspondence added

From:	raeburn@mit.edu
Subject:	CVS Commit

Download (untitled) / with headers
text/plain 204B

* Makefile.in (test_getpw): Depend on, and link against, the support library.

To generate a diff of this commit:

cvs diff -r1.1 -r1.2 krb5/src/tests/misc/ChangeLog
krb5/src/tests/misc/Makefile.in

# Tue May 31 20:56:04 2005 raeburn (Ken Raeburn) - Correspondence added

From:	raeburn@mit.edu
Subject:	CVS Commit

Download (untitled) / with headers
text/plain 787B

Add run-time determination of whether thread support is available in the
support library. Check lots of weak references, and call pthread_once a couple
of times, to distinguish a broken stub from the real thing.

* threads.c [HAVE_PRAGMA_WEAK_REF]: Declare pthread_{g,s}etspecific,
pthread_key_{create,delete}, pthread_{create,join} as weak references.
(krb5int_pthread_loaded, loaded_test_aux) [HAVE_PRAGMA_WEAK_REF]: New
functions.
(flag_pthread_loaded, loaded_test_once) [HAVE_PRAGMA_WEAK_REF]: New variables.
* libkrb5support.exports: Add krb5int_pthread_loaded.

To generate a diff of this commit:

cvs diff -r1.32 -r1.33 krb5/src/util/support/ChangeLog
cvs diff -r1.6 -r1.7 krb5/src/util/support/libkrb5support.exports
cvs diff -r1.19 -r1.20 krb5/src/util/support/threads.c

# Tue May 31 21:57:18 2005 raeburn (Ken Raeburn) - Correspondence added

From:	raeburn@mit.edu
Subject:	CVS Commit

Download (untitled) / with headers
text/plain 271B

* k5-thread.h (krb5int_pthread_loaded) [HAVE_PRAGMA_WEAK_REF]: Declare.
(K5_PTHREADS_LOADED) [HAVE_PRAGMA_WEAK_REF]: Use it.

To generate a diff of this commit:

cvs diff -r1.502 -r1.503 krb5/src/include/ChangeLog
cvs diff -r1.28 -r1.29 krb5/src/include/k5-thread.h

# Wed Jun 01 02:53:42 2005 raeburn (Ken Raeburn) - Comments added

Download (untitled) / with headers
text/plain 288B

KAMADA Ken'ichi confirms this fixes an initialization problem on NetBSD-current (3.99.5),
which has stubs for some pthread_* routines, which stubs weren't in NetBSD 2.0 that I'm
testing with.

Asked for confirmation from Albert Chin as to whether it fixes the same problem on HP-UX
11.

# Wed Jun 01 17:25:51 2005 raeburn (Ken Raeburn) - Component krb5-libs added

# Wed Jun 01 17:25:52 2005 raeburn (Ken Raeburn) - Version_reported 1.4.1 added

# Wed Jun 01 17:25:53 2005 raeburn (Ken Raeburn) - Target_Version 1.4.2 added

# Wed Jun 01 17:25:53 2005 raeburn (Ken Raeburn) - Tags pullup added

# Wed Jun 15 19:36:10 2005 raeburn (Ken Raeburn) - Status changed from 'open' to 'resolved'

# Mon Jul 18 19:23:01 2005 tlyu (Taylor Yu) - Version_Fixed 1.4.2 added

# Mon Jul 18 19:23:01 2005 tlyu (Taylor Yu) - Correspondence added

From:	tlyu@mit.edu
Subject:	CVS Commit

Download (untitled) / with headers
text/plain 383B

pullup from trunk

To generate a diff of this commit:

cvs diff -r1.471.2.13 -r1.471.2.14 krb5/src/include/ChangeLog
cvs diff -r1.22.2.2 -r1.22.2.3 krb5/src/include/k5-thread.h
cvs diff -r1.19.2.4 -r1.19.2.5 krb5/src/util/support/ChangeLog
cvs diff -r1.3 -r1.3.2.1
krb5/src/util/support/libkrb5support.exports
cvs diff -r1.11.4.3 -r1.11.4.4 krb5/src/util/support/threads.c

# Wed Dec 16 18:02:44 2015 tlyu (Taylor Yu) - CustomField pullup deleted