On 08.04.2022 10:45, Luca Fancellu wrote: > --- > docs/misc/arm/device-tree/cpupools.txt | 140 +++++++++++++++++ > xen/arch/arm/include/asm/smp.h | 3 + > xen/common/Kconfig | 7 + > xen/common/Makefile | 1 + > xen/common/boot_cpupools.c | 207 +++++++++++++++++++++++++ > xen/common/sched/cpupool.c | 12 +- > xen/include/xen/sched.h | 14 ++ > 7 files changed, 383 insertions(+), 1 deletion(-) > create mode 100644 docs/misc/arm/device-tree/cpupools.txt > create mode 100644 xen/common/boot_cpupools.c
Under whose maintainership is the new file to fall? Without an addition to ./MAINTAINERS and without the file being placed in xen/common/sched/, it'll be REST maintainers, which I think would better be avoided. Would it perhaps make sense to have this as xen/common/sched/boot.c, allowing other boot-only code to potentially be moved there over time? This would then also avoid me asking about the underscore in the file name: Underscores are a somewhat artificial thing for use in places where dashes can't be used. Yet in the file system dashes are fine, and dashes are (slightly) easier to type. Jan