From 38161ac76efe4c50f13e244903a44645023fec83 Mon Sep 17 00:00:00 2001 From: Ulrich Drepper Date: Wed, 14 Jun 2000 06:13:45 +0000 Subject: Update. 2000-06-13 Kaz Kylheku A few optimizations. Got rid of unnecessary wakeups of timer threads, tightened up some critical regions and micro-optimized some list manipulation code. * sysdeps/pthread/timer_routines.c (__timer_thread_queue_timer): Returns int value now to indicate whether timer was queued at head. * sysdeps/pthread/posix-timer.h: Likewise. * sysdeps/pthread/timer_settime.c (timer_settime): Takes advantage of new return value from __timer_thread_queue_timer to avoid waking up timer thread unnecessarily. * sysdeps/pthread/posix-timer.h (timer_id2ptr): No longer checks inuse flag, because this requires mutex to be held. Callers updated to do the check when they have the mutex. * sysdeps/pthread/timer_getoverr.c: Add check for inuse here. * sysdeps/pthread/timer_settime.c (timer_settime): Tighter critical regions: avoids making system calls while holding timer mutex, and a few computations were moved outside of the mutex as well. * sysdeps/pthread/timer_gettime.c (timer_gettime): Likewise. * sysdeps/pthread/posix-timer.h (list_unlink_ip): Function name changed to list_unlink_ip, meaning idempotent. Pointer manipulation changed to get better better code out of gcc. * sysdeps/pthread/timer_routines.c (list_unlink): Non-idempotent version of list_unlink added here. * sysdeps/pthread/timer_delete.c: Use appropriate list unlink function in all places: idempotent one for timers, non-idempotent one for thread nodes. * sysdeps/pthread/timer_settime: Likewise. * sysdeps/pthread/timer_routines.c: Likewise. --- linuxthreads/sysdeps/pthread/timer_getoverr.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) (limited to 'linuxthreads/sysdeps/pthread/timer_getoverr.c') diff --git a/linuxthreads/sysdeps/pthread/timer_getoverr.c b/linuxthreads/sysdeps/pthread/timer_getoverr.c index f3aee45..8630f57 100644 --- a/linuxthreads/sysdeps/pthread/timer_getoverr.c +++ b/linuxthreads/sysdeps/pthread/timer_getoverr.c @@ -29,16 +29,15 @@ int timer_getoverrun (timerid) timer_t timerid; { + struct timer_node *timer; int retval = -1; pthread_mutex_lock (&__timer_mutex); - if (timer_id2ptr (timerid) == NULL) + if ((timer = timer_id2ptr (timerid)) == NULL || !timer->inuse) errno = EINVAL; else - { - retval = 0; /* TODO: overrun counting not supported */ - } + retval = 0; /* TODO: overrun counting not supported */ pthread_mutex_lock (&__timer_mutex); -- cgit v1.1