Browse Source

MDEV-27058 fixup: Fix a bogus assertion

buf_page_get_low(): Do not expect a valid state of
buf_page_t::in_zip_hash for blocks that are not file pages.
This debug assertion had been misplaced in
commit aaef2e1d8c (MDEV-27058)
that removed the condition
block->page.state() == BUF_BLOCK_FILE_PAGE.
pull/4194/head
Marko Mäkelä 3 months ago
parent
commit
499fa24d63
  1. 2
      storage/innobase/buf/buf0buf.cc

2
storage/innobase/buf/buf0buf.cc

@ -2730,7 +2730,6 @@ loop:
transactional_shared_lock_guard<page_hash_latch> g{hash_lock};
if (buf_pool.is_uncompressed(block)
&& page_id == block->page.id()) {
ut_ad(!block->page.in_zip_hash);
state = block->page.state();
/* Ignore guesses that point to read-fixed blocks.
We can only avoid a race condition by
@ -2738,6 +2737,7 @@ loop:
if ((state >= buf_page_t::FREED
&& state < buf_page_t::READ_FIX)
|| state >= buf_page_t::WRITE_FIX) {
ut_ad(!block->page.in_zip_hash);
state = block->page.fix();
goto got_block;
}

Loading…
Cancel
Save