kbuild: ARCH=um archheaders failed

kbuild: ARCH=um archheaders failed

[Randy Dunlap]

Hi Yamada-san,

I see this every few weeks.  It's not reproducible (it depends
on the moon, Mars, etc.).

I'm using O=builddir.  The build log is short (below).

Do you see a makefile problem here?  (nothing to do with mmotm;
I see it more frequently with linux-next since I do many daily
builds with it)


thanks.
-- 
make[1]: Entering directory '/home/rdunlap/mmotm-2018-0802-1529/UM64'
  UPD     include/generated/uapi/linux/version.h
  WRAP    arch/x86/include/generated/asm/dma-contiguous.h
  WRAP    arch/x86/include/generated/asm/export.h
  WRAP    arch/x86/include/generated/asm/early_ioremap.h
  WRAP    arch/x86/include/generated/asm/mcs_spinlock.h
  WRAP    arch/x86/include/generated/asm/mm-arch-hooks.h
  WRAP    arch/x86/include/generated/uapi/asm/bpf_perf_event.h
  WRAP    arch/x86/include/generated/uapi/asm/poll.h
  GEN     ./Makefile
make[2]: *** No rule to make target 'archheaders'.  Stop.
arch/um/Makefile:119: recipe for target 'archheaders' failed
make[1]: *** [archheaders] Error 2
make[1]: *** Waiting for unfinished jobs....
  UPD     include/config/kernel.release
make[1]: *** wait: No child processes.  Stop.
Makefile:146: recipe for target 'sub-make' failed
make: *** [sub-make] Error 2

Re: kbuild: ARCH=um archheaders failed

[Randy Dunlap]

On 08/02/2018 05:13 PM, Randy Dunlap wrote:
> Hi Yamada-san,
> 
> I see this every few weeks.  It's not reproducible (it depends
> on the moon, Mars, etc.).
> 
> I'm using O=builddir.  The build log is short (below).
> 
> Do you see a makefile problem here?  (nothing to do with mmotm;
> I see it more frequently with linux-next since I do many daily
> builds with it)

BTW, this is just um defconfig with -j4.

-- 
~Randy

Re: kbuild: ARCH=um archheaders failed

[Masahiro Yamada]

2018-08-04 0:41 GMT+09:00 Randy Dunlap <rdunlap@infradead.org>:
> On 08/02/2018 05:13 PM, Randy Dunlap wrote:
>> Hi Yamada-san,
>>
>> I see this every few weeks.  It's not reproducible (it depends
>> on the moon, Mars, etc.).
>>
>> I'm using O=builddir.  The build log is short (below).
>>
>> Do you see a makefile problem here?  (nothing to do with mmotm;
>> I see it more frequently with linux-next since I do many daily
>> builds with it)
>
> BTW, this is just um defconfig with -j4.
>
> --
> ~Randy
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html



I will send a patch with commit description later
after more testing,
but the following patch should fix the issue.


diff --git a/arch/um/Makefile b/arch/um/Makefile
index d3975e7..1cec053 100644
--- a/arch/um/Makefile
+++ b/arch/um/Makefile
@@ -120,7 +120,7 @@ archheaders:
        $(Q)$(MAKE) -f $(srctree)/scripts/Makefile.asm-generic \
                    kbuild-file=$(HOST_DIR)/include/uapi/asm/Kbuild \
                    obj=$(HOST_DIR)/include/generated/uapi/asm
-       $(Q)$(MAKE) KBUILD_SRC= ARCH=$(HEADER_ARCH) archheaders
+       $(Q)$(MAKE) -f $(srctree)/Makefile ARCH=$(HEADER_ARCH) archheaders


 archprepare: include/generated/user_constants.h



-- 
Best Regards
Masahiro Yamada

Re: kbuild: ARCH=um archheaders failed

[Randy Dunlap]

On 08/03/2018 07:23 PM, Masahiro Yamada wrote:
> 2018-08-04 0:41 GMT+09:00 Randy Dunlap <rdunlap@infradead.org>:
>> On 08/02/2018 05:13 PM, Randy Dunlap wrote:
>>> Hi Yamada-san,
>>>
>>> I see this every few weeks.  It's not reproducible (it depends
>>> on the moon, Mars, etc.).
>>>
>>> I'm using O=builddir.  The build log is short (below).
>>>
>>> Do you see a makefile problem here?  (nothing to do with mmotm;
>>> I see it more frequently with linux-next since I do many daily
>>> builds with it)
>>
>> BTW, this is just um defconfig with -j4.
>>
>> --
>> ~Randy
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
> 
> 
> I will send a patch with commit description later
> after more testing,
> but the following patch should fix the issue.
> 
> 
> diff --git a/arch/um/Makefile b/arch/um/Makefile
> index d3975e7..1cec053 100644
> --- a/arch/um/Makefile
> +++ b/arch/um/Makefile
> @@ -120,7 +120,7 @@ archheaders:
>         $(Q)$(MAKE) -f $(srctree)/scripts/Makefile.asm-generic \
>                     kbuild-file=$(HOST_DIR)/include/uapi/asm/Kbuild \
>                     obj=$(HOST_DIR)/include/generated/uapi/asm
> -       $(Q)$(MAKE) KBUILD_SRC= ARCH=$(HEADER_ARCH) archheaders
> +       $(Q)$(MAKE) -f $(srctree)/Makefile ARCH=$(HEADER_ARCH) archheaders
> 
> 
>  archprepare: include/generated/user_constants.h
> 
> 
> 

I tested this patch and it works.  :)
but then it's difficult to reproduce the problem.

Thanks!
-- 
~Randy