tree 03093f1f337193ae966f1b8e83cb204d364bfb9d
parent 0da0e51729959e047e2e6f4a8e3554826b8cc607
author Jia-Ju Bai <baijiaju1990@163.com> 1496665420 +0800
committer Greg Kroah-Hartman <gregkh@linuxfoundation.org> 1541864566 -0800

rxe: Fix a sleep-in-atomic bug in post_one_send

[ Upstream commit 07d432bb97f19dd5e784175152f9fce3b2646133 ]

The driver may sleep under a spin lock, and the function call path is:
post_one_send (acquire the lock by spin_lock_irqsave)
  init_send_wqe
    copy_from_user --> may sleep

There is no flow that makes "qp->is_user" true, and copy_from_user may
cause bug when a non-user pointer is used. So the lines of copy_from_user
and check of "qp->is_user" are removed.

Signed-off-by: Jia-Ju Bai <baijiaju1990@163.com>
Reviewed-by: Leon Romanovsky <leonro@mellanox.com>
Acked-by: Moni Shoua <monis@mellanox.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>