Re: [PATCH 5.10 000/132] 5.10.85-rc1 review

[Pavel Machek]

Hi!

> > > > I'm getting a lot of build failures -- missing gmp.h:
> > > > 
> > > >   UPD     include/generated/utsrelease.h
> > > > 1317In file included from /builds/hVatwYBy/68/cip-project/cip-testing/linux-stable-rc-ci/gcc/gcc-8.1.0-nolibc/arm-linux-gnueabi/bin/../lib/gcc/arm-linux-gnueabi/8.1.0/plugin/include/gcc-plugin.h:28:0,
> > > > 1318                 from scripts/gcc-plugins/gcc-common.h:7,
> > > > 1319                 from scripts/gcc-plugins/arm_ssp_per_task_plugin.c:3:
> > > > 1320/builds/hVatwYBy/68/cip-project/cip-testing/linux-stable-rc-ci/gcc/gcc-8.1.0-nolibc/arm-linux-gnueabi/bin/../lib/gcc/arm-linux-gnueabi/8.1.0/plugin/include/system.h:687:10: fatal error: gmp.h: No such file or directory
> > > > 1321 #include <gmp.h>
> > > > 1322          ^~~~~~~
> > > > 1323compilation terminated.
> > > > 1324scripts/gcc-plugins/Makefile:47: recipe for target 'scripts/gcc-plugins/arm_ssp_per_task_plugin.so' failed
> > > > 1325
> > > > 
> > > > https://gitlab.com/cip-project/cip-testing/linux-stable-rc-ci/-/tree/linux-5.10.y
> > > 
> > > What gcc plugins are you trying to build with?
> > 
> > Also, kernelci seems normal for this release:
> > 	https://linux.kernelci.org/build/stable-rc/branch/linux-5.10.y/kernel/v5.10.84-133-gf6a609e247c6/
> > 
> > But your tests show problems:
> > 	https://gitlab.com/cip-project/cip-testing/linux-stable-rc-ci/-/pipelines/428150268/failures
> > all for gcc plugins.
> > 
> > I did take changes for the gcc plugins to get them to work for gcc 11,
> > maybe gcc8 is too old for them now?
> 
> Not sure. Maybe they never worked for us, but without test compile,
> they are now enabled and thus can break the build?
> 
> 1e8600 gcc-plugins: simplify GCC plugin-dev capability test
> 
> Could be capable of doing that. Let me cc our people doing the
> testing and let me try to do some test compiles...

Ok, so if I revert 1e8600 things start working again:

https://gitlab.com/cip-project/cip-kernel/linux-cip/-/pipelines/428666243

as that is only speedup and apparently not needed for following
patches, that might be the easiest solution?

Best regards,
								Pavel

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany

Attachment: signature.asc
Description: PGP signature