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

unsupported directive '.stabs' #1418

Closed
nickdesaulniers opened this issue Jun 28, 2021 · 8 comments
Closed

unsupported directive '.stabs' #1418

nickdesaulniers opened this issue Jun 28, 2021 · 8 comments
Labels
[ARCH] powerpc This bug impacts ARCH=powerpc [BUG] linux A bug that should be fixed in the mainline kernel. [FIXED][LINUX] 5.18 This bug was fixed in Linux 5.18 Reported upstream This bug was filed on LLVM’s issue tracker, Phabricator, or the kernel mailing list. [TOOL] integrated-as The issue is relevant to LLVM integrated assembler

Comments

@nickdesaulniers
Copy link
Member

$ ARCH=powerpc CROSS_COMPILE=powerpc64le-linux-gnu- make LLVM=1 LLVM_IAS=1 -j72 pseries_defconfig arch/powerpc/kernel/vdso32/gettimeofday.o
...
arch/powerpc/kernel/vdso32/gettimeofday.S:72:8: error: unsupported directive '.stabs'
.stabs "_restgpr_31_x:F-1",36,0,0,_restgpr_31_x; .globl _restgpr_31_x; _restgpr_31_x:
       ^
arch/powerpc/kernel/vdso32/gettimeofday.S:73:8: error: unsupported directive '.stabs'
.stabs "_rest32gpr_31_x:F-1",36,0,0,_rest32gpr_31_x; .globl _rest32gpr_31_x; _rest32gpr_31_x:
       ^

https://sourceware.org/binutils/docs/as/Stab.html

@nickdesaulniers nickdesaulniers added [BUG] Untriaged Something isn't working [TOOL] integrated-as The issue is relevant to LLVM integrated assembler [ARCH] powerpc This bug impacts ARCH=powerpc labels Jun 28, 2021
@nickdesaulniers
Copy link
Member Author

.stabs is also used in arch/powerpc/lib/copy_32.S.

@paulmenzel
Copy link

The LLVM developers say in bug report[inline asm] .stabs directive support , STABS is replaced by DWARF.

@nickdesaulniers nickdesaulniers added the Reported upstream This bug was filed on LLVM’s issue tracker, Phabricator, or the kernel mailing list. label Sep 9, 2021
@nickdesaulniers
Copy link
Member Author

@segher mentions on IRC:

stabs is an older debug format, like dwarf
it will be good to have it eradicated :-)
you need to figure out what it does, instead [of just removing it]

@segher
Copy link

segher commented Sep 9, 2021

"to have it eradicated"... I was talking about the GCC support for STABS. This has no
bearing on the VDSO issue.

@nickdesaulniers nickdesaulniers self-assigned this Oct 29, 2021
@nickdesaulniers nickdesaulniers added [BUG] linux A bug that should be fixed in the mainline kernel. and removed [BUG] Untriaged Something isn't working labels Oct 29, 2021
@nickdesaulniers nickdesaulniers removed their assignment Dec 28, 2021
@nickdesaulniers
Copy link
Member Author

cc @aik

@nathanchance
Copy link
Member

Patch removing these annotations has been picked up: https://git.kernel.org/powerpc/c/12318163737cd8808d13faa6e2393774191a6182

@nathanchance nathanchance added the [PATCH] Accepted A submitted patch has been accepted upstream label Feb 16, 2022
nathanchance added a commit to nathanchance/continuous-integration2 that referenced this issue Mar 4, 2022
On mainline, SUSE's configuration needs to build with LLVM_IAS=0 because
of ClangBuiltLinux/linux#1418.

Closes: ClangBuiltLinux#265
Signed-off-by: Nathan Chancellor <nathan@kernel.org>
@nickdesaulniers
Copy link
Member Author

(Noting that we should revisit ClangBuiltLinux/continuous-integration2#321 once this is closed out)

nathanchance added a commit to nathanchance/continuous-integration2 that referenced this issue Mar 4, 2022
On mainline, SUSE's configuration needs to build with LLVM_IAS=0 because
of ClangBuiltLinux/linux#1418.

Closes: ClangBuiltLinux#265
Signed-off-by: Nathan Chancellor <nathan@kernel.org>
nathanchance added a commit to nathanchance/continuous-integration2 that referenced this issue Mar 8, 2022
On mainline, SUSE's configuration needs to build with LLVM_IAS=0 because
of ClangBuiltLinux/linux#1418.

The distribution configs link with ld.bfd for the time being due to an
outstanding boot failure when the kernel is linked with ld.lld.

Closes: ClangBuiltLinux#265
Signed-off-by: Nathan Chancellor <nathan@kernel.org>
nathanchance added a commit to nathanchance/continuous-integration2 that referenced this issue Mar 9, 2022
On mainline, SUSE's configuration needs to build with LLVM_IAS=0 because
of ClangBuiltLinux/linux#1418.

Closes: ClangBuiltLinux#265
Signed-off-by: Nathan Chancellor <nathan@kernel.org>
nathanchance added a commit to nathanchance/continuous-integration2 that referenced this issue Mar 10, 2022
On mainline, SUSE's configuration needs to build with LLVM_IAS=0 because
of ClangBuiltLinux/linux#1418.

Closes: ClangBuiltLinux#265
Signed-off-by: Nathan Chancellor <nathan@kernel.org>
nathanchance added a commit to nathanchance/continuous-integration2 that referenced this issue Mar 11, 2022
On mainline, SUSE's configuration needs to build with LLVM_IAS=0 because
of ClangBuiltLinux/linux#1418.

Closes: ClangBuiltLinux#265
Signed-off-by: Nathan Chancellor <nathan@kernel.org>
@nathanchance
Copy link
Member

Merged into mainline: https://git.kernel.org/linus/12318163737cd8808d13faa6e2393774191a6182

@nathanchance nathanchance added [FIXED][LINUX] 5.18 This bug was fixed in Linux 5.18 and removed [PATCH] Accepted A submitted patch has been accepted upstream labels Mar 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[ARCH] powerpc This bug impacts ARCH=powerpc [BUG] linux A bug that should be fixed in the mainline kernel. [FIXED][LINUX] 5.18 This bug was fixed in Linux 5.18 Reported upstream This bug was filed on LLVM’s issue tracker, Phabricator, or the kernel mailing list. [TOOL] integrated-as The issue is relevant to LLVM integrated assembler
Projects
None yet
Development

No branches or pull requests

4 participants