-Wframe-larger-than in fs/namespace.c #1964
Labels
[ARCH] x86
This bug impacts ARCH=i386
[BUG] linux-next
This is an issue only seen in linux-next
Reported upstream
This bug was filed on LLVM’s issue tracker, Phabricator, or the kernel mailing list.
[WORKAROUND] Applied
This bug has an applied workaround
CI sees this warning with
ARCH=i386 defconfig
with LLVM 13 and older after https://git.kernel.org/vfs/vfs/c/49889374ab925e10708d638ae7ebec5da7ac69a6:I bisected the fix in LLVM to llvm/llvm-project@bb15861. I think this is exacerbated by
ARCH=i386
's use of-ffreestanding
(which has come up before in #1583), as the warning disappears if I remove that addition fromarch/x86/Makefile
(and the stack usage for that function drops to 684 bytes, which I can see with-Wframe-larger-than=512
).The diff of the IR before and after that change is pretty massive but I think it is the compound literal assignment in
prepare_kstatmount()
that messes things up here, as it appears that the compound literal is stack allocated viaalloca
in addition to the variablesks
andkreq
?IR diff of
__ia32_sys_statmount
infs/namespace.ll
I am able to avoid this warning with the following diff (which just cause the allocations of the compound literal to be in
prepare_kstatmount()
) but I am not sure if that would be acceptable upstream.The text was updated successfully, but these errors were encountered: