d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Use the correct link entry for walking the list of threads.

Browse Source 
While I'm here, use the TAILQ_FOREACH macro instead of a more
manual method which was inherited from libc_r (so we could
remove elements from the list which isn't needed for libpthread).

Submitted by:	Kazuaki Oda <kaakun@highway.ne.jp>
This commit is contained in:
deischen 2003-04-28 21:35:06 +00:00
parent 389061c125
commit 4d8bc81d7d
2 changed files with 4 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
lib/libkse/thread/thr_sig.c
View File

@ -258,7 +258,7 @@ struct pthread *
thr_sig_find(struct kse *curkse, int sig, siginfo_t *info)
{
int handler_installed;
struct pthread *pthread, *pthread_next;
struct pthread *pthread;
struct pthread *suspended_thread, *signaled_thread;
DBG_MSG("Looking for thread to handle signal %d\n", sig);
@ -285,14 +285,7 @@ thr_sig_find(struct kse *curkse, int sig, siginfo_t *info)
signaled_thread = NULL;
KSE_LOCK_ACQUIRE(curkse, &_thread_list_lock);
for (pthread = TAILQ_FIRST(&_thread_list);
pthread != NULL; pthread = pthread_next) {
/*
* Grab the next thread before possibly destroying
* the link entry.
*/
pthread_next = TAILQ_NEXT(pthread, pqe);
TAILQ_FOREACH(pthread, &_thread_list, tle) {
if ((pthread->state == PS_SIGWAIT) &&
sigismember(pthread->data.sigwait, sig)) {
/* Take the scheduling lock. */

11
lib/libpthread/thread/thr_sig.c
View File

@ -258,7 +258,7 @@ struct pthread *
thr_sig_find(struct kse *curkse, int sig, siginfo_t *info)
{
int handler_installed;
struct pthread *pthread, *pthread_next;
struct pthread *pthread;
struct pthread *suspended_thread, *signaled_thread;
DBG_MSG("Looking for thread to handle signal %d\n", sig);
@ -285,14 +285,7 @@ thr_sig_find(struct kse *curkse, int sig, siginfo_t *info)
signaled_thread = NULL;
KSE_LOCK_ACQUIRE(curkse, &_thread_list_lock);
for (pthread = TAILQ_FIRST(&_thread_list);
pthread != NULL; pthread = pthread_next) {
/*
* Grab the next thread before possibly destroying
* the link entry.
*/
pthread_next = TAILQ_NEXT(pthread, pqe);
TAILQ_FOREACH(pthread, &_thread_list, tle) {
if ((pthread->state == PS_SIGWAIT) &&
sigismember(pthread->data.sigwait, sig)) {
/* Take the scheduling lock. */