diff options
| author | Arnd Bergmann <arnd@arndb.de> | 2016-09-19 11:57:26 (GMT) |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-01-26 07:24:34 (GMT) |
| commit | 45f3a7eb2ca05a8cd0e56889fba96b548077f7a3 (patch) | |
| tree | 2a93ec8f6753597b17c13da2196790f4dbc03530 /net/ncsi/ncsi-manage.c | |
| parent | 25c72b518734ec5d4dfb7ad90d785ff8efc04eb7 (diff) | |
| download | linux-45f3a7eb2ca05a8cd0e56889fba96b548077f7a3.tar.xz | |
IB/rxe: avoid putting a large struct rxe_qp on stack
commit a0fa72683e78979ef1123d679b1c40ae28bd9096 upstream.
A race condition fix added an rxe_qp structure to the stack in order
to be able to perform rollback in rxe_requester(), but the structure
is large enough to trigger the warning for possible stack overflow:
drivers/infiniband/sw/rxe/rxe_req.c: In function 'rxe_requester':
drivers/infiniband/sw/rxe/rxe_req.c:757:1: error: the frame size of 2064 bytes is larger than 1024 bytes [-Werror=frame-larger-than=]
This changes the rollback function to only save the psn inside
the qp, which is the only field we access in the rollback_qp
anyway.
Fixes: 3050b9985024 ("IB/rxe: Fix race condition between requester and completer")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Reviewed-by: Leon Romanovsky <leonro@mellanox.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'net/ncsi/ncsi-manage.c')
0 files changed, 0 insertions, 0 deletions