summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--storage/innobase/buf/buf0buf.cc24
-rw-r--r--storage/innobase/include/fil0fil.ic4
-rw-r--r--storage/xtradb/buf/buf0buf.cc24
-rw-r--r--storage/xtradb/include/fil0fil.ic4
4 files changed, 6 insertions, 50 deletions
diff --git a/storage/innobase/buf/buf0buf.cc b/storage/innobase/buf/buf0buf.cc
index ae73c3ce9c5..2d2b3236a65 100644
--- a/storage/innobase/buf/buf0buf.cc
+++ b/storage/innobase/buf/buf0buf.cc
@@ -5708,9 +5708,7 @@ buf_page_encrypt_before_write(
fil_space_crypt_t* crypt_data = fil_space_get_crypt_data(space_id);
bpage->real_size = UNIV_PAGE_SIZE;
-#ifdef UNIV_DEBUG
fil_page_type_validate(src_frame);
-#endif
if (crypt_data != NULL && crypt_data->encryption == FIL_SPACE_ENCRYPTION_OFF) {
/* Encryption is disabled */
@@ -5767,9 +5765,7 @@ buf_page_encrypt_before_write(
bpage->key_version = key_version;
bpage->real_size = page_size;
-#ifdef UNIV_DEBUG
fil_page_type_validate(dst_frame);
-#endif
} else {
/* First we compress the page content */
@@ -5783,18 +5779,12 @@ buf_page_encrypt_before_write(
fil_space_get_page_compression_level(bpage->space),
block_size,
&out_len,
-#ifdef HAVE_LZO
- slot->lzo_mem
-#else
- NULL
-#endif
+ IF_LZO(slot->lzo_mem, NULL)
);
bpage->real_size = out_len;
-#ifdef UNIV_DEBUG
fil_page_type_validate(tmp);
-#endif
/* And then we encrypt the page content */
fil_space_encrypt(bpage->space,
@@ -5805,9 +5795,7 @@ buf_page_encrypt_before_write(
dst_frame);
}
-#ifdef UNIV_DEBUG
fil_page_type_validate(dst_frame);
-#endif
// return dst_frame which will be written
return dst_frame;
@@ -5847,9 +5835,7 @@ buf_page_decrypt_after_read(
/* Find free slot from temporary memory array */
buf_tmp_buffer_t* slot = buf_pool_reserve_tmp_slot(buf_pool, page_compressed);
-#ifdef UNIV_DEBUG
fil_page_type_validate(dst_frame);
-#endif
fil_decompress_page(slot->comp_buf,
dst_frame,
@@ -5860,9 +5846,7 @@ buf_page_decrypt_after_read(
slot->reserved = false;
}
-#ifdef UNIV_DEBUG
fil_page_type_validate(dst_frame);
-#endif
} else {
/* the page we read is encrypted */
/* Find free slot from temporary memory array */
@@ -5870,10 +5854,8 @@ buf_page_decrypt_after_read(
memcpy(slot->crypt_buf, dst_frame, size);
-#ifdef UNIV_DEBUG
fil_page_type_validate(dst_frame);
fil_page_type_validate(slot->crypt_buf);
-#endif
/* decrypt from crypt_buf to dst_frame */
fil_space_decrypt(bpage->space,
@@ -5881,10 +5863,8 @@ buf_page_decrypt_after_read(
size,
dst_frame);
-#ifdef UNIV_DEBUG
fil_page_type_validate(dst_frame);
fil_page_type_validate(slot->crypt_buf);
-#endif
/* decompress from dst_frame to comp_buf and then copy to
buffer pool */
@@ -5895,9 +5875,7 @@ buf_page_decrypt_after_read(
&bpage->write_size);
}
-#ifdef UNIV_DEBUG
fil_page_type_validate(dst_frame);
-#endif
/* Mark this slot as free */
slot->reserved = false;
diff --git a/storage/innobase/include/fil0fil.ic b/storage/innobase/include/fil0fil.ic
index 6ec7c1adec8..d00b01097ea 100644
--- a/storage/innobase/include/fil0fil.ic
+++ b/storage/innobase/include/fil0fil.ic
@@ -105,7 +105,6 @@ fil_node_get_block_size(
return (node->file_block_size);
}
-#ifdef UNIV_DEBUG
/****************************************************************//**
Validate page type.
@return true if valid, false if not */
@@ -114,6 +113,7 @@ bool
fil_page_type_validate(
const byte* page) /*!< in: page */
{
+#ifdef UNIV_DEBUG
ulint page_type = mach_read_from_2(page + FIL_PAGE_TYPE);
#ifdef UNIV_ENCRYPTION_EXTRA_DEBUG
uint key_version = mach_read_from_4(page + FIL_PAGE_FILE_FLUSH_LSN_OR_KEY_VERSION);
@@ -167,8 +167,8 @@ fil_page_type_validate(
return false;
}
+#endif /* UNIV_DEBUG */
return true;
}
-#endif /* UNIV_DEBUG */
#endif /* fil0fil_ic */
diff --git a/storage/xtradb/buf/buf0buf.cc b/storage/xtradb/buf/buf0buf.cc
index 86ac8c14ac2..de557b8b469 100644
--- a/storage/xtradb/buf/buf0buf.cc
+++ b/storage/xtradb/buf/buf0buf.cc
@@ -5867,9 +5867,7 @@ buf_page_encrypt_before_write(
fil_space_crypt_t* crypt_data = fil_space_get_crypt_data(space_id);
bpage->real_size = UNIV_PAGE_SIZE;
-#ifdef UNIV_DEBUG
fil_page_type_validate(src_frame);
-#endif
if (crypt_data != NULL && crypt_data->encryption == FIL_SPACE_ENCRYPTION_OFF) {
/* Encryption is disabled */
@@ -5926,9 +5924,7 @@ buf_page_encrypt_before_write(
bpage->key_version = key_version;
bpage->real_size = page_size;
-#ifdef UNIV_DEBUG
fil_page_type_validate(dst_frame);
-#endif
} else {
/* First we compress the page content */
@@ -5942,18 +5938,12 @@ buf_page_encrypt_before_write(
fil_space_get_page_compression_level(bpage->space),
block_size,
&out_len,
-#ifdef HAVE_LZO
- slot->lzo_mem
-#else
- NULL
-#endif
+ IF_LZO(slot->lzo_mem, NULL)
);
bpage->real_size = out_len;
-#ifdef UNIV_DEBUG
fil_page_type_validate(tmp);
-#endif
/* And then we encrypt the page content */
fil_space_encrypt(bpage->space,
@@ -5964,9 +5954,7 @@ buf_page_encrypt_before_write(
dst_frame);
}
-#ifdef UNIV_DEBUG
fil_page_type_validate(dst_frame);
-#endif
// return dst_frame which will be written
return dst_frame;
@@ -6003,9 +5991,7 @@ buf_page_decrypt_after_read(
/* Find free slot from temporary memory array */
buf_tmp_buffer_t* slot = buf_pool_reserve_tmp_slot(buf_pool, page_compressed);
-#ifdef UNIV_DEBUG
fil_page_type_validate(dst_frame);
-#endif
fil_decompress_page(slot->comp_buf,
dst_frame,
@@ -6016,18 +6002,14 @@ buf_page_decrypt_after_read(
slot->reserved = false;
}
-#ifdef UNIV_DEBUG
fil_page_type_validate(dst_frame);
-#endif
} else {
/* Find free slot from temporary memory array */
buf_tmp_buffer_t* slot = buf_pool_reserve_tmp_slot(buf_pool, page_compressed);
memcpy(slot->crypt_buf, dst_frame, size);
-#ifdef UNIV_DEBUG
fil_page_type_validate(dst_frame);
fil_page_type_validate(slot->crypt_buf);
-#endif
/* decrypt from crypt_buf to dst_frame */
fil_space_decrypt(bpage->space,
@@ -6035,10 +6017,8 @@ buf_page_decrypt_after_read(
size,
dst_frame);
-#ifdef UNIV_DEBUG
fil_page_type_validate(dst_frame);
fil_page_type_validate(slot->crypt_buf);
-#endif
if (page_compressed) {
fil_decompress_page(slot->comp_buf,
@@ -6046,9 +6026,7 @@ buf_page_decrypt_after_read(
size,
&bpage->write_size);
-#ifdef UNIV_DEBUG
fil_page_type_validate(dst_frame);
-#endif
}
/* Mark this slot as free */
diff --git a/storage/xtradb/include/fil0fil.ic b/storage/xtradb/include/fil0fil.ic
index cfb82aba31b..8b3809594aa 100644
--- a/storage/xtradb/include/fil0fil.ic
+++ b/storage/xtradb/include/fil0fil.ic
@@ -105,7 +105,6 @@ fil_node_get_block_size(
return (node->file_block_size);
}
-#ifdef UNIV_DEBUG
/****************************************************************//**
Validate page type.
@return true if valid, false if not */
@@ -114,6 +113,7 @@ bool
fil_page_type_validate(
const byte* page) /*!< in: page */
{
+#ifdef UNIV_DEBUG
ulint page_type = mach_read_from_2(page + FIL_PAGE_TYPE);
#ifdef UNIV_ENCRYPTION_EXTRA_DEBUG
uint key_version = mach_read_from_4(page + FIL_PAGE_FILE_FLUSH_LSN_OR_KEY_VERSION);
@@ -168,8 +168,8 @@ fil_page_type_validate(
return false;
}
+#endif /* UNIV_DEBUG */
return true;
}
-#endif /* UNIV_DEBUG */
#endif /* fil0fil_ic */