Re: [GIT PULL] scheduler updates for v5.18

From: Nathan Chancellor <nathan@kernel.org>
To: Ingo Molnar <mingo@kernel.org>
Cc: Linus Torvalds <torvalds@linux-foundation.org>,
	linux-kernel@vger.kernel.org,
	Peter Zijlstra <peterz@infradead.org>,
	Juri Lelli <juri.lelli@redhat.com>,
	Vincent Guittot <vincent.guittot@linaro.org>,
	Dietmar Eggemann <dietmar.eggemann@arm.com>,
	Steven Rostedt <rostedt@goodmis.org>,
	Ben Segall <bsegall@google.com>, Mel Gorman <mgorman@suse.de>,
	Daniel Bristot de Oliveira <bristot@redhat.com>,
	Andrew Morton <akpm@linux-foundation.org>,
	Borislav Petkov <bp@alien8.de>,
	Thomas Gleixner <tglx@linutronix.de>
Subject: Re: [GIT PULL] scheduler updates for v5.18
Date: Mon, 21 Mar 2022 08:44:52 -0700	[thread overview]
Message-ID: <YjiddAnoCCz7Tbt3@dev-arch.thelio-3990X> (raw)
In-Reply-To: <YjhZUezhnamHAl0H@gmail.com>

Hi Ingo,

On Mon, Mar 21, 2022 at 11:54:09AM +0100, Ingo Molnar wrote:
> Ingo Molnar (17):
>       sched/headers: Fix header to build standalone: <linux/sched_clock.h>
>       sched/headers: Add header guard to kernel/sched/sched.h
>       sched/headers: Add header guard to kernel/sched/stats.h and kernel/sched/autogroup.h
>       sched/headers: sched/clock: Mark all functions 'notrace', remove CC_FLAGS_FTRACE build asymmetry
>       sched/headers: Add initial new headers as identity mappings
>       sched/headers: Fix comment typo in kernel/sched/cpudeadline.c
>       sched/headers: Make the <linux/sched/deadline.h> header build standalone
>       sched/headers: Introduce kernel/sched/build_utility.c and build multiple .c files there
>       sched/headers: Introduce kernel/sched/build_policy.c and build multiple .c files there
>       sched/headers: Standardize kernel/sched/sched.h header dependencies
>       sched/headers: Reorganize, clean up and optimize kernel/sched/core.c dependencies
>       sched/headers: Reorganize, clean up and optimize kernel/sched/fair.c dependencies
>       sched/headers: Reorganize, clean up and optimize kernel/sched/build_policy.c dependencies
>       sched/headers: Reorganize, clean up and optimize kernel/sched/build_utility.c dependencies
>       sched/headers: Reorganize, clean up and optimize kernel/sched/sched.h dependencies
>       sched/headers: Only include <linux/entry-common.h> when CONFIG_GENERIC_ENTRY=y
>       headers/prep: Fix header to build standalone: <linux/psi.h>

This series regresses ARCH=arm allmodconfig:

$ make -skj"$(nproc)" ARCH=arm CROSS_COMPILE=arm-linux-gnueabi- mrproper allmodconfig kernel/sched/
In file included from kernel/sched/fair.c:52:
kernel/sched/sched.h:87:11: fatal error: asm/paravirt_api_clock.h: No such file or directory
   87 | # include <asm/paravirt_api_clock.h>
      |           ^~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
make[3]: *** [scripts/Makefile.build:288: kernel/sched/fair.o] Error 1
In file included from kernel/sched/core.c:81:
kernel/sched/sched.h:87:11: fatal error: asm/paravirt_api_clock.h: No such file or directory
   87 | # include <asm/paravirt_api_clock.h>
      |           ^~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
make[3]: *** [scripts/Makefile.build:288: kernel/sched/core.o] Error 1
In file included from kernel/sched/build_policy.c:33:
kernel/sched/sched.h:87:11: fatal error: asm/paravirt_api_clock.h: No such file or directory
   87 | # include <asm/paravirt_api_clock.h>
      |           ^~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
make[3]: *** [scripts/Makefile.build:288: kernel/sched/build_policy.o] Error 1
In file included from kernel/sched/build_utility.c:52:
kernel/sched/sched.h:87:11: fatal error: asm/paravirt_api_clock.h: No such file or directory
   87 | # include <asm/paravirt_api_clock.h>
      |           ^~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
make[3]: *** [scripts/Makefile.build:288: kernel/sched/build_utility.o] Error 1
make[3]: Target '__build' not remade because of errors.
make[2]: *** [scripts/Makefile.build:550: kernel/sched] Error 2

Randy Dunlap has sent a patch that fixes it, which I just reviewed:

https://lore.kernel.org/r/20220316204146.14000-1-rdunlap@infradead.org/

It would be nice if this could be taken with the pull (or submitted with
the pull in a v2 as I see Qian's comment) to avoid regressing the build.

Cheers,
Nathan