Re: [PATCH 06/10] sched_ext: Delegate proxy donor admission to BPF schedulers

John Stultz <[email protected]>
Newsgroups dev.linux.lists.sched-ext,org.kernel.vger.linux-kernel
Message-ID <CANDhNCr2eTjjvFv-iNwD-XpStibW_NwyR7Zg+5Zh20mEuPsQ1w@mail.gmail.com>
On Fri, Jul 10, 2026 at 1:40 AM Andrea Righi <[email protected]> wrote:
> @@ -2866,18 +2938,12 @@ static void put_prev_task_scx(struct rq *rq, struct task_struct *p,
>                 set_task_runnable(rq, p);
>
>                 /*
> -                * Mutex-blocked donors stay queued on the runqueue under proxy
> -                * execution, but the donor never runs as itself, proxy-exec
> -                * walks the blocked_on chain on the next __schedule() and runs
> -                * the lock owner in its place.
> -                *
> -                * Put the donor on the local DSQ directly so pick_next_task()
> -                * can still see it. find_proxy_task() will either run the chain
> -                * owner or deactivate the donor so the wakeup path can return it
> -                * and let BPF make a new dispatch decision once it is unblocked.
> +                * Mutex-blocked donors only stay queued when their BPF scheduler
> +                * enables %SCX_OPS_ENQ_BLOCKED, so always delegate their admission.
>                  */
>                 if (task_is_blocked(p)) {
> -                       scx_dispatch_enqueue(sch, rq, &rq->scx.local_dsq, p, 0);
> +                       WARN_ON_ONCE(!(sch->ops.flags & SCX_OPS_ENQ_BLOCKED));
> +                       scx_do_enqueue_task(rq, p, 0, -1);
>                         goto switch_class;
>                 }
>

Just FYI: So in doing some testing running my priority-inversion-demo
test in a loop with this series (rebased on 7.2-rc2), goining through
the example schedulers in tools/sched_ext/ I hit this WARN_ON after
running awhile with the scx_pair scheduler.

Haven't done much debugging on it yet, may have to wait until monday,
but the splat is below.

thanks
-john

[25733.271828] WARNING: kernel/sched/ext/ext.c:2945 at
put_prev_task_scx+0x449/0x460, CPU#0: foreground/25080
[25733.277072] CPU: 0 UID: 0 PID: 25080 Comm: foreground Not tainted
7.2.0-rc2-00043-g9699f4add4b0 #39 PREEMPT(full)
[25733.286458] Hardware name: QEMU Standard PC (i440FX + PIIX, 1996),
BIOS 1.17.0-debian-1.17.0-1 04/01/2014
[25733.295933] Sched_ext: pair (enabled+all), task: runnable_at=-798ms
[25733.295937] RIP: 0010:put_prev_task_scx+0x449/0x460
[25733.306251] Code: 48 8b 83 68 11 00 00 48 8b b5 10 04 00 00 48 39
b0 98 11 00 00 0f 85 79 ff ff ff e9 65 ff ff ff 90 0f 0b 90 e9 46 fe
ff ff 90 <0f> 0b 90 e9 b5 fd ff ff b8 02 00 00 00 e9 94 fc ff ff e8 90
bb 2d
[25733.322497] RSP: 0018:ffffc90011f7faa8 EFLAGS: 00010046
[25733.325750] RAX: 0000000000000001 RBX: ffff8881b8e2db80 RCX: ffff8880bd482650
[25733.333460] RDX: ffff8881b8e2e618 RSI: ffff8880bd482650 RDI: ffff8881073b2650
[25733.338640] RBP: ffff8881073b22c0 R08: 0000000000000000 R09: 0000000000000000
[25733.344710] R10: 0000000000000000 R11: 0000000000000000 R12: ffff88800ba23000
[25733.351179] R13: ffff8880bd4822c0 R14: ffffffff82f6a518 R15: ffffffff82f6a040
[25733.356585] FS:  00007f9badd90780(0000) GS:ffff888234841000(0000)
knlGS:0000000000000000
[25733.362523] CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
[25733.365426] CR2: 00007f91f1e48110 CR3: 000000001113c006 CR4: 0000000000370ef0
[25733.372285] Call Trace:
[25733.373817]  <TASK>
[25733.375178]  __pick_next_task+0xa8/0x180
[25733.378968]  __schedule+0x268/0x2600
[25733.381092]  ? mark_held_locks+0x40/0x70
[25733.383556]  preempt_schedule_notrace+0x43/0xa0
[25733.386311]  ? preempt_schedule_notrace_thunk+0x16/0x30
[25733.390954]  preempt_schedule_notrace_thunk+0x16/0x30
[25733.395476]  rcu_is_watching+0x3e/0x50
[25733.397722]  lock_acquire+0x2c0/0x310
[25733.401168]  ? _raw_spin_unlock_irqrestore+0x50/0x60
[25733.404224]  schedule+0x9d/0x100
[25733.406428]  ? schedule+0x7e/0x100
[25733.408130]  schedule_preempt_disabled+0x18/0x30
[25733.411008]  __mutex_lock+0x800/0x1100
[25733.415788]  ? __start_renaming+0x5d/0x190
[25733.419011]  ? lock_acquire+0xd9/0x310
[25733.421518]  ? __start_renaming+0x5d/0x190
[25733.424188]  __start_renaming+0x5d/0x190
[25733.426676]  filename_renameat2+0x318/0x420
[25733.429258]  ? find_held_lock+0x2b/0x80
[25733.432581]  __x64_sys_rename+0x48/0x70
[25733.435240]  do_syscall_64+0xf3/0x6c0
[25733.437675]  entry_SYSCALL_64_after_hwframe+0x77/0x7f
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.