aboutsummaryrefslogtreecommitdiffstats
path: root/kernel
diff options
context:
space:
mode:
authorJia-Ju Bai <baijiaju1990@gmail.com>2018-04-11 15:32:48 +0800
committerJason Gunthorpe <jgg@mellanox.com>2018-04-17 19:57:11 -0600
commitf9af8730143a0fdc572f90b8a388795ee812cd74 (patch)
tree32e818461d20ca9a9f8920195f6127222d24a9af /kernel
parentinfiniband: i40iw: Replace GFP_ATOMIC with GFP_KERNEL in i40iw_add_mqh_4 (diff)
downloadlinux-dev-f9af8730143a0fdc572f90b8a388795ee812cd74.tar.xz
linux-dev-f9af8730143a0fdc572f90b8a388795ee812cd74.zip
infiniband: i40iw: Replace GFP_ATOMIC with GFP_KERNEL in i40iw_make_listen_node
i40iw_make_listen_node() is never called in atomic context. i40iw_make_listen_node() is only called by i40iw_create_listen, which is set as ".create_listen" in struct iw_cm_verbs. Despite never getting called from atomic context, i40iw_make_listen_node() 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 <baijiaju1990@gmail.com> Acked-by: Shiraz Saleem <shiraz.saleem@intel.com> Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>
Diffstat (limited to 'kernel')
0 files changed, 0 insertions, 0 deletions