Skip to content

Commit

Permalink
MDEV-31939 Adaptive flush recommendation ignores dirty ratio and chec…
Browse files Browse the repository at this point in the history
…kpoint age

buf_flush_page_cleaner(): Pass pct_lwm=srv_max_dirty_pages_pct_lwm
(innodb_max_dirty_pages_pct_lwm) to
page_cleaner_flush_pages_recommendation() unless the dirty page ratio
of the buffer pool is below that. Starting with
commit d4265fb we used to always
pass pct_lwm=0.0, which was not intended.

Reviewed by: Vladislav Vaintroub
  • Loading branch information
dr-m committed Dec 8, 2023
1 parent d7c6d30 commit c8346c0
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions storage/innobase/buf/buf0flu.cc
Expand Up @@ -2379,10 +2379,11 @@ static os_thread_ret_t DECLARE_THREAD(buf_flush_page_cleaner)(void*)
else
{
maybe_unemployed:
const bool below{dirty_pct < pct_lwm};
pct_lwm= 0.0;
if (below)
if (dirty_pct < pct_lwm)
{
pct_lwm= 0.0;
goto possibly_unemployed;
}
}
}
else if (dirty_pct < srv_max_buf_pool_modified_pct)
Expand Down

0 comments on commit c8346c0

Please sign in to comment.