diff options
author | Arnd Bergmann <arnd@arndb.de> | 2017-12-11 15:11:17 +0300 |
---|---|---|
committer | Jens Axboe <axboe@kernel.dk> | 2018-01-06 19:18:00 +0300 |
commit | 91f7b74aca363da5f294c9a24e870ae6ba3a27d3 (patch) | |
tree | 04f6489c7eb1d59331b0fefefcd081ac68f8d524 /fs/sync.c | |
parent | cf8c0c6a3830583bd0e7c94933e155bf97cd162b (diff) | |
download | linux-91f7b74aca363da5f294c9a24e870ae6ba3a27d3.tar.xz |
DAC960: split up ioctl function to reduce stack size
When CONFIG_KASAN is set, all the local variables in this function are
allocated on the stack together, leading to a warning about possible
kernel stack overflow:
drivers/block/DAC960.c: In function 'DAC960_gam_ioctl':
drivers/block/DAC960.c:7061:1: error: the frame size of 2240 bytes is larger than 2048 bytes [-Werror=frame-larger-than=]
By splitting up the function into smaller chunks, we can avoid that and
make the code slightly more readable at the same time. The coding style
in this file is completely nonstandard, and I chose to not touch that
at all, leaving the unconventional intendation unchanged to make it
easier to review the diff.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'fs/sync.c')
0 files changed, 0 insertions, 0 deletions