Isn't it too early? I put the sched_instance_select() after init_param1(). This is not strictly necessary now, but makes some amount of tuning knobs initialized before scheduler is selected.

You should at least add your copyright there, if not replacing mine/FF altogether.

Also we normally put SPDX tag instead of the license text in modern times.

Now that only riscv is the exempt, perhaps change this to check that MACHINE_CPUARCH != riscv instead?

Similarly, #ifndef riscv ? No strong opinion.

It's before init_param1 on amd64, arm64, and riscv. It might be difficult to move later on architectures where the mem* functions are an ifunc.

I open-coded several str* functions in sched_instance_select() for this reason.

It's the target. I want to use ifunc later also for cache operations and maily for SW/HW divide. So these must be resolved ASAP.

I copied it from x86 with minimal changes. It's no problem to steal it :)

Oki, I'll change it before commit.

ditto

Please do fix this.
Otherwise, I see no issues with the patch.