diff options
author | Roland Dreier <roland@purestorage.com> | 2016-07-29 06:58:43 +0200 |
---|---|---|
committer | Doug Ledford <dledford@redhat.com> | 2016-08-04 03:03:32 +0200 |
commit | 0c87b672098be368dec9a1ab8f1b897e78233ed5 (patch) | |
tree | 11bb391f7fa6cd441e939a2ca4bf8754bb631824 /.mailmap | |
parent | IB/hfi1: Disable by default (diff) | |
download | linux-0c87b672098be368dec9a1ab8f1b897e78233ed5.tar.xz linux-0c87b672098be368dec9a1ab8f1b897e78233ed5.zip |
IB/mlx4: Don't use GFP_ATOMIC for CQ resize struct
We allocate a small tracking structure as part of mlx4_ib_resize_cq().
However, we don't need to use GFP_ATOMIC -- immediately after the
allocation, we call mlx4_cq_resize(), which allocates a command
mailbox with GFP_KERNEL and then sleeps on a firmware command, so we
better not be in an atomic context.
This actually has a real impact, because when this GFP_ATOMIC
allocation fails (and GFP_ATOMIC does fail in practice) then a
userspace consumer resizing a CQ will get a spurious failure that we
can easily avoid.
Signed-off-by: Roland Dreier <roland@purestorage.com>
Reviewed-by: Leon Romanovsky <leonro@mellanox.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
Diffstat (limited to '.mailmap')
0 files changed, 0 insertions, 0 deletions