]> git.baikalelectronics.ru Git - kernel.git/commit
sched/rt: pick_next_rt_entity(): check list_entry
authorPietro Borrello <borrello@diag.uniroma1.it>
Mon, 6 Feb 2023 22:33:54 +0000 (22:33 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 10 Mar 2023 08:32:45 +0000 (09:32 +0100)
commit4474bf8be58e580610889a646d0738ec3ed83d9e
tree5c7da9083e2e22b50349e303065b998dad86a71c
parentb076c36d843a73b8629a122b2ea13bcdab7a5d14
sched/rt: pick_next_rt_entity(): check list_entry

[ Upstream commit 229ce2fe618a0aad9ca6f10358b2c43a496f5fce ]

Commit 8b55ac78e88d ("sched: fix goto retry in pick_next_task_rt()")
removed any path which could make pick_next_rt_entity() return NULL.
However, BUG_ON(!rt_se) in _pick_next_task_rt() (the only caller of
pick_next_rt_entity()) still checks the error condition, which can
never happen, since list_entry() never returns NULL.
Remove the BUG_ON check, and instead emit a warning in the only
possible error condition here: the queue being empty which should
never happen.

Fixes: 8b55ac78e88d ("sched: fix goto retry in pick_next_task_rt()")
Signed-off-by: Pietro Borrello <borrello@diag.uniroma1.it>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Reviewed-by: Phil Auld <pauld@redhat.com>
Reviewed-by: Steven Rostedt (Google) <rostedt@goodmis.org>
Link: https://lore.kernel.org/r/20230128-list-entry-null-check-sched-v3-1-b1a71bd1ac6b@diag.uniroma1.it
Signed-off-by: Sasha Levin <sashal@kernel.org>
kernel/sched/rt.c