mirror of
https://github.com/MariaDB/server.git
synced 2025-01-29 02:05:57 +01:00
InnoDB: Fix compiler warnings and some bad formatting introduced
in ChangeSet@1.1721.48.1 innobase/include/ut0ut.h: Add prototype for ut_usectime() innobase/srv/srv0srv.c: Fix some misformatted code. Remove a compiler warning about possibly uninitialized variable in srv_suspend_mysql_thread(). innobase/sync/sync0sync.c: Remove a compiler warning about possibly uninitialized variable in mutex_spin_wait(). innobase/ut/ut0ut.c: Improve the documentation and formatting of ut_usectime().
This commit is contained in:
parent
509554c3cd
commit
0f8b518bee
4 changed files with 44 additions and 38 deletions
|
@ -139,6 +139,14 @@ ib_time_t
|
|||
ut_time(void);
|
||||
/*=========*/
|
||||
/**************************************************************
|
||||
Returns system time. */
|
||||
|
||||
void
|
||||
ut_usectime(
|
||||
/*========*/
|
||||
ulint* sec, /* out: seconds since the Epoch */
|
||||
ulint* ms); /* out: microseconds since the Epoch+*sec */
|
||||
/**************************************************************
|
||||
Returns the difference of two times in seconds. */
|
||||
|
||||
double
|
||||
|
|
|
@ -346,10 +346,10 @@ static ulint srv_n_rows_updated_old = 0;
|
|||
static ulint srv_n_rows_deleted_old = 0;
|
||||
static ulint srv_n_rows_read_old = 0;
|
||||
|
||||
ulint srv_n_lock_wait_count= 0;
|
||||
ulint srv_n_lock_wait_current_count= 0;
|
||||
ib_longlong srv_n_lock_wait_time= 0;
|
||||
ulint srv_n_lock_max_wait_time= 0;
|
||||
ulint srv_n_lock_wait_count = 0;
|
||||
ulint srv_n_lock_wait_current_count = 0;
|
||||
ib_longlong srv_n_lock_wait_time = 0;
|
||||
ulint srv_n_lock_max_wait_time = 0;
|
||||
|
||||
|
||||
/*
|
||||
|
@ -1384,10 +1384,11 @@ srv_suspend_mysql_thread(
|
|||
trx_t* trx;
|
||||
ibool had_dict_lock = FALSE;
|
||||
ibool was_declared_inside_innodb = FALSE;
|
||||
ib_longlong start_time, finish_time;
|
||||
ulint diff_time;
|
||||
ulint sec;
|
||||
ulint ms;
|
||||
ib_longlong start_time = 0;
|
||||
ib_longlong finish_time;
|
||||
ulint diff_time;
|
||||
ulint sec;
|
||||
ulint ms;
|
||||
|
||||
#ifdef UNIV_SYNC_DEBUG
|
||||
ut_ad(!mutex_own(&kernel_mutex));
|
||||
|
@ -1430,15 +1431,14 @@ srv_suspend_mysql_thread(
|
|||
os_event_reset(event);
|
||||
|
||||
slot->suspend_time = ut_time();
|
||||
if (thr->lock_state == QUE_THR_LOCK_ROW)
|
||||
{
|
||||
srv_n_lock_wait_count++;
|
||||
srv_n_lock_wait_current_count++;
|
||||
|
||||
ut_usectime(&sec, &ms);
|
||||
start_time= (ib_longlong)sec * 1000000 + ms;
|
||||
if (thr->lock_state == QUE_THR_LOCK_ROW) {
|
||||
srv_n_lock_wait_count++;
|
||||
srv_n_lock_wait_current_count++;
|
||||
|
||||
}
|
||||
ut_usectime(&sec, &ms);
|
||||
start_time = (ib_longlong)sec * 1000000 + ms;
|
||||
}
|
||||
/* Wake the lock timeout monitor thread, if it is suspended */
|
||||
|
||||
os_event_set(srv_lock_timeout_thread_event);
|
||||
|
@ -1490,20 +1490,18 @@ srv_suspend_mysql_thread(
|
|||
|
||||
wait_time = ut_difftime(ut_time(), slot->suspend_time);
|
||||
|
||||
if (thr->lock_state == QUE_THR_LOCK_ROW)
|
||||
{
|
||||
ut_usectime(&sec, &ms);
|
||||
finish_time= (ib_longlong)sec * 1000000 + ms;
|
||||
if (thr->lock_state == QUE_THR_LOCK_ROW) {
|
||||
ut_usectime(&sec, &ms);
|
||||
finish_time = (ib_longlong)sec * 1000000 + ms;
|
||||
|
||||
diff_time = finish_time - start_time;
|
||||
|
||||
diff_time= finish_time-start_time;
|
||||
|
||||
srv_n_lock_wait_current_count--;
|
||||
srv_n_lock_wait_time= srv_n_lock_wait_time + diff_time;
|
||||
if (diff_time > srv_n_lock_max_wait_time)
|
||||
{
|
||||
srv_n_lock_max_wait_time= diff_time;
|
||||
}
|
||||
}
|
||||
srv_n_lock_wait_current_count--;
|
||||
srv_n_lock_wait_time = srv_n_lock_wait_time + diff_time;
|
||||
if (diff_time > srv_n_lock_max_wait_time) {
|
||||
srv_n_lock_max_wait_time = diff_time;
|
||||
}
|
||||
}
|
||||
|
||||
if (trx->was_chosen_as_deadlock_victim) {
|
||||
|
||||
|
|
|
@ -368,7 +368,7 @@ mutex_spin_wait(
|
|||
{
|
||||
ulint index; /* index of the reserved wait cell */
|
||||
ulint i; /* spin round count */
|
||||
ib_longlong lstart_time, lfinish_time; /* for timing os_wait */
|
||||
ib_longlong lstart_time = 0, lfinish_time; /* for timing os_wait */
|
||||
ulint ltime_diff;
|
||||
ulint sec;
|
||||
ulint ms;
|
||||
|
|
|
@ -74,18 +74,18 @@ ut_time(void)
|
|||
}
|
||||
|
||||
/**************************************************************
|
||||
Returns system time. We do not specify the format of the time returned:
|
||||
the only way to manipulate it is to use the function ut_difftime. */
|
||||
Returns system time. */
|
||||
|
||||
void
|
||||
ut_usectime(ulint* sec, ulint* ms)
|
||||
/*=========*/
|
||||
ut_usectime(
|
||||
/*========*/
|
||||
ulint* sec, /* out: seconds since the Epoch */
|
||||
ulint* ms) /* out: microseconds since the Epoch+*sec */
|
||||
{
|
||||
struct timeval tv;
|
||||
gettimeofday(&tv,NULL);
|
||||
*sec = (ulint) tv.tv_sec;
|
||||
*ms = (ulint) tv.tv_usec;
|
||||
return;
|
||||
struct timeval tv;
|
||||
gettimeofday(&tv,NULL);
|
||||
*sec = (ulint) tv.tv_sec;
|
||||
*ms = (ulint) tv.tv_usec;
|
||||
}
|
||||
|
||||
/**************************************************************
|
||||
|
|
Loading…
Add table
Reference in a new issue