Skip to content

Commit cb90c8d

Browse files
committed
Merge tag 'sched-urgent-2025-03-21' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip
Pull scheduler fix from Ingo Molnar: "Revert a scheduler performance optimization that regressed other workloads" * tag 'sched-urgent-2025-03-21' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip: Revert "sched/core: Reduce cost of sched_move_task when config autogroup"
2 parents b3ee1e4 + 76f970c commit cb90c8d

File tree

1 file changed

+3
-18
lines changed

1 file changed

+3
-18
lines changed

kernel/sched/core.c

+3-18
Original file line numberDiff line numberDiff line change
@@ -9016,7 +9016,7 @@ void sched_release_group(struct task_group *tg)
90169016
spin_unlock_irqrestore(&task_group_lock, flags);
90179017
}
90189018

9019-
static struct task_group *sched_get_task_group(struct task_struct *tsk)
9019+
static void sched_change_group(struct task_struct *tsk)
90209020
{
90219021
struct task_group *tg;
90229022

@@ -9028,13 +9028,7 @@ static struct task_group *sched_get_task_group(struct task_struct *tsk)
90289028
tg = container_of(task_css_check(tsk, cpu_cgrp_id, true),
90299029
struct task_group, css);
90309030
tg = autogroup_task_group(tsk, tg);
9031-
9032-
return tg;
9033-
}
9034-
9035-
static void sched_change_group(struct task_struct *tsk, struct task_group *group)
9036-
{
9037-
tsk->sched_task_group = group;
9031+
tsk->sched_task_group = tg;
90389032

90399033
#ifdef CONFIG_FAIR_GROUP_SCHED
90409034
if (tsk->sched_class->task_change_group)
@@ -9055,20 +9049,11 @@ void sched_move_task(struct task_struct *tsk, bool for_autogroup)
90559049
{
90569050
int queued, running, queue_flags =
90579051
DEQUEUE_SAVE | DEQUEUE_MOVE | DEQUEUE_NOCLOCK;
9058-
struct task_group *group;
90599052
struct rq *rq;
90609053

90619054
CLASS(task_rq_lock, rq_guard)(tsk);
90629055
rq = rq_guard.rq;
90639056

9064-
/*
9065-
* Esp. with SCHED_AUTOGROUP enabled it is possible to get superfluous
9066-
* group changes.
9067-
*/
9068-
group = sched_get_task_group(tsk);
9069-
if (group == tsk->sched_task_group)
9070-
return;
9071-
90729057
update_rq_clock(rq);
90739058

90749059
running = task_current_donor(rq, tsk);
@@ -9079,7 +9064,7 @@ void sched_move_task(struct task_struct *tsk, bool for_autogroup)
90799064
if (running)
90809065
put_prev_task(rq, tsk);
90819066

9082-
sched_change_group(tsk, group);
9067+
sched_change_group(tsk);
90839068
if (!for_autogroup)
90849069
scx_cgroup_move_task(tsk);
90859070

0 commit comments

Comments
 (0)