-
Notifications
You must be signed in to change notification settings - Fork 15
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
drivers/net/wireless/realtek/rtlwifi/rtl8192cu/rtl8192cu.lto.o: warning: objtool: .text.rtl92cu_hw_init: unexpected end of section #1613
Comments
Here my kernel-config. |
My make line:
|
I suspect this is just one instance of #1480. You could try to verify it by adding |
This here seems not to work: --- a/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/Makefile
+++ b/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/Makefile
@@ -10,4 +10,8 @@ rtl8192cu-objs := \
table.o \
trx.o
+# CBL issue #1613
+# https://github.com/ClangBuiltLinux/linux/issues/1613
+CFLAGS_rtl8192cu.o += -Xclang -no-enable-noundef-analysis
+
obj-$(CONFIG_RTL8192CU) += rtl8192cu.o |
I do not want to re-compile the whole Linux-kernel again by passing Looks like this will be in
With this diff: --- a/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/Makefile
+++ b/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/Makefile
@@ -1,4 +1,8 @@
# SPDX-License-Identifier: GPL-2.0
+
+# https://github.com/ClangBuiltLinux/linux/issues/1613
+ccflags-y += -disable-noundef-analysis
+
rtl8192cu-objs := \
dm.o \
hw.o \ Nope.
|
Just as a note:
|
|
Nope.
|
Ah, the flag was only renamed in main, my apologies (I did correct it after the fact but that is not obvious): llvm/llvm-project@b529744 Regardless, if I add I won't dupe this report for now though, as this might be a bug in the kernel sources. |
This fixes the warning for me: --- a/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/Makefile
+++ b/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/Makefile
@@ -1,4 +1,8 @@
# SPDX-License-Identifier: GPL-2.0
+
+# https://github.com/ClangBuiltLinux/linux/issues/1613
+ccflags-y += -Xclang -disable-noundef-analysis
+
rtl8192cu-objs := \
dm.o \
hw.o \ |
@dileks can you please supply the output of |
With and without patch:
NOTE: I have commented the |
Checking again:
Attached output of the last llvm-objdump-14_rtl8192cu_lto_o-text_rtl92cu_hw_init.txt llvm-objdump-14_rtl8192cu_lto_o-rela_text_rtl92cu_hw_init.txt |
Seeing:
|
I can send you the llvm-objdumps if you desire.
|
For the posterity:
[ Update 2022-03-25 ] Patch of PeterZ |
Re-checking:
Looks like UPDATE: For both cases BAD and GOOD the same. |
Attached output of:
llvm-objdump-14-Dr-disassemble_symbols-rtl92cu_hw_init_rtl8192cu-lto-o.txt |
Above was with my patch applied and from my today's build. Here now without (BAD) and with my patch (GOOD):
|
See this issue with Linux v5.18-rc7+.
|
Is there a more elegant way to check for the LLVM version? |
I'm not yet convinced that noundef-analysis is broken; I don't think disabling the analysis in Kbuild is the way to go. @dileks can you confirm that you don't have |
Full kernel-config attached. |
With patch applied (no objtool warnings):
llvm-objdump_rtl8192cu_o.txt |
Attached llvm-objdump without patch applied (objtool warning visible). I can provide both object-files if needed. |
@dileks I'm not sure about this one, please provide the .o file for the broken case and I can take a look. |
I see this new
objtool
warning when usingLLVM-14 version 14.0.0
(Debian or selfmade/stage1-only) andCONFIG_LTO_CLANG_THIN=y
:With
LLVM-13
v13.0.0
I do not see it.On a first view in the issue-tracker I have not seen this reported.
The text was updated successfully, but these errors were encountered: