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

unexpected end of section #1480

Closed
nickdesaulniers opened this issue Oct 18, 2021 · 8 comments
Closed

unexpected end of section #1480

nickdesaulniers opened this issue Oct 18, 2021 · 8 comments
Labels
duplicate This issue or pull request already exists

Comments

@nickdesaulniers
Copy link
Member

via https://github.com/ClangBuiltLinux/continuous-integration2/runs/3927890481?check_suite_focus=true:

Warning: sound/pci/trident/snd-trident.lto.o: warning: objtool: .text.snd_trident_free_voice: unexpected end of section
Warning: drivers/video/fbdev/intelfb/intelfb.lto.o: warning: objtool: .text.intelfbhw_validate_mode: unexpected end of section
Warning: drivers/net/ethernet/wiznet/w5100.lto.o: warning: objtool: .text.w5100_tx_skb: unexpected end of section
Warning: drivers/net/ethernet/wiznet/w5100.lto.o: warning: objtool: .text.w5100_readbuf: unexpected end of section

I'm surprised to see .lto.o files as part of allmodconfig. Isn't LTO disabled for allmodconfig?

@nickdesaulniers nickdesaulniers added [BUG] Untriaged Something isn't working [BUG] linux A bug that should be fixed in the mainline kernel. [ARCH] x86_64 This bug impacts ARCH=x86_64 [TOOL] objtool warning is produced by the kernel's objtool labels Oct 18, 2021
@nathanchance
Copy link
Member

That is an allmodconfig + ThinLTO build.

@nickdesaulniers
Copy link
Member Author

ah, right, the "kconfig" is printed from the build.json (otherwise the config/build name is cutoff in github UI for me).

@nathanchance
Copy link
Member

Reported upstream: https://reviews.llvm.org/D105169#3116810

I believe current ToT should be fine, as it has a revert of that change: llvm/llvm-project@fd9b099

@nathanchance nathanchance added Reported upstream This bug was filed on LLVM’s issue tracker, Phabricator, or the kernel mailing list. [BUG] llvm (main) A bug in an unreleased version of LLVM (this label is appropriate for regressions) and removed [BUG] Untriaged Something isn't working [BUG] linux A bug that should be fixed in the mainline kernel. labels Nov 8, 2021
@nathanchance
Copy link
Member

Closing as the problematic commit has been reverted and we have a reproducer so that it is fixed in the next revision.

@nathanchance nathanchance added the [FIXED][LLVM] main This bug was only present and fixed in an unreleased version of LLVM label Nov 10, 2021
@nathanchance
Copy link
Member

nathanchance commented Jan 15, 2022

This is likely going to come back so I am preemptively reopening this.

On Monday, I will try to distill down some of the warnings I see and start a thread with Josh and Peter upstream to see if there is any way to minimize the impact of this change in objtool.

@nathanchance nathanchance reopened this Jan 15, 2022
@nathanchance nathanchance removed [FIXED][LLVM] main This bug was only present and fixed in an unreleased version of LLVM Reported upstream This bug was filed on LLVM’s issue tracker, Phabricator, or the kernel mailing list. labels Jan 15, 2022
@nathanchance
Copy link
Member

@nickdesaulniers
Copy link
Member Author

-Xclang -disable-noundef-analysis might be able to get us back to the previous state, for now.

@nathanchance
Copy link
Member

I am pretty sure all of these were a result of CONFIG_UBSAN_DIV_ZERO (LTO enables -ffunction-sections so "fallthrough" warnings become "unexpected end of section" warnings), so I am going to dedup this for #1657, which has more information.

@nathanchance nathanchance removed [ARCH] x86_64 This bug impacts ARCH=x86_64 [TOOL] objtool warning is produced by the kernel's objtool labels Jul 15, 2022
@nathanchance nathanchance added duplicate This issue or pull request already exists and removed [FEATURE] LTO Related to building the kernel with LLVM Link Time Optimization [BUG] llvm (main) A bug in an unreleased version of LLVM (this label is appropriate for regressions) labels Jul 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

2 participants