Re: [PATCH v3 2/2] kbuild: Don't mess with the .cache.mk when root

From: Doug Anderson
Date: Tue Mar 13 2018 - 02:24:00 EST


Hi,

On Mon, Mar 12, 2018 at 11:16 PM, Ingo Molnar <mingo@xxxxxxxxxx> wrote:
>
> * Douglas Anderson <dianders@xxxxxxxxxxxx> wrote:
>
>> +# Don't create Makefile caches if running as root since they can't be deleted
>> +# easily; in the real world we might be root when doing "sudo make install"
>> +ifeq ($(shell id -u),0)
>> +export KBUILD_NOCACHE := 1
>> +endif
>
> Please don't do this - many prominent kernel developers build their kernels as
> root - this makes the build slower for them, and also bifurcates testing.

Ah, interesting. I hadn't realized that!

I'm OK with dropping this patch myself. It was mostly addressing a
potential problem pointed out by Masahiro Yamada when we were talking
about .cache.mk, but I don't think anyone has actually experienced the
problems listed in the CL description. I suppose the other option
would be to go back to keying off the "install" target (like v2 of
this patch did), but it kinda feels like if someone did "sudo make
install" and then followed up by a non-sudo "make" they probably would
be able to figure out how to fix it without much trouble (just do a
"sudo make clean").

In case it's not obvious, though, we should still try to land patch #1
of this series. That's a real problems that people reported when the
.cache.mk stuff first landed and then folks tried to upgrade their
gcc.

-Doug