diff options
author | Joel Sherrill <joel.sherrill@OARcorp.com> | 2005-09-01 14:44:04 +0000 |
---|---|---|
committer | Joel Sherrill <joel.sherrill@OARcorp.com> | 2005-09-01 14:44:04 +0000 |
commit | 84b6c0502e63946733036517ca8cb302ce22521a (patch) | |
tree | 784ccb1dab0a1081010116ab79d625c530d4d3d2 /cpukit | |
parent | Added PR number to existing ChangeLog entry. (diff) | |
download | rtems-84b6c0502e63946733036517ca8cb302ce22521a.tar.bz2 |
2005-09-01 Joel Sherrill <joel@OARcorp.com>
PR 796/rtems
* posix/src/semtimedwait.c: sem_timedwait is supposed to use absolute
time for timeout specification. This patch is a modified version of
the one suggested by Peter Dufault.
Diffstat (limited to 'cpukit')
-rw-r--r-- | cpukit/ChangeLog | 7 | ||||
-rw-r--r-- | cpukit/posix/src/semtimedwait.c | 40 |
2 files changed, 41 insertions, 6 deletions
diff --git a/cpukit/ChangeLog b/cpukit/ChangeLog index 93dc8ef7e5..19818f96c4 100644 --- a/cpukit/ChangeLog +++ b/cpukit/ChangeLog @@ -1,3 +1,10 @@ +2005-09-01 Joel Sherrill <joel@OARcorp.com> + + PR 796/rtems + * posix/src/semtimedwait.c: sem_timedwait is supposed to use absolute + time for timeout specification. This patch is a modified version of + the one suggested by Peter Dufault. + 2005-09-01 Nuno Costa <nuno-costa@iol.pt> PR 804 diff --git a/cpukit/posix/src/semtimedwait.c b/cpukit/posix/src/semtimedwait.c index f5904c59fd..6acb704f7c 100644 --- a/cpukit/posix/src/semtimedwait.c +++ b/cpukit/posix/src/semtimedwait.c @@ -29,12 +29,40 @@ int sem_timedwait( sem_t *sem, - const struct timespec *timeout + const struct timespec *abstime ) { - return _POSIX_Semaphore_Wait_support( - sem, - TRUE, - _POSIX_Timespec_to_interval( timeout ) - ); + /* + * The abstime is a walltime. We turn it into an interval. + */ + Watchdog_Interval ticks; + struct timespec current_time; + struct timespec difference; + + /* + * Error check the absolute time to timeout + */ + if ( /* abstime->tv_sec < 0 || */ abstime->tv_nsec ) /* tv_sec is unsigned */ + return EINVAL; + + if ( abstime->tv_nsec >= TOD_NANOSECONDS_PER_SECOND ) + return EINVAL; + + (void) clock_gettime( CLOCK_REALTIME, ¤t_time ); + + /* + * Make sure the abstime is in the future + */ + if ( abstime->tv_sec < current_time.tv_sec ) + return EINVAL; + if ( (abstime->tv_sec == current_time.tv_sec) && + (abstime->tv_nsec <= current_time.tv_nsec) ) + return EINVAL; + + _POSIX_Timespec_subtract( ¤t_time, abstime, &difference ); + + ticks = _POSIX_Timespec_to_interval( &difference ); + + return _POSIX_Semaphore_Wait_support( sem, TRUE, ticks ); } + |