Commit 626b2d6a authored by Sunny Bains's avatar Sunny Bains

Fix bug encountered during testing on BSD. In the original fix for bug# 55681

the check for overflow was wrong and the OS_SYNC_INFINITE_TIME was handled
incorrectly.
  
Approved by Jimmy Yang (IM)
parent ac7348ff
...@@ -137,7 +137,7 @@ os_cond_wait_timed( ...@@ -137,7 +137,7 @@ os_cond_wait_timed(
const struct timespec* abstime /*!< in: timeout */ const struct timespec* abstime /*!< in: timeout */
#else #else
ulint time_in_ms /*!< in: timeout in ulint time_in_ms /*!< in: timeout in
milliseconds */ milliseconds*/
#endif /* !__WIN__ */ #endif /* !__WIN__ */
) )
{ {
...@@ -160,7 +160,17 @@ os_cond_wait_timed( ...@@ -160,7 +160,17 @@ os_cond_wait_timed(
ret = pthread_cond_timedwait(cond, mutex, abstime); ret = pthread_cond_timedwait(cond, mutex, abstime);
ut_a(ret == 0 || ret == ETIMEDOUT); switch (ret) {
case 0:
case ETIMEDOUT:
break;
default:
fprintf(stderr, " InnoDB: pthread_cond_timedwait() returned: "
"%d: abstime={%lu,%lu}\n",
ret, abstime->tv_sec, abstime->tv_nsec);
ut_error;
}
return(ret == ETIMEDOUT); return(ret == ETIMEDOUT);
#endif #endif
...@@ -637,14 +647,15 @@ os_event_wait_time_low( ...@@ -637,14 +647,15 @@ os_event_wait_time_low(
ib_int64_t old_signal_count; ib_int64_t old_signal_count;
#ifdef __WIN__ #ifdef __WIN__
DWORD time_in_ms = time_in_usec / 1000; DWORD time_in_ms;
if (!srv_use_native_conditions) { if (!srv_use_native_conditions) {
DWORD err; DWORD err;
ut_a(event); ut_a(event);
if (time_in_ms != OS_SYNC_INFINITE_TIME) { if (time_in_usec != OS_SYNC_INFINITE_TIME) {
time_in_ms = time_in_ms / 1000;
err = WaitForSingleObject(event->handle, time_in_ms); err = WaitForSingleObject(event->handle, time_in_ms);
} else { } else {
err = WaitForSingleObject(event->handle, INFINITE); err = WaitForSingleObject(event->handle, INFINITE);
...@@ -661,13 +672,21 @@ os_event_wait_time_low( ...@@ -661,13 +672,21 @@ os_event_wait_time_low(
return(42); return(42);
} else { } else {
ut_a(sleep_condition_variable != NULL); ut_a(sleep_condition_variable != NULL);
if (time_in_usec != OS_SYNC_INFINITE_TIME) {
time_in_ms = time_in_usec / 1000;
} else {
time_in_ms = INFINITE;
}
} }
#else #else
struct timespec abstime;
if (time_in_usec != OS_SYNC_INFINITE_TIME) {
struct timeval tv; struct timeval tv;
int ret;
ulint sec; ulint sec;
ulint usec; ulint usec;
int ret;
struct timespec abstime;
ret = ut_usectime(&sec, &usec); ret = ut_usectime(&sec, &usec);
ut_a(ret == 0); ut_a(ret == 0);
...@@ -677,14 +696,20 @@ os_event_wait_time_low( ...@@ -677,14 +696,20 @@ os_event_wait_time_low(
tv.tv_usec += time_in_usec; tv.tv_usec += time_in_usec;
if ((ulint) tv.tv_usec > MICROSECS_IN_A_SECOND) { if ((ulint) tv.tv_usec >= MICROSECS_IN_A_SECOND) {
tv.tv_sec += time_in_usec / MICROSECS_IN_A_SECOND; tv.tv_sec += time_in_usec / MICROSECS_IN_A_SECOND;
tv.tv_usec %= MICROSECS_IN_A_SECOND; tv.tv_usec %= MICROSECS_IN_A_SECOND;
} }
/* Convert to nano seconds. We ignore overflow. */
abstime.tv_sec = tv.tv_sec; abstime.tv_sec = tv.tv_sec;
abstime.tv_nsec = tv.tv_usec * 1000; abstime.tv_nsec = tv.tv_usec * 1000;
} else {
abstime.tv_nsec = 999999999;
abstime.tv_sec = (time_t) ULINT_MAX;
}
ut_a(abstime.tv_nsec <= 999999999);
#endif /* __WIN__ */ #endif /* __WIN__ */
os_fast_mutex_lock(&event->os_mutex); os_fast_mutex_lock(&event->os_mutex);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment