[MDEV-24295] Reduce wakeups by tpool maintenance timer, when server is idle Created: 2020-11-26 Updated: 2021-04-29 Resolved: 2020-11-30 |
|
| Status: | Closed |
| Project: | MariaDB Server |
| Component/s: | Server |
| Affects Version/s: | 10.5 |
| Fix Version/s: | 10.5.9 |
| Type: | Bug | Priority: | Major |
| Reporter: | Vladislav Vaintroub | Assignee: | Vladislav Vaintroub |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Description |
|
monty notes that idle server still wakes up now and then. While most of the wakeups can be attrbiuted to Innodb's periodic timers, there is a single timer which is mostly there to prevent deadlocks, the "maintenance" timer. It currently wakes up every 0.4 second to check if a thread needs to be created or an idle thread needs to be woken. It is there to prevent "stalls". When the database is idle, and stalls are not foreseen in the near future, this timer could be switched off. Due to technical reasons (the underlying mysys timer likes to deadlock for anything else that trivial use), switching off and on is not possible, but there is a workaround that allows to change the timer period (unfortunately, not the next "deadline") for periodic timers, without LOCK_timer lock. Note that since changing next deadline when timer fires won't be possible,as explained above, The timer needs to be switched back on, when there is throttling in thread creation, as otherwise deadlocks are possible, and maybe in some other scenarios |
| Comments |
| Comment by Seth Shelnutt [ 2020-12-06 ] | |||
|
We are seeing an error with our OSX testing from this change still. I know a few fixes went in around this commit but there still seems to be a minor problem. Error:
cmake command:
| |||
| Comment by Marko Mäkelä [ 2021-04-29 ] | |||
|
Shelnutt2, I think that I saw a similar error when compiling with clang++-11 on GNU/Linux, but only when using cmake -DWITH_MSAN. Would the replacement of constexpr with static const work for you? |