diff options
| author | Jia-Ju Bai <[email protected]> | 2018-04-11 03:07:32 +0000 |
|---|---|---|
| committer | Greg Kroah-Hartman <[email protected]> | 2018-04-23 11:31:27 +0000 |
| commit | 24b6d3eee183ddde814b47aedc7f0c0c5ab7b72d (patch) | |
| tree | 5dc36fab24796fdc1ef0091cf992e713e0bdda34 /drivers/fpga/fpga-region.c | |
| parent | misc: tifm: Remove VLA (diff) | |
| download | kernel-24b6d3eee183ddde814b47aedc7f0c0c5ab7b72d.tar.gz kernel-24b6d3eee183ddde814b47aedc7f0c0c5ab7b72d.zip | |
misc: ti-st: Replace GFP_ATOMIC with GFP_KERNEL in kim_probe
kim_probe() is never called in atomic context.
This function is only set as ".probe" in struct platform_driver.
Despite never getting called from atomic context,
kim_probe() calls kzalloc() with GFP_ATOMIC,
which does not sleep for allocation.
GFP_ATOMIC is not necessary and can be replaced with GFP_KERNEL,
which can sleep and improve the possibility of sucessful allocation.
This is found by a static analysis tool named DCNS written by myself.
And I also manually check it.
Signed-off-by: Jia-Ju Bai <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
Diffstat (limited to 'drivers/fpga/fpga-region.c')
0 files changed, 0 insertions, 0 deletions