* [mcgrof-next:20210908-firmware-builtin-v4 2/11] drivers/base/firmware_loader/builtin/main.c:36:6: error: no previous prototype for function 'firmware_is_builtin'
@ 2021-09-10 7:41 kernel test robot
2021-09-11 0:48 ` Luis Chamberlain
0 siblings, 1 reply; 10+ messages in thread
From: kernel test robot @ 2021-09-10 7:41 UTC (permalink / raw)
To: Luis Chamberlain; +Cc: llvm, kbuild-all, linux-kernel, Borislav Petkov
[-- Attachment #1: Type: text/plain, Size: 1989 bytes --]
tree: https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git 20210908-firmware-builtin-v4
head: 1c69d6a17750179d68bcaf6b16f9a08d2e475989
commit: 79e9fce20ee88ffe37542a66277628e6c53dde14 [2/11] firmware_loader: formalize built-in firmware API
config: hexagon-buildonly-randconfig-r004-20210910 (attached as .config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 261cbe98c38f8c1ee1a482fe76511110e790f58a)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git/commit/?id=79e9fce20ee88ffe37542a66277628e6c53dde14
git remote add mcgrof-next https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git
git fetch --no-tags mcgrof-next 20210908-firmware-builtin-v4
git checkout 79e9fce20ee88ffe37542a66277628e6c53dde14
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=hexagon
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All errors (new ones prefixed by >>):
>> drivers/base/firmware_loader/builtin/main.c:36:6: error: no previous prototype for function 'firmware_is_builtin' [-Werror,-Wmissing-prototypes]
bool firmware_is_builtin(const struct firmware *fw)
^
drivers/base/firmware_loader/builtin/main.c:36:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
bool firmware_is_builtin(const struct firmware *fw)
^
static
1 error generated.
vim +/firmware_is_builtin +36 drivers/base/firmware_loader/builtin/main.c
35
> 36 bool firmware_is_builtin(const struct firmware *fw)
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 22236 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [mcgrof-next:20210908-firmware-builtin-v4 2/11] drivers/base/firmware_loader/builtin/main.c:36:6: error: no previous prototype for function 'firmware_is_builtin' 2021-09-10 7:41 [mcgrof-next:20210908-firmware-builtin-v4 2/11] drivers/base/firmware_loader/builtin/main.c:36:6: error: no previous prototype for function 'firmware_is_builtin' kernel test robot @ 2021-09-11 0:48 ` Luis Chamberlain 2021-09-11 1:28 ` [kbuild-all] " Philip Li 2021-09-11 1:52 ` Nathan Chancellor 0 siblings, 2 replies; 10+ messages in thread From: Luis Chamberlain @ 2021-09-11 0:48 UTC (permalink / raw) To: kernel test robot; +Cc: llvm, kbuild-all, linux-kernel, Borislav Petkov On Fri, Sep 10, 2021 at 03:41:31PM +0800, kernel test robot wrote: > tree: https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git 20210908-firmware-builtin-v4 > head: 1c69d6a17750179d68bcaf6b16f9a08d2e475989 > commit: 79e9fce20ee88ffe37542a66277628e6c53dde14 [2/11] firmware_loader: formalize built-in firmware API > config: hexagon-buildonly-randconfig-r004-20210910 (attached as .config) > compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 261cbe98c38f8c1ee1a482fe76511110e790f58a) > reproduce (this is a W=1 build): > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > chmod +x ~/bin/make.cross > # https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git/commit/?id=79e9fce20ee88ffe37542a66277628e6c53dde14 > git remote add mcgrof-next https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git > git fetch --no-tags mcgrof-next 20210908-firmware-builtin-v4 > git checkout 79e9fce20ee88ffe37542a66277628e6c53dde14 > # save the attached .config to linux build tree > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=hexagon > > If you fix the issue, kindly add following tag as appropriate > Reported-by: kernel test robot <lkp@intel.com> > > All errors (new ones prefixed by >>): > > >> drivers/base/firmware_loader/builtin/main.c:36:6: error: no previous prototype for function 'firmware_is_builtin' [-Werror,-Wmissing-prototypes] > bool firmware_is_builtin(const struct firmware *fw) This is a lie though its defined on drivers/base/firmware_loader/firmware.h > ^ > drivers/base/firmware_loader/builtin/main.c:36:1: note: declare 'static' if the function is not intended to be used outside of this translation unit > bool firmware_is_builtin(const struct firmware *fw) > ^ > static > 1 error generated. I get these odd errors: Compiler will be installed in /home/mcgrof/0day cd: received redirection to `https://download.01.org/0day-ci/cross-package/' lftpget -c https://download.01.org/0day-ci/cross-package/./clang_hexagon/clang+llvm-12.0.0-cross-hexagon-unknown-linux-musl.tar.xz tar Jxf clang_hexagon/clang+llvm-12.0.0-cross-hexagon-unknown-linux-musl.tar.xz -C /home/mcgrof/0day make --keep-going HOSTCC=/home/mcgrof/0day/clang+llvm-12.0.0-cross-hexagon-unknown-linux-musl/x86_64-linux-gnu/bin/clang CC=/home/mcgrof/0day/clang+llvm-12.0.0-cross-hexagon-unknown-linux-musl/x86_64-linux-gnu/bin/clang LD=/home/mcgrof/0day/clang+llvm-12.0.0-cross-hexagon-unknown-linux-musl/x86_64-linux-gnu/bin/ld.lld HOSTLD=/home/mcgrof/0day/clang+llvm-12.0.0-cross-hexagon-unknown-linux-musl/x86_64-linux-gnu/bin/ld.lld AR=llvm-ar NM=llvm-nm STRIP=llvm-strip OBJDUMP=llvm-objdump OBJSIZE=llvm-size READELF=llvm-readelf HOSTCXX=clang++ HOSTAR=llvm-ar LLVM_IAS=1 CROSS_COMPILE=hexagon-unknown-linux-musl- --jobs=24 ARCH=hexagon SYNC include/config/auto.conf.cmd HOSTCC scripts/basic/fixdep /home/mcgrof/0day/clang+llvm-12.0.0-cross-hexagon-unknown-linux-musl/x86_64-linux-gnu/bin/clang: error while loading shared libraries: libtinfo.so.5: cannot open shared object file: No such file or directory make[2]: *** [scripts/Makefile.host:95: scripts/basic/fixdep] Error 127 make[2]: Target '__build' not remade because of errors. make[1]: *** [Makefile:594: scripts_basic] Error 2 make[1]: Target 'syncconfig' not remade because of errors. make: *** [Makefile:771: include/config/auto.conf.cmd] Error 2 make: Failed to remake makefile 'include/config/auto.conf.cmd'. make: Failed to remake makefile 'include/config/auto.conf'. HOSTCC scripts/basic/fixdep /home/mcgrof/0day/clang+llvm-12.0.0-cross-hexagon-unknown-linux-musl/x86_64-linux-gnu/bin/clang: error while loading shared libraries: libtinfo.so.5: cannot open shared object file: No such file or directory make[1]: *** [scripts/Makefile.host:95: scripts/basic/fixdep] Error 127 I have a feeling these issues are not correct... Luis ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [kbuild-all] Re: [mcgrof-next:20210908-firmware-builtin-v4 2/11] drivers/base/firmware_loader/builtin/main.c:36:6: error: no previous prototype for function 'firmware_is_builtin' 2021-09-11 0:48 ` Luis Chamberlain @ 2021-09-11 1:28 ` Philip Li 2021-09-13 15:59 ` Luis Chamberlain 2021-09-11 1:52 ` Nathan Chancellor 1 sibling, 1 reply; 10+ messages in thread From: Philip Li @ 2021-09-11 1:28 UTC (permalink / raw) To: Luis Chamberlain Cc: kernel test robot, llvm, kbuild-all, linux-kernel, Borislav Petkov On Fri, Sep 10, 2021 at 05:48:23PM -0700, Luis Chamberlain wrote: > On Fri, Sep 10, 2021 at 03:41:31PM +0800, kernel test robot wrote: > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git 20210908-firmware-builtin-v4 > > head: 1c69d6a17750179d68bcaf6b16f9a08d2e475989 > > commit: 79e9fce20ee88ffe37542a66277628e6c53dde14 [2/11] firmware_loader: formalize built-in firmware API > > config: hexagon-buildonly-randconfig-r004-20210910 (attached as .config) > > compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 261cbe98c38f8c1ee1a482fe76511110e790f58a) > > reproduce (this is a W=1 build): > > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > > chmod +x ~/bin/make.cross > > # https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git/commit/?id=79e9fce20ee88ffe37542a66277628e6c53dde14 > > git remote add mcgrof-next https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git > > git fetch --no-tags mcgrof-next 20210908-firmware-builtin-v4 > > git checkout 79e9fce20ee88ffe37542a66277628e6c53dde14 > > # save the attached .config to linux build tree > > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=hexagon > > > > If you fix the issue, kindly add following tag as appropriate > > Reported-by: kernel test robot <lkp@intel.com> > > > > All errors (new ones prefixed by >>): > > > > >> drivers/base/firmware_loader/builtin/main.c:36:6: error: no previous prototype for function 'firmware_is_builtin' [-Werror,-Wmissing-prototypes] > > bool firmware_is_builtin(const struct firmware *fw) > > This is a lie though its defined on drivers/base/firmware_loader/firmware.h > > > ^ > > drivers/base/firmware_loader/builtin/main.c:36:1: note: declare 'static' if the function is not intended to be used outside of this translation unit > > bool firmware_is_builtin(const struct firmware *fw) > > ^ > > static > > 1 error generated. > > I get these odd errors: > > Compiler will be installed in /home/mcgrof/0day hi Luis, would you mind to download the make.cross tool again to give a try, it was updated recently to use latest clang instead of this 12.0.0? > cd: received redirection to > `https://download.01.org/0day-ci/cross-package/' > lftpget -c > https://download.01.org/0day-ci/cross-package/./clang_hexagon/clang+llvm-12.0.0-cross-hexagon-unknown-linux-musl.tar.xz > tar Jxf > clang_hexagon/clang+llvm-12.0.0-cross-hexagon-unknown-linux-musl.tar.xz > -C /home/mcgrof/0day > make --keep-going > HOSTCC=/home/mcgrof/0day/clang+llvm-12.0.0-cross-hexagon-unknown-linux-musl/x86_64-linux-gnu/bin/clang > CC=/home/mcgrof/0day/clang+llvm-12.0.0-cross-hexagon-unknown-linux-musl/x86_64-linux-gnu/bin/clang > LD=/home/mcgrof/0day/clang+llvm-12.0.0-cross-hexagon-unknown-linux-musl/x86_64-linux-gnu/bin/ld.lld > HOSTLD=/home/mcgrof/0day/clang+llvm-12.0.0-cross-hexagon-unknown-linux-musl/x86_64-linux-gnu/bin/ld.lld > AR=llvm-ar NM=llvm-nm STRIP=llvm-strip OBJDUMP=llvm-objdump > OBJSIZE=llvm-size READELF=llvm-readelf HOSTCXX=clang++ HOSTAR=llvm-ar > LLVM_IAS=1 CROSS_COMPILE=hexagon-unknown-linux-musl- --jobs=24 > ARCH=hexagon > SYNC include/config/auto.conf.cmd > HOSTCC scripts/basic/fixdep > /home/mcgrof/0day/clang+llvm-12.0.0-cross-hexagon-unknown-linux-musl/x86_64-linux-gnu/bin/clang: > error while loading shared libraries: libtinfo.so.5: cannot open > shared object file: No such file or directory > make[2]: *** [scripts/Makefile.host:95: scripts/basic/fixdep] Error > 127 > make[2]: Target '__build' not remade because of errors. > make[1]: *** [Makefile:594: scripts_basic] Error 2 > make[1]: Target 'syncconfig' not remade because of errors. > make: *** [Makefile:771: include/config/auto.conf.cmd] Error 2 > make: Failed to remake makefile 'include/config/auto.conf.cmd'. > make: Failed to remake makefile 'include/config/auto.conf'. > HOSTCC scripts/basic/fixdep > /home/mcgrof/0day/clang+llvm-12.0.0-cross-hexagon-unknown-linux-musl/x86_64-linux-gnu/bin/clang: > error while loading shared libraries: libtinfo.so.5: cannot open > shared object file: No such file or directory > make[1]: *** [scripts/Makefile.host:95: scripts/basic/fixdep] > Error 127 > > > I have a feeling these issues are not correct... > > Luis > _______________________________________________ > kbuild-all mailing list -- kbuild-all@lists.01.org > To unsubscribe send an email to kbuild-all-leave@lists.01.org ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [kbuild-all] Re: [mcgrof-next:20210908-firmware-builtin-v4 2/11] drivers/base/firmware_loader/builtin/main.c:36:6: error: no previous prototype for function 'firmware_is_builtin' 2021-09-11 1:28 ` [kbuild-all] " Philip Li @ 2021-09-13 15:59 ` Luis Chamberlain 2021-09-13 16:21 ` Luis Chamberlain 0 siblings, 1 reply; 10+ messages in thread From: Luis Chamberlain @ 2021-09-13 15:59 UTC (permalink / raw) To: Philip Li Cc: kernel test robot, llvm, kbuild-all, linux-kernel, Borislav Petkov On Sat, Sep 11, 2021 at 09:28:53AM +0800, Philip Li wrote: > On Fri, Sep 10, 2021 at 05:48:23PM -0700, Luis Chamberlain wrote: > > On Fri, Sep 10, 2021 at 03:41:31PM +0800, kernel test robot wrote: > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git 20210908-firmware-builtin-v4 > > > head: 1c69d6a17750179d68bcaf6b16f9a08d2e475989 > > > commit: 79e9fce20ee88ffe37542a66277628e6c53dde14 [2/11] firmware_loader: formalize built-in firmware API > > > config: hexagon-buildonly-randconfig-r004-20210910 (attached as .config) > > > compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 261cbe98c38f8c1ee1a482fe76511110e790f58a) > > > reproduce (this is a W=1 build): > > > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > > > chmod +x ~/bin/make.cross > > > # https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git/commit/?id=79e9fce20ee88ffe37542a66277628e6c53dde14 > > > git remote add mcgrof-next https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git > > > git fetch --no-tags mcgrof-next 20210908-firmware-builtin-v4 > > > git checkout 79e9fce20ee88ffe37542a66277628e6c53dde14 > > > # save the attached .config to linux build tree > > > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=hexagon > > > > > > If you fix the issue, kindly add following tag as appropriate > > > Reported-by: kernel test robot <lkp@intel.com> > > > > > > All errors (new ones prefixed by >>): > > > > > > >> drivers/base/firmware_loader/builtin/main.c:36:6: error: no previous prototype for function 'firmware_is_builtin' [-Werror,-Wmissing-prototypes] > > > bool firmware_is_builtin(const struct firmware *fw) > > > > This is a lie though its defined on drivers/base/firmware_loader/firmware.h > > > > > ^ > > > drivers/base/firmware_loader/builtin/main.c:36:1: note: declare 'static' if the function is not intended to be used outside of this translation unit > > > bool firmware_is_builtin(const struct firmware *fw) > > > ^ > > > static > > > 1 error generated. > > > > I get these odd errors: > > > > Compiler will be installed in /home/mcgrof/0day > hi Luis, would you mind to download the make.cross tool again to give a try, it > was updated recently to use latest clang instead of this 12.0.0? I had clang+llvm-12.0.0-cross-hexagon-unknown-linux-musl and this was used already. Is there a more recent one? Just in case I rm -rf'd it and tried again with the latest make.cross. Luis ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [kbuild-all] Re: [mcgrof-next:20210908-firmware-builtin-v4 2/11] drivers/base/firmware_loader/builtin/main.c:36:6: error: no previous prototype for function 'firmware_is_builtin' 2021-09-13 15:59 ` Luis Chamberlain @ 2021-09-13 16:21 ` Luis Chamberlain 2021-09-15 8:15 ` Chen, Rong A 0 siblings, 1 reply; 10+ messages in thread From: Luis Chamberlain @ 2021-09-13 16:21 UTC (permalink / raw) To: Philip Li Cc: kernel test robot, llvm, kbuild-all, linux-kernel, Borislav Petkov On Mon, Sep 13, 2021 at 08:59:50AM -0700, Luis Chamberlain wrote: > On Sat, Sep 11, 2021 at 09:28:53AM +0800, Philip Li wrote: > > On Fri, Sep 10, 2021 at 05:48:23PM -0700, Luis Chamberlain wrote: > > > On Fri, Sep 10, 2021 at 03:41:31PM +0800, kernel test robot wrote: > > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git 20210908-firmware-builtin-v4 > > > > head: 1c69d6a17750179d68bcaf6b16f9a08d2e475989 > > > > commit: 79e9fce20ee88ffe37542a66277628e6c53dde14 [2/11] firmware_loader: formalize built-in firmware API > > > > config: hexagon-buildonly-randconfig-r004-20210910 (attached as .config) > > > > compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 261cbe98c38f8c1ee1a482fe76511110e790f58a) > > > > reproduce (this is a W=1 build): > > > > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > > > > chmod +x ~/bin/make.cross > > > > # https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git/commit/?id=79e9fce20ee88ffe37542a66277628e6c53dde14 > > > > git remote add mcgrof-next https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git > > > > git fetch --no-tags mcgrof-next 20210908-firmware-builtin-v4 > > > > git checkout 79e9fce20ee88ffe37542a66277628e6c53dde14 > > > > # save the attached .config to linux build tree > > > > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=hexagon > > > > > > > > If you fix the issue, kindly add following tag as appropriate > > > > Reported-by: kernel test robot <lkp@intel.com> > > > > > > > > All errors (new ones prefixed by >>): > > > > > > > > >> drivers/base/firmware_loader/builtin/main.c:36:6: error: no previous prototype for function 'firmware_is_builtin' [-Werror,-Wmissing-prototypes] > > > > bool firmware_is_builtin(const struct firmware *fw) > > > > > > This is a lie though its defined on drivers/base/firmware_loader/firmware.h > > > > > > > ^ > > > > drivers/base/firmware_loader/builtin/main.c:36:1: note: declare 'static' if the function is not intended to be used outside of this translation unit > > > > bool firmware_is_builtin(const struct firmware *fw) > > > > ^ > > > > static > > > > 1 error generated. > > > > > > I get these odd errors: > > > > > > Compiler will be installed in /home/mcgrof/0day > > hi Luis, would you mind to download the make.cross tool again to give a try, it > > was updated recently to use latest clang instead of this 12.0.0? > > I had clang+llvm-12.0.0-cross-hexagon-unknown-linux-musl and this was > used already. Is there a more recent one? Just in case I rm -rf'd it > and tried again with the latest make.cross. I rm -rf ~/0day and tried again but the failure I get is: CALL scripts/checksyscalls.sh <stdin>:1515:2: warning: syscall clone3 not implemented [-W#warnings] #warning syscall clone3 not implemented Even if I disable CONFIG_WERROR and COMPILE_TEST this still fails here and so I can't even test compile the code in question. Luis ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [kbuild-all] Re: [mcgrof-next:20210908-firmware-builtin-v4 2/11] drivers/base/firmware_loader/builtin/main.c:36:6: error: no previous prototype for function 'firmware_is_builtin' 2021-09-13 16:21 ` Luis Chamberlain @ 2021-09-15 8:15 ` Chen, Rong A 0 siblings, 0 replies; 10+ messages in thread From: Chen, Rong A @ 2021-09-15 8:15 UTC (permalink / raw) To: Luis Chamberlain, Philip Li Cc: kernel test robot, llvm, kbuild-all, linux-kernel, Borislav Petkov On 9/14/2021 12:21 AM, Luis Chamberlain wrote: > On Mon, Sep 13, 2021 at 08:59:50AM -0700, Luis Chamberlain wrote: >> On Sat, Sep 11, 2021 at 09:28:53AM +0800, Philip Li wrote: >>> On Fri, Sep 10, 2021 at 05:48:23PM -0700, Luis Chamberlain wrote: >>>> On Fri, Sep 10, 2021 at 03:41:31PM +0800, kernel test robot wrote: >>>>> tree: https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git 20210908-firmware-builtin-v4 >>>>> head: 1c69d6a17750179d68bcaf6b16f9a08d2e475989 >>>>> commit: 79e9fce20ee88ffe37542a66277628e6c53dde14 [2/11] firmware_loader: formalize built-in firmware API >>>>> config: hexagon-buildonly-randconfig-r004-20210910 (attached as .config) >>>>> compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 261cbe98c38f8c1ee1a482fe76511110e790f58a) >>>>> reproduce (this is a W=1 build): >>>>> wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross >>>>> chmod +x ~/bin/make.cross >>>>> # https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git/commit/?id=79e9fce20ee88ffe37542a66277628e6c53dde14 >>>>> git remote add mcgrof-next https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git >>>>> git fetch --no-tags mcgrof-next 20210908-firmware-builtin-v4 >>>>> git checkout 79e9fce20ee88ffe37542a66277628e6c53dde14 >>>>> # save the attached .config to linux build tree >>>>> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=hexagon >>>>> >>>>> If you fix the issue, kindly add following tag as appropriate >>>>> Reported-by: kernel test robot <lkp@intel.com> >>>>> >>>>> All errors (new ones prefixed by >>): >>>>> >>>>>>> drivers/base/firmware_loader/builtin/main.c:36:6: error: no previous prototype for function 'firmware_is_builtin' [-Werror,-Wmissing-prototypes] >>>>> bool firmware_is_builtin(const struct firmware *fw) >>>> >>>> This is a lie though its defined on drivers/base/firmware_loader/firmware.h >>>> >>>>> ^ >>>>> drivers/base/firmware_loader/builtin/main.c:36:1: note: declare 'static' if the function is not intended to be used outside of this translation unit >>>>> bool firmware_is_builtin(const struct firmware *fw) >>>>> ^ >>>>> static >>>>> 1 error generated. >>>> >>>> I get these odd errors: >>>> >>>> Compiler will be installed in /home/mcgrof/0day >>> hi Luis, would you mind to download the make.cross tool again to give a try, it >>> was updated recently to use latest clang instead of this 12.0.0? >> >> I had clang+llvm-12.0.0-cross-hexagon-unknown-linux-musl and this was >> used already. Is there a more recent one? Just in case I rm -rf'd it >> and tried again with the latest make.cross. > > I rm -rf ~/0day and tried again but the failure I get is: > > CALL scripts/checksyscalls.sh > <stdin>:1515:2: warning: syscall clone3 not implemented [-W#warnings] > #warning syscall clone3 not implemented > > Even if I disable CONFIG_WERROR and COMPILE_TEST this still fails here > and so I can't even test compile the code in question. > > Luis Hi Luis, Sorry for the inconvenience, the error can be reproduced with W=1, we'll update the reproduce step. Best Regards, Rong Chen ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [mcgrof-next:20210908-firmware-builtin-v4 2/11] drivers/base/firmware_loader/builtin/main.c:36:6: error: no previous prototype for function 'firmware_is_builtin' 2021-09-11 0:48 ` Luis Chamberlain 2021-09-11 1:28 ` [kbuild-all] " Philip Li @ 2021-09-11 1:52 ` Nathan Chancellor 2021-09-13 15:56 ` Luis Chamberlain 1 sibling, 1 reply; 10+ messages in thread From: Nathan Chancellor @ 2021-09-11 1:52 UTC (permalink / raw) To: Luis Chamberlain Cc: kernel test robot, llvm, kbuild-all, linux-kernel, Borislav Petkov On Fri, Sep 10, 2021 at 05:48:23PM -0700, Luis Chamberlain wrote: > On Fri, Sep 10, 2021 at 03:41:31PM +0800, kernel test robot wrote: > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git 20210908-firmware-builtin-v4 > > head: 1c69d6a17750179d68bcaf6b16f9a08d2e475989 > > commit: 79e9fce20ee88ffe37542a66277628e6c53dde14 [2/11] firmware_loader: formalize built-in firmware API > > config: hexagon-buildonly-randconfig-r004-20210910 (attached as .config) > > compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 261cbe98c38f8c1ee1a482fe76511110e790f58a) > > reproduce (this is a W=1 build): > > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > > chmod +x ~/bin/make.cross > > # https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git/commit/?id=79e9fce20ee88ffe37542a66277628e6c53dde14 > > git remote add mcgrof-next https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git > > git fetch --no-tags mcgrof-next 20210908-firmware-builtin-v4 > > git checkout 79e9fce20ee88ffe37542a66277628e6c53dde14 > > # save the attached .config to linux build tree > > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=hexagon > > > > If you fix the issue, kindly add following tag as appropriate > > Reported-by: kernel test robot <lkp@intel.com> > > > > All errors (new ones prefixed by >>): > > > > >> drivers/base/firmware_loader/builtin/main.c:36:6: error: no previous prototype for function 'firmware_is_builtin' [-Werror,-Wmissing-prototypes] > > bool firmware_is_builtin(const struct firmware *fw) > > This is a lie though its defined on drivers/base/firmware_loader/firmware.h Unless I am missing something, you don't include the firmware_loader copy of firmware.h in builtin/main.c (only the linux one)? Isn't that the source of the warning? Cheers, Nathan ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [mcgrof-next:20210908-firmware-builtin-v4 2/11] drivers/base/firmware_loader/builtin/main.c:36:6: error: no previous prototype for function 'firmware_is_builtin' 2021-09-11 1:52 ` Nathan Chancellor @ 2021-09-13 15:56 ` Luis Chamberlain 2021-09-13 17:25 ` Nathan Chancellor 0 siblings, 1 reply; 10+ messages in thread From: Luis Chamberlain @ 2021-09-13 15:56 UTC (permalink / raw) To: Nathan Chancellor Cc: kernel test robot, llvm, kbuild-all, linux-kernel, Borislav Petkov On Fri, Sep 10, 2021 at 06:52:03PM -0700, Nathan Chancellor wrote: > On Fri, Sep 10, 2021 at 05:48:23PM -0700, Luis Chamberlain wrote: > > On Fri, Sep 10, 2021 at 03:41:31PM +0800, kernel test robot wrote: > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git 20210908-firmware-builtin-v4 > > > head: 1c69d6a17750179d68bcaf6b16f9a08d2e475989 > > > commit: 79e9fce20ee88ffe37542a66277628e6c53dde14 [2/11] firmware_loader: formalize built-in firmware API > > > config: hexagon-buildonly-randconfig-r004-20210910 (attached as .config) > > > compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 261cbe98c38f8c1ee1a482fe76511110e790f58a) > > > reproduce (this is a W=1 build): > > > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > > > chmod +x ~/bin/make.cross > > > # https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git/commit/?id=79e9fce20ee88ffe37542a66277628e6c53dde14 > > > git remote add mcgrof-next https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git > > > git fetch --no-tags mcgrof-next 20210908-firmware-builtin-v4 > > > git checkout 79e9fce20ee88ffe37542a66277628e6c53dde14 > > > # save the attached .config to linux build tree > > > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=hexagon > > > > > > If you fix the issue, kindly add following tag as appropriate > > > Reported-by: kernel test robot <lkp@intel.com> > > > > > > All errors (new ones prefixed by >>): > > > > > > >> drivers/base/firmware_loader/builtin/main.c:36:6: error: no previous prototype for function 'firmware_is_builtin' [-Werror,-Wmissing-prototypes] > > > bool firmware_is_builtin(const struct firmware *fw) > > > > This is a lie though its defined on drivers/base/firmware_loader/firmware.h > > Unless I am missing something, you don't include the firmware_loader > copy of firmware.h in builtin/main.c (only the linux one)? Isn't that > the source of the warning? You would think, but nope, its there: #include "firmware.h" Luis ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [mcgrof-next:20210908-firmware-builtin-v4 2/11] drivers/base/firmware_loader/builtin/main.c:36:6: error: no previous prototype for function 'firmware_is_builtin' 2021-09-13 15:56 ` Luis Chamberlain @ 2021-09-13 17:25 ` Nathan Chancellor 2021-09-13 17:30 ` Luis Chamberlain 0 siblings, 1 reply; 10+ messages in thread From: Nathan Chancellor @ 2021-09-13 17:25 UTC (permalink / raw) To: Luis Chamberlain Cc: kernel test robot, llvm, kbuild-all, linux-kernel, Borislav Petkov On 9/13/2021 8:56 AM, Luis Chamberlain wrote: > On Fri, Sep 10, 2021 at 06:52:03PM -0700, Nathan Chancellor wrote: >> On Fri, Sep 10, 2021 at 05:48:23PM -0700, Luis Chamberlain wrote: >>> On Fri, Sep 10, 2021 at 03:41:31PM +0800, kernel test robot wrote: >>>> tree: https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git 20210908-firmware-builtin-v4 >>>> head: 1c69d6a17750179d68bcaf6b16f9a08d2e475989 >>>> commit: 79e9fce20ee88ffe37542a66277628e6c53dde14 [2/11] firmware_loader: formalize built-in firmware API >>>> config: hexagon-buildonly-randconfig-r004-20210910 (attached as .config) >>>> compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 261cbe98c38f8c1ee1a482fe76511110e790f58a) >>>> reproduce (this is a W=1 build): >>>> wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross >>>> chmod +x ~/bin/make.cross >>>> # https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git/commit/?id=79e9fce20ee88ffe37542a66277628e6c53dde14 >>>> git remote add mcgrof-next https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git >>>> git fetch --no-tags mcgrof-next 20210908-firmware-builtin-v4 >>>> git checkout 79e9fce20ee88ffe37542a66277628e6c53dde14 >>>> # save the attached .config to linux build tree >>>> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=hexagon >>>> >>>> If you fix the issue, kindly add following tag as appropriate >>>> Reported-by: kernel test robot <lkp@intel.com> >>>> >>>> All errors (new ones prefixed by >>): >>>> >>>>>> drivers/base/firmware_loader/builtin/main.c:36:6: error: no previous prototype for function 'firmware_is_builtin' [-Werror,-Wmissing-prototypes] >>>> bool firmware_is_builtin(const struct firmware *fw) >>> >>> This is a lie though its defined on drivers/base/firmware_loader/firmware.h >> >> Unless I am missing something, you don't include the firmware_loader >> copy of firmware.h in builtin/main.c (only the linux one)? Isn't that >> the source of the warning? > > You would think, but nope, its there: > > #include "firmware.h" I see that in drivers/base/firmware_loader/main.c but the warning is complaining about drivers/base/firmware_loader/builtin/main.c, where I only see #include <linux/firmware.h> Sorry for the issues with reproducing. For what it's worth, I can reproduce this locally and adding #include "../firmware.h" to drivers/base/firmware_loader/builtin/main.c fixes it for me. Cheers, Nathan ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [mcgrof-next:20210908-firmware-builtin-v4 2/11] drivers/base/firmware_loader/builtin/main.c:36:6: error: no previous prototype for function 'firmware_is_builtin' 2021-09-13 17:25 ` Nathan Chancellor @ 2021-09-13 17:30 ` Luis Chamberlain 0 siblings, 0 replies; 10+ messages in thread From: Luis Chamberlain @ 2021-09-13 17:30 UTC (permalink / raw) To: Nathan Chancellor Cc: kernel test robot, llvm, kbuild-all, linux-kernel, Borislav Petkov On Mon, Sep 13, 2021 at 10:25:35AM -0700, Nathan Chancellor wrote: > On 9/13/2021 8:56 AM, Luis Chamberlain wrote: > > On Fri, Sep 10, 2021 at 06:52:03PM -0700, Nathan Chancellor wrote: > > > On Fri, Sep 10, 2021 at 05:48:23PM -0700, Luis Chamberlain wrote: > > > > On Fri, Sep 10, 2021 at 03:41:31PM +0800, kernel test robot wrote: > > > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git 20210908-firmware-builtin-v4 > > > > > head: 1c69d6a17750179d68bcaf6b16f9a08d2e475989 > > > > > commit: 79e9fce20ee88ffe37542a66277628e6c53dde14 [2/11] firmware_loader: formalize built-in firmware API > > > > > config: hexagon-buildonly-randconfig-r004-20210910 (attached as .config) > > > > > compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 261cbe98c38f8c1ee1a482fe76511110e790f58a) > > > > > reproduce (this is a W=1 build): > > > > > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > > > > > chmod +x ~/bin/make.cross > > > > > # https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git/commit/?id=79e9fce20ee88ffe37542a66277628e6c53dde14 > > > > > git remote add mcgrof-next https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git > > > > > git fetch --no-tags mcgrof-next 20210908-firmware-builtin-v4 > > > > > git checkout 79e9fce20ee88ffe37542a66277628e6c53dde14 > > > > > # save the attached .config to linux build tree > > > > > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=hexagon > > > > > > > > > > If you fix the issue, kindly add following tag as appropriate > > > > > Reported-by: kernel test robot <lkp@intel.com> > > > > > > > > > > All errors (new ones prefixed by >>): > > > > > > > > > > > > drivers/base/firmware_loader/builtin/main.c:36:6: error: no previous prototype for function 'firmware_is_builtin' [-Werror,-Wmissing-prototypes] > > > > > bool firmware_is_builtin(const struct firmware *fw) > > > > > > > > This is a lie though its defined on drivers/base/firmware_loader/firmware.h > > > > > > Unless I am missing something, you don't include the firmware_loader > > > copy of firmware.h in builtin/main.c (only the linux one)? Isn't that > > > the source of the warning? > > > > You would think, but nope, its there: > > > > #include "firmware.h" > > I see that in drivers/base/firmware_loader/main.c but the warning is > complaining about drivers/base/firmware_loader/builtin/main.c, where I only > see > > #include <linux/firmware.h> > > Sorry for the issues with reproducing. For what it's worth, I can reproduce > this locally and adding > > #include "../firmware.h" > > to drivers/base/firmware_loader/builtin/main.c fixes it for me. Got it, thanks! I've ammended this change. Luis ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2021-09-15 8:15 UTC | newest] Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-09-10 7:41 [mcgrof-next:20210908-firmware-builtin-v4 2/11] drivers/base/firmware_loader/builtin/main.c:36:6: error: no previous prototype for function 'firmware_is_builtin' kernel test robot 2021-09-11 0:48 ` Luis Chamberlain 2021-09-11 1:28 ` [kbuild-all] " Philip Li 2021-09-13 15:59 ` Luis Chamberlain 2021-09-13 16:21 ` Luis Chamberlain 2021-09-15 8:15 ` Chen, Rong A 2021-09-11 1:52 ` Nathan Chancellor 2021-09-13 15:56 ` Luis Chamberlain 2021-09-13 17:25 ` Nathan Chancellor 2021-09-13 17:30 ` Luis Chamberlain
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).