Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

error: "__always_inline" redefined [-Werror] #366

Open
ElyesH opened this issue Jul 15, 2024 · 2 comments
Open

error: "__always_inline" redefined [-Werror] #366

ElyesH opened this issue Jul 15, 2024 · 2 comments

Comments

@ElyesH
Copy link

ElyesH commented Jul 15, 2024

We can't update opensbi submodule to upstream (since 2022 !) as it fails to build :
CC ramstage/arch/riscv/misc.o CC ramstage/arch/riscv/opensbi.o In file included from 3rdparty/opensbi/include/sbi/fw_dynamic.h:46, from src/arch/riscv/opensbi.c:8: 3rdparty/opensbi/include/sbi/sbi_types.h:72:9: error: "__always_inline" redefined [-Werror] 72 | #define __always_inline inline __attribute__((always_inline)) | ^~~~~~~~~~~~~~~ In file included from <command-line>: ./src/commonlib/bsd/include/commonlib/bsd/compiler.h:58:9: note: this is the location of the previous definition 58 | #define __always_inline inline __attribute__((__always_inline__)) | ^~~~~~~~~~~~~~~ cc1: all warnings being treated as errors make[2]: *** [Makefile:423: /cb-build/coreboot-gerrit.0/gcc/EMULATION_QEMU_RISCV_RV64_/ramstage/arch/riscv/opensbi.o] Error 1 make[2]: Leaving directory '/home/coreboot/node-root/workspace/coreboot-gerrit'

(see https://qa.coreboot.org/job/coreboot-gerrit/264570/testReport/junit/(root)/gcc/EMULATION_QEMU_RISCV_RV64_/)

@ElyesH
Copy link
Author

ElyesH commented Sep 15, 2024

No body at home?

@avpatel
Copy link
Collaborator

avpatel commented Sep 15, 2024

@ElyesH Sorry for the delay. Most of the discussions generally happen on mailing list so Github issues get overlooked.

I see you have raised a GitHub PR as well. I will send your patch to the mailing list.

avpatel pushed a commit to avpatel/opensbi that referenced this issue Sep 15, 2024
avpatel pushed a commit to avpatel/opensbi that referenced this issue Sep 15, 2024
Update __always_inline macro define to fix opensbi upstream
build for coreboot.
Refer, https://qa.coreboot.org/job/coreboot-gerrit/257449/testReport/junit/(root)/clang/EMULATION_QEMU_RISCV_RV64_/

Closes: riscv-software-src#366
Signed-off-by: Elyes Haouas <[email protected]>
Signed-off-by: Anup Patel <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants