Commit 7bf45bec authored by Jan Lindström's avatar Jan Lindström

Fix too agressive long semaphore wait output and add guard against introducing

compression failures on insert buffer.
parent 06655a19
...@@ -159,6 +159,7 @@ INSERT INTO global_suppressions VALUES ...@@ -159,6 +159,7 @@ INSERT INTO global_suppressions VALUES
("InnoDB: Error: in ALTER TABLE `test`.`t[123]`"), ("InnoDB: Error: in ALTER TABLE `test`.`t[123]`"),
("InnoDB: Error: in RENAME TABLE table `test`.`t1`"), ("InnoDB: Error: in RENAME TABLE table `test`.`t1`"),
("InnoDB: Error: table `test`.`t[123]` .*does not exist in the InnoDB internal"), ("InnoDB: Error: table `test`.`t[123]` .*does not exist in the InnoDB internal"),
("InnoDB: Warning: semaphore wait:"),
/* /*
BUG#32080 - Excessive warnings on Solaris: setrlimit could not BUG#32080 - Excessive warnings on Solaris: setrlimit could not
......
...@@ -1272,6 +1272,7 @@ page_zip_compress( ...@@ -1272,6 +1272,7 @@ page_zip_compress(
records. */ records. */
if (srv_simulate_comp_failures if (srv_simulate_comp_failures
&& !dict_index_is_ibuf(index)
&& page_get_n_recs(page) >= 2 && page_get_n_recs(page) >= 2
&& ((ulint)(rand() % 100) < srv_simulate_comp_failures)) { && ((ulint)(rand() % 100) < srv_simulate_comp_failures)) {
......
...@@ -971,11 +971,39 @@ sync_array_print_long_waits( ...@@ -971,11 +971,39 @@ sync_array_print_long_waits(
stderr); stderr);
sync_array_cell_print(stderr, cell, &reserver); sync_array_cell_print(stderr, cell, &reserver);
noticed = TRUE; noticed = TRUE;
} else { }
if (diff > fatal_timeout) {
fatal = TRUE;
}
if (diff > longest_diff) {
longest_diff = diff;
*sema = wait_object;
*waiter = cell->thread;
}
}
/* We found a long semaphore wait, wait all threads that are
waiting for a semaphore. */
if (noticed) {
for (i = 0; i < sync_primary_wait_array->n_cells; i++) {
void* wait_object;
os_thread_id_t reserver=0;
cell = sync_array_get_nth_cell(sync_primary_wait_array, i);
wait_object = cell->wait_object;
if (wait_object == NULL || !cell->waiting) {
continue;
}
fputs("InnoDB: Warning: semaphore wait:\n", fputs("InnoDB: Warning: semaphore wait:\n",
stderr); stderr);
sync_array_cell_print(stderr, cell, &reserver); sync_array_cell_print(stderr, cell, &reserver);
} noticed = TRUE;
/* Try to output cell information for writer recursive way */ /* Try to output cell information for writer recursive way */
while (reserver != 0) { while (reserver != 0) {
...@@ -993,15 +1021,6 @@ sync_array_print_long_waits( ...@@ -993,15 +1021,6 @@ sync_array_print_long_waits(
reserver = 0; reserver = 0;
} }
} }
if (diff > fatal_timeout) {
fatal = TRUE;
}
if (diff > longest_diff) {
longest_diff = diff;
*sema = wait_object;
*waiter = cell->thread;
} }
} }
......
...@@ -1276,6 +1276,7 @@ page_zip_compress( ...@@ -1276,6 +1276,7 @@ page_zip_compress(
records. */ records. */
if (srv_simulate_comp_failures if (srv_simulate_comp_failures
&& !dict_index_is_ibuf(index)
&& page_get_n_recs(page) >= 2 && page_get_n_recs(page) >= 2
&& ((ulint)(rand() % 100) < srv_simulate_comp_failures)) { && ((ulint)(rand() % 100) < srv_simulate_comp_failures)) {
......
...@@ -995,11 +995,39 @@ sync_array_print_long_waits( ...@@ -995,11 +995,39 @@ sync_array_print_long_waits(
stderr); stderr);
sync_array_cell_print(stderr, cell, &reserver); sync_array_cell_print(stderr, cell, &reserver);
noticed = TRUE; noticed = TRUE;
} else { }
if (diff > fatal_timeout) {
fatal = TRUE;
}
if (diff > longest_diff) {
longest_diff = diff;
*sema = wait_object;
*waiter = cell->thread;
}
}
/* We found a long semaphore wait, wait all threads that are
waiting for a semaphore. */
if (noticed) {
for (i = 0; i < sync_primary_wait_array->n_cells; i++) {
void* wait_object;
os_thread_id_t reserver=0;
cell = sync_array_get_nth_cell(sync_primary_wait_array, i);
wait_object = cell->wait_object;
if (wait_object == NULL || !cell->waiting) {
continue;
}
fputs("InnoDB: Warning: semaphore wait:\n", fputs("InnoDB: Warning: semaphore wait:\n",
stderr); stderr);
sync_array_cell_print(stderr, cell, &reserver); sync_array_cell_print(stderr, cell, &reserver);
} noticed = TRUE;
/* Try to output cell information for writer recursive way */ /* Try to output cell information for writer recursive way */
while (reserver != 0) { while (reserver != 0) {
...@@ -1017,15 +1045,6 @@ sync_array_print_long_waits( ...@@ -1017,15 +1045,6 @@ sync_array_print_long_waits(
reserver = 0; reserver = 0;
} }
} }
if (diff > fatal_timeout) {
fatal = TRUE;
}
if (diff > longest_diff) {
longest_diff = diff;
*sema = wait_object;
*waiter = cell->thread;
} }
} }
......
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