diff options
| author | Paulo Alcantara <pc@manguebit.com> | 2023-08-17 12:34:11 -0300 |
|---|---|---|
| committer | Steve French <stfrench@microsoft.com> | 2023-08-20 16:05:50 -0500 |
| commit | 946ad1b8b18dd91804e2dbfd9a9cefb6a8196bc0 (patch) | |
| tree | b2292f07ae41a72fb6cb7622285e509cc0791487 /fs/smb | |
| parent | 69a4e06c0e7bceeba95104180110841d232e94e4 (diff) | |
| download | lwn-946ad1b8b18dd91804e2dbfd9a9cefb6a8196bc0.tar.gz lwn-946ad1b8b18dd91804e2dbfd9a9cefb6a8196bc0.zip | |
smb: client: reduce stack usage in cifs_demultiplex_thread()
Clang warns about exceeded stack frame size
fs/smb/client/connect.c:1109:1: warning: stack frame size (1048)
exceeds limit (1024) in 'cifs_demultiplex_thread'
[-Wframe-larger-than]
It turns out that clean_demultiplex_info() got inlined into
cifs_demultiplex_thread(), so mark it as noinline_for_stack to save
some stack space.
Signed-off-by: Paulo Alcantara (SUSE) <pc@manguebit.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
Diffstat (limited to 'fs/smb')
| -rw-r--r-- | fs/smb/client/connect.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/smb/client/connect.c b/fs/smb/client/connect.c index b3461d5d0f7d..e19a9c81a5fa 100644 --- a/fs/smb/client/connect.c +++ b/fs/smb/client/connect.c @@ -911,8 +911,8 @@ cifs_enable_signing(struct TCP_Server_Info *server, bool mnt_sign_required) return 0; } - -static void clean_demultiplex_info(struct TCP_Server_Info *server) +static noinline_for_stack void +clean_demultiplex_info(struct TCP_Server_Info *server) { int length; |