Commit 7750cda1 authored by Marko Mäkelä's avatar Marko Mäkelä

Cleanup: Remove innodb_flush_method (use srv_file_flush_method)

parent 3a427c56
...@@ -187,8 +187,6 @@ static char* innobase_disable_monitor_counter; ...@@ -187,8 +187,6 @@ static char* innobase_disable_monitor_counter;
static char* innobase_reset_monitor_counter; static char* innobase_reset_monitor_counter;
static char* innobase_reset_all_monitor_counter; static char* innobase_reset_all_monitor_counter;
static ulong innodb_flush_method;
/* This variable can be set in the server configure file, specifying /* This variable can be set in the server configure file, specifying
stopword table to be used */ stopword table to be used */
static char* innobase_server_stopword_table; static char* innobase_server_stopword_table;
...@@ -3591,12 +3589,12 @@ static int innodb_init_params() ...@@ -3591,12 +3589,12 @@ static int innodb_init_params()
Force O_DIRECT on Unixes (on Windows writes are always Force O_DIRECT on Unixes (on Windows writes are always
unbuffered) unbuffered)
*/ */
switch (innodb_flush_method) { switch (srv_file_flush_method) {
case SRV_O_DIRECT: case SRV_O_DIRECT:
case SRV_O_DIRECT_NO_FSYNC: case SRV_O_DIRECT_NO_FSYNC:
break; break;
default: default:
innodb_flush_method = SRV_O_DIRECT; srv_file_flush_method = SRV_O_DIRECT;
fprintf(stderr, "InnoDB: using O_DIRECT due to atomic writes.\n"); fprintf(stderr, "InnoDB: using O_DIRECT due to atomic writes.\n");
} }
} }
...@@ -3615,21 +3613,19 @@ static int innodb_init_params() ...@@ -3615,21 +3613,19 @@ static int innodb_init_params()
#endif #endif
#ifndef _WIN32 #ifndef _WIN32
ut_ad(innodb_flush_method <= SRV_O_DIRECT_NO_FSYNC); ut_ad(srv_file_flush_method <= SRV_O_DIRECT_NO_FSYNC);
#else #else
switch (innodb_flush_method) { switch (srv_file_flush_method) {
case SRV_ALL_O_DIRECT_FSYNC + 1 /* "async_unbuffered"="unbuffered" */: case SRV_ALL_O_DIRECT_FSYNC + 1 /* "async_unbuffered"="unbuffered" */:
innodb_flush_method = SRV_ALL_O_DIRECT_FSYNC; srv_file_flush_method = SRV_ALL_O_DIRECT_FSYNC;
break; break;
case SRV_ALL_O_DIRECT_FSYNC + 2 /* "normal"="fsync" */: case SRV_ALL_O_DIRECT_FSYNC + 2 /* "normal"="fsync" */:
innodb_flush_method = SRV_FSYNC; srv_file_flush_method = SRV_FSYNC;
break; break;
default: default:
ut_ad(innodb_flush_method <= SRV_ALL_O_DIRECT_FSYNC); ut_ad(srv_file_flush_method <= SRV_ALL_O_DIRECT_FSYNC);
} }
#endif #endif
srv_file_flush_method = srv_flush_t(innodb_flush_method);
innodb_buffer_pool_size_init(); innodb_buffer_pool_size_init();
srv_lock_table_size = 5 * (srv_buf_pool_size >> srv_page_size_shift); srv_lock_table_size = 5 * (srv_buf_pool_size >> srv_page_size_shift);
...@@ -18376,7 +18372,7 @@ static MYSQL_SYSVAR_ULONG(flush_log_at_trx_commit, srv_flush_log_at_trx_commit, ...@@ -18376,7 +18372,7 @@ static MYSQL_SYSVAR_ULONG(flush_log_at_trx_commit, srv_flush_log_at_trx_commit,
" guarantees in case of crash. 0 and 2 can be faster than 1 or 3.", " guarantees in case of crash. 0 and 2 can be faster than 1 or 3.",
NULL, NULL, 1, 0, 3, 0); NULL, NULL, 1, 0, 3, 0);
static MYSQL_SYSVAR_ENUM(flush_method, innodb_flush_method, static MYSQL_SYSVAR_ENUM(flush_method, srv_file_flush_method,
PLUGIN_VAR_RQCMDARG | PLUGIN_VAR_READONLY, PLUGIN_VAR_RQCMDARG | PLUGIN_VAR_READONLY,
"With which method to flush data.", "With which method to flush data.",
NULL, NULL, IF_WIN(SRV_ALL_O_DIRECT_FSYNC, SRV_O_DIRECT), NULL, NULL, IF_WIN(SRV_ALL_O_DIRECT_FSYNC, SRV_O_DIRECT),
......
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