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.

Sign up for GitHub

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

Jump to bottom

-Wframe-larger-than in drivers/gpu/drm/omapdrm/dss/dispc.c #2038

Open
nathanchance opened this issue Jul 17, 2024 · 0 comments
Open

-Wframe-larger-than in drivers/gpu/drm/omapdrm/dss/dispc.c #2038

nathanchance opened this issue Jul 17, 2024 · 0 comments
Labels
[BUG] llvm A bug that should be fixed in upstream LLVM Reported upstream This bug was filed on LLVM’s issue tracker, Phabricator, or the kernel mailing list.

Comments

@nathanchance
Copy link
Member

Continuous integration started seeing

drivers/gpu/drm/omapdrm/dss/dispc.c:4866:27: warning: stack frame size (2320) exceeds limit (2048) in 'dispc_runtime_suspend' [-Wframe-larger-than]
 4866 | static __maybe_unused int dispc_runtime_suspend(struct device *dev)
      |                           ^
drivers/gpu/drm/omapdrm/dss/dispc.c:4881:27: warning: stack frame size (2704) exceeds limit (2048) in 'dispc_runtime_resume' [-Wframe-larger-than]
 4881 | static __maybe_unused int dispc_runtime_resume(struct device *dev)
      |                           ^
2 warnings generated.

after this driver was allowed to be built under CONFIG_COMPILE_TEST when building ARCH=loongarch allmodconfig.

See my upstream report for a reduction and further details: llvm/llvm-project#99265
@nathanchance nathanchance added [BUG] llvm A bug that should be fixed in upstream LLVM Reported upstream This bug was filed on LLVM’s issue tracker, Phabricator, or the kernel mailing list. labels Jul 17, 2024
nathanchance added a commit to nathanchance/continuous-integration2 that referenced this issue Jul 25, 2024
@nathanchance
generator: yml: Disable CONFIG_DRM_OMAP with LoongArch configurations
23f652f 
A recent change in linux-next allows this driver to be built with
CONFIG_COMPILE_TEST=y, which causes a couple of -Wframe-larger-than
warnings to show up with ARCH=loongarch. This appears to be an issue in
LLVM, for which I filed an issue. As this code won't actually run on
LoongArch machines, just disable it to avoid having to turn off
CONFIG_WERROR altogether with this configuration.

Link: ClangBuiltLinux/linux#2038
Signed-off-by: Nathan Chancellor <[email protected]>
@nathanchance nathanchance mentioned this issue Jul 25, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
[BUG] llvm A bug that should be fixed in upstream LLVM Reported upstream This bug was filed on LLVM’s issue tracker, Phabricator, or the kernel mailing list.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@nathanchance