Commit d4342702 authored by Marko Mäkelä's avatar Marko Mäkelä

Remove dead references to NO_FALLOCATE.

parent cbf80b0d
...@@ -484,7 +484,7 @@ fil_space_is_flushed( ...@@ -484,7 +484,7 @@ fil_space_is_flushed(
return(true); return(true);
} }
#if !defined(NO_FALLOCATE) && defined(UNIV_LINUX) #ifdef UNIV_LINUX
#include <sys/ioctl.h> #include <sys/ioctl.h>
/** FusionIO atomic write control info */ /** FusionIO atomic write control info */
...@@ -511,7 +511,7 @@ fil_fusionio_enable_atomic_write(os_file_t file) ...@@ -511,7 +511,7 @@ fil_fusionio_enable_atomic_write(os_file_t file)
return(false); return(false);
} }
#endif /* !NO_FALLOCATE && UNIV_LINUX */ #endif /* UNIV_LINUX */
/** Append a file to the chain of files of a space. /** Append a file to the chain of files of a space.
@param[in] name file name of a file that is not open @param[in] name file name of a file that is not open
...@@ -3520,11 +3520,10 @@ fil_ibd_create( ...@@ -3520,11 +3520,10 @@ fil_ibd_create(
return(DB_ERROR); return(DB_ERROR);
} }
bool atomic_write; #ifdef UNIV_LINUX
const bool atomic_write = fil_fusionio_enable_atomic_write(file);
#if !defined(NO_FALLOCATE) && defined(UNIV_LINUX)
if (fil_fusionio_enable_atomic_write(file)) {
if (atomic_write) {
/* This is required by FusionIO HW/Firmware */ /* This is required by FusionIO HW/Firmware */
int ret = posix_fallocate(file, 0, size * UNIV_PAGE_SIZE); int ret = posix_fallocate(file, 0, size * UNIV_PAGE_SIZE);
...@@ -3547,21 +3546,14 @@ fil_ibd_create( ...@@ -3547,21 +3546,14 @@ fil_ibd_create(
} else { } else {
success = true; success = true;
} }
} else
atomic_write = true; #else
} else { const bool atomic_write = false;
atomic_write = false; #endif /* UNIV_LINUX */
{
success = os_file_set_size( success = os_file_set_size(
path, file, size * UNIV_PAGE_SIZE, srv_read_only_mode); path, file, size * UNIV_PAGE_SIZE, srv_read_only_mode);
} }
#else
atomic_write = false;
success = os_file_set_size(
path, file, size * UNIV_PAGE_SIZE, srv_read_only_mode);
#endif /* !NO_FALLOCATE && UNIV_LINUX */
if (!success) { if (!success) {
os_file_close(file); os_file_close(file);
...@@ -3913,18 +3905,13 @@ fil_ibd_open( ...@@ -3913,18 +3905,13 @@ fil_ibd_open(
df_dict.close(); df_dict.close();
} }
bool atomic_write; #ifdef UNIV_LINUX
const bool atomic_write = !srv_use_doublewrite_buf
#if !defined(NO_FALLOCATE) && defined(UNIV_LINUX) && df_default.is_open()
if (!srv_use_doublewrite_buf && df_default.is_open()) { && fil_fusionio_enable_atomic_write(df_default.handle());
atomic_write = fil_fusionio_enable_atomic_write(
df_default.handle());
} else {
atomic_write = false;
}
#else #else
atomic_write = false; const bool atomic_write = false;
#endif /* !NO_FALLOCATE && UNIV_LINUX */ #endif /* UNIV_LINUX */
/* We have now checked all possible tablespace locations and /* We have now checked all possible tablespace locations and
have a count of how many unique files we found. If things are have a count of how many unique files we found. If things are
...@@ -5052,7 +5039,7 @@ fil_space_extend( ...@@ -5052,7 +5039,7 @@ fil_space_extend(
ut_ad(len > 0); ut_ad(len > 0);
const char* name = node->name == NULL ? space->name : node->name; const char* name = node->name == NULL ? space->name : node->name;
#if !defined(NO_FALLOCATE) && defined(UNIV_LINUX) #ifdef UNIV_LINUX
/* This is required by FusionIO HW/Firmware */ /* This is required by FusionIO HW/Firmware */
int ret = posix_fallocate(node->handle, node_start, len); int ret = posix_fallocate(node->handle, node_start, len);
...@@ -5077,7 +5064,7 @@ fil_space_extend( ...@@ -5077,7 +5064,7 @@ fil_space_extend(
err = DB_IO_ERROR; err = DB_IO_ERROR;
} }
#endif /* NO_FALLOCATE || !UNIV_LINUX */ #endif
if (!node->atomic_write || err == DB_IO_ERROR) { if (!node->atomic_write || err == DB_IO_ERROR) {
......
...@@ -122,18 +122,12 @@ Tablespace::open_or_create(bool is_temp) ...@@ -122,18 +122,12 @@ Tablespace::open_or_create(bool is_temp)
break; break;
} }
bool atomic_write; #ifdef UNIV_LINUX
const bool atomic_write = fil_fusionio_enable_atomic_write(
#if !defined(NO_FALLOCATE) && defined(UNIV_LINUX) it->m_handle);
if (!srv_use_doublewrite_buf) {
atomic_write = fil_fusionio_enable_atomic_write(
it->m_handle);
} else {
atomic_write = false;
}
#else #else
atomic_write = false; const bool atomic_write = false;
#endif /* !NO_FALLOCATE && UNIV_LINUX */ #endif
/* We can close the handle now and open the tablespace /* We can close the handle now and open the tablespace
the proper way. */ the proper way. */
......
...@@ -909,28 +909,24 @@ SysTablespace::open_or_create( ...@@ -909,28 +909,24 @@ SysTablespace::open_or_create(
return(err); return(err);
} }
#if !defined(NO_FALLOCATE) && defined(UNIV_LINUX) #ifdef UNIV_LINUX
/* Note: This should really be per node and not per /* Note: This should really be per node and not per
tablespace because a tablespace can contain multiple tablespace because a tablespace can contain multiple
files (nodes). The implication is that all files of files (nodes). The implication is that all files of
the tablespace should be on the same medium. */ the tablespace should be on the same medium. */
if (fil_fusionio_enable_atomic_write(it->m_handle)) { it->m_atomic_write
= fil_fusionio_enable_atomic_write(it->m_handle);
if (srv_use_doublewrite_buf) { if (it->m_atomic_write && srv_use_doublewrite_buf) {
ib::info() << "FusionIO atomic IO enabled," ib::info() << "FusionIO atomic IO enabled,"
" disabling the double write buffer"; " disabling the double write buffer";
srv_use_doublewrite_buf = false; srv_use_doublewrite_buf = false;
}
it->m_atomic_write = true;
} else {
it->m_atomic_write = false;
} }
#else #else
it->m_atomic_write = false; it->m_atomic_write = false;
#endif /* !NO_FALLOCATE && UNIV_LINUX*/ #endif
} }
if (!create_new_db && flush_lsn) { if (!create_new_db && flush_lsn) {
......
...@@ -1799,14 +1799,14 @@ fil_names_clear( ...@@ -1799,14 +1799,14 @@ fil_names_clear(
lsn_t lsn, lsn_t lsn,
bool do_write); bool do_write);
#if !defined(NO_FALLOCATE) && defined(UNIV_LINUX) #ifdef UNIV_LINUX
/** /**
Try and enable FusionIO atomic writes. Try and enable FusionIO atomic writes.
@param[in] file OS file handle @param[in] file OS file handle
@return true if successful */ @return true if successful */
bool bool
fil_fusionio_enable_atomic_write(os_file_t file); fil_fusionio_enable_atomic_write(os_file_t file);
#endif /* !NO_FALLOCATE && UNIV_LINUX */ #endif /* UNIV_LINUX */
/** Note that the file system where the file resides doesn't support PUNCH HOLE /** Note that the file system where the file resides doesn't support PUNCH HOLE
@param[in,out] node Node to set */ @param[in,out] node Node to set */
......
...@@ -681,17 +681,12 @@ srv_undo_tablespace_open( ...@@ -681,17 +681,12 @@ srv_undo_tablespace_open(
os_offset_t size; os_offset_t size;
fil_space_t* space; fil_space_t* space;
bool atomic_write; #ifdef UNIV_LINUX
const bool atomic_write = !srv_use_doublewrite_buf
#if !defined(NO_FALLOCATE) && defined(UNIV_LINUX) && fil_fusionio_enable_atomic_write(fh);
if (!srv_use_doublewrite_buf) {
atomic_write = fil_fusionio_enable_atomic_write(fh);
} else {
atomic_write = false;
}
#else #else
atomic_write = false; const bool atomic_write = false;
#endif /* !NO_FALLOCATE && UNIV_LINUX */ #endif
size = os_file_get_size(fh); size = os_file_get_size(fh);
ut_a(size != (os_offset_t) -1); ut_a(size != (os_offset_t) -1);
......
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