diff options
author | Tim Gardner <[email protected]> | 2013-02-13 08:40:16 -0700 |
---|---|---|
committer | J. Bruce Fields <[email protected]> | 2013-02-15 11:29:38 -0500 |
commit | f25cc71e634edcf8a15bc60a48f2b5f3ec9fbb1d (patch) | |
tree | b66f98dacd1c095dfded8ee673290230366dd8c8 /include/linux/fpga/fpga-mgr.h | |
parent | deb4534f4f3be7aea7d9d24c3b0d58f370cbf9ef (diff) |
lockd: nlmclnt_reclaim(): avoid stack overflow
Even though nlmclnt_reclaim() is only one call into the stack frame,
928 bytes on the stack seems like a lot. Recode to dynamically
allocate the request structure once from within the reclaimer task,
then pass this pointer into nlmclnt_reclaim() for reuse on
subsequent calls.
smatch analysis:
fs/lockd/clntproc.c:620 nlmclnt_reclaim() warn: 'reqst' puts
928 bytes on stack
Also remove redundant assignment of 0 after memset.
Cc: Trond Myklebust <[email protected]>
Signed-off-by: Tim Gardner <[email protected]>
Reviewed-by: Jeff Layton <[email protected]>
Signed-off-by: J. Bruce Fields <[email protected]>
Diffstat (limited to 'include/linux/fpga/fpga-mgr.h')
0 files changed, 0 insertions, 0 deletions