diff options
author | Benjamin Segall <bsegall@google.com> | 2024-10-25 18:35:35 -0700 |
---|---|---|
committer | Thomas Gleixner <tglx@linutronix.de> | 2024-10-27 10:36:04 +0100 |
commit | b5413156bad91dc2995a5c4eab1b05e56914638a (patch) | |
tree | 9da21dec930a7921db7e2a35e549d68cb98fba4f /kernel/workqueue.c | |
parent | 42f7652d3eb527d03665b09edac47f85fb600924 (diff) |
posix-cpu-timers: Clear TICK_DEP_BIT_POSIX_TIMER on clone
When cloning a new thread, its posix_cputimers are not inherited, and
are cleared by posix_cputimers_init(). However, this does not clear the
tick dependency it creates in tsk->tick_dep_mask, and the handler does
not reach the code to clear the dependency if there were no timers to
begin with.
Thus if a thread has a cputimer running before clone/fork, all
descendants will prevent nohz_full unless they create a cputimer of
their own.
Fix this by entirely clearing the tick_dep_mask in copy_process().
(There is currently no inherited state that needs a tick dependency)
Process-wide timers do not have this problem because fork does not copy
signal_struct as a baseline, it creates one from scratch.
Fixes: b78783000d5c ("posix-cpu-timers: Migrate to use new tick dependency mask model")
Signed-off-by: Ben Segall <bsegall@google.com>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Reviewed-by: Frederic Weisbecker <frederic@kernel.org>
Cc: stable@vger.kernel.org
Link: https://lore.kernel.org/all/xm26o737bq8o.fsf@google.com
Diffstat (limited to 'kernel/workqueue.c')
0 files changed, 0 insertions, 0 deletions