From: Greg KH <gregkh@linuxfoundation.org>
To: Nathaniel Filardo <nwfilardo@gmail.com>
Cc: stable@vger.kernel.org
Subject: Re: 6.12 build failure for non-SMP systems
Date: Mon, 9 Mar 2026 16:15:27 +0100 [thread overview]
Message-ID: <2026030904-entering-nuptials-d410@gregkh> (raw)
In-Reply-To: <CAKsvP2YbMv+iiVb7NWSKmK_Ugi0Wgt78m1qtdDg9OkNVdhcEcQ@mail.gmail.com>
On Sat, Feb 21, 2026 at 02:04:49PM -0500, Nathaniel Filardo wrote:
> Hi stable@
>
> The commit https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=e61f636cc31042b717a46f6937a2dfaf21f45c91,
> first included in 6.12.64, does not properly guard its new references
> to `stop_sched_class` under `CONFIG_SMP`, leading to build failures
> like the following, even with the significantly newer 6.12.74 release:
>
> > linux-armv5tel-unknown-linux-gnueabi> LD .tmp_vmlinux1
> linux-armv5tel-unknown-linux-gnueabi>
> /nix/store/cn2gkcaa5kfgzjah31c0qlifv9y4yqg5-armv5tel-unknown-linux-gnueabi-binutils-2.44/bin/armv5tel-unknown-linux-gnueabi-ld:
> kernel/sched/build_policy.o: in function `scx_ops_disable_workfn':
> > linux-armv5tel-unknown-linux-gnueabi> /build/linux-6.12.74/build/../kernel/sched/ext.c:4746:(.text+0x704c): undefined reference to `stop_sched_class'
> > linux-armv5tel-unknown-linux-gnueabi> /nix/store/cn2gkcaa5kfgzjah31c0qlifv9y4yqg5-armv5tel-unknown-linux-gnueabi-binutils-2.44/bin/armv5tel-unknown-linux-gnueabi-ld: kernel/sched/build_policy.o: in function `atomic_set':
> > linux-armv5tel-unknown-linux-gnueabi> /build/linux-6.12.74/build/../include/linux/atomic/atomic-instrumented.h:69:(.text+0x11c6c): undefined reference to `stop_sched_class'
>
> A possible fix is simply to add the requisite guard, thus:
> diff --git a/kernel/sched/ext.c.orig b/kernel/sched/ext.c
> index 9f03255..9a3e5b2 100644
> --- a/kernel/sched/ext.c.orig
> +++ b/kernel/sched/ext.c
> @@ -1059,8 +1059,10 @@ static struct scx_dispatch_q *find_user_dsq(u64 dsq_id)
>
> static const struct sched_class *scx_setscheduler_class(struct task_struct *p)
> {
> +#ifdef CONFIG_SMP
> if (p->sched_class == &stop_sched_class)
> return &stop_sched_class;
> +#endif
>
> return __setscheduler_class(p->policy, p->prio);
> }
>
> But it may also be desirable to backport
> https://github.com/torvalds/linux/commit/cac5cefbade90ff0bb0b393d301fa3b5234cf056
> from mainline, which removes much of the distinction between UP and
> SMP schedulers.
That patch doesn't backport cleanly, so can you just submit this as a
fixup for now in a format that we can apply it in?
thanks,
greg k-h
prev parent reply other threads:[~2026-03-09 15:15 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-02-21 19:04 6.12 build failure for non-SMP systems Nathaniel Filardo
2026-03-09 15:15 ` Greg KH [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=2026030904-entering-nuptials-d410@gregkh \
--to=gregkh@linuxfoundation.org \
--cc=nwfilardo@gmail.com \
--cc=stable@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox