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

-Wframe-larger-than= in fs/select.c #200

Closed
nickdesaulniers opened this issue Oct 10, 2018 · 6 comments
Closed

-Wframe-larger-than= in fs/select.c #200

nickdesaulniers opened this issue Oct 10, 2018 · 6 comments
Labels
-Wframe-larger-than= [ARCH] arm32 This bug impacts ARCH=arm [BUG] linux A bug that should be fixed in the mainline kernel. unreproducible Not or no longer reproducible

Comments

@nickdesaulniers
Copy link
Member

  CC      kernel/task_work.o
fs/select.c:926:12: warning: stack frame size of 1064 bytes in function 'do_sys_poll' [-Wframe-larger-than=]
static int do_sys_poll(struct pollfd __user *ufds, unsigned int nfds,
           ^
1 warning generated.

This is not a duplicate of #39; CONFIG_KASAN is not set.

@nickdesaulniers nickdesaulniers added [BUG] linux A bug that should be fixed in the mainline kernel. [ARCH] x86_64 This bug impacts ARCH=x86_64 low priority This bug is not critical and not a priority -Wframe-larger-than= labels Oct 10, 2018
@nathanchance
Copy link
Member

nathanchance commented Oct 10, 2018

Hmmm this is x86_64? CONFIG_FRAME_WARN should be 2048 if CONFIG_64BIT is set.

@nickdesaulniers
Copy link
Member Author

ah, I'll bet that my leftover localmodconfig has the old values.

@nathanchance
Copy link
Member

s/isn't/is/, whoops lol. It's 1024 if CONFIG_64BIT isn't set.

@nathanchance
Copy link
Member

I see this on arm32 (multi_v7_defconfig):

fs/select.c:594:5: warning: stack frame size of 1064 bytes in function 'core_sys_select' [-Wframe-larger-than=]
int core_sys_select(int n, fd_set __user *inp, fd_set __user *outp,
    ^
1 warning generated.

@nathanchance nathanchance added the [ARCH] arm32 This bug impacts ARCH=arm label Oct 17, 2018
@bulwahn
Copy link

bulwahn commented Apr 27, 2019

I have seen these two:

fs/select.c:619:5: warning: stack frame size of 1224 bytes in function 'core_sys_select' [-Wframe-larger-than=]
int core_sys_select(int n, fd_set __user *inp, fd_set __user *outp,
    ^
fs/select.c:965:12: warning: stack frame size of 1048 bytes in function 'do_sys_poll' [-Wframe-larger-than=]
static int do_sys_poll(struct pollfd __user *ufds, unsigned int nfds,
           ^
2 warnings generated.

on v5.1-rc5 with x86-64 allnoconfig + 64-bit kernel.

@nickdesaulniers
Copy link
Member Author

I no longer see these w/ x86 (asm goto patches applied)

@nickdesaulniers nickdesaulniers removed the [ARCH] x86_64 This bug impacts ARCH=x86_64 label May 20, 2019
@nickdesaulniers nickdesaulniers added the unreproducible Not or no longer reproducible label Feb 8, 2020
@tpimh tpimh removed the low priority This bug is not critical and not a priority label Feb 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-Wframe-larger-than= [ARCH] arm32 This bug impacts ARCH=arm [BUG] linux A bug that should be fixed in the mainline kernel. unreproducible Not or no longer reproducible
Projects
None yet
Development

No branches or pull requests

4 participants