]> git.proxmox.com Git - mirror_ubuntu-bionic-kernel.git/commit
IB/umem: Use the correct mm during ib_umem_release
authorLidong Chen <jemmy858585@gmail.com>
Tue, 8 May 2018 08:50:16 +0000 (16:50 +0800)
committerKleber Sacilotto de Souza <kleber.souza@canonical.com>
Mon, 27 Aug 2018 14:40:05 +0000 (16:40 +0200)
commitdcb5b430e8da6e4a4b3ef0b516f1e8d8d9b9f307
tree0f42b6740575b7d521de6fcd91c565412cfe09cc
parent38c7f58cd386c312042e936a903522e500abebfb
IB/umem: Use the correct mm during ib_umem_release

BugLink: http://bugs.launchpad.net/bugs/1786352
commit 8e907ed4882714fd13cfe670681fc6cb5284c780 upstream.

User-space may invoke ibv_reg_mr and ibv_dereg_mr in different threads.

If ibv_dereg_mr is called after the thread which invoked ibv_reg_mr has
exited, get_pid_task will return NULL and ib_umem_release will not
decrease mm->pinned_vm.

Instead of using threads to locate the mm, use the overall tgid from the
ib_ucontext struct instead. This matches the behavior of ODP and
disassociate in handling the mm of the process that called ibv_reg_mr.

Cc: <stable@vger.kernel.org>
Fixes: 87773dd56d54 ("IB: ib_umem_release() should decrement mm->pinned_vm from ib_umem_get")
Signed-off-by: Lidong Chen <lidongchen@tencent.com>
Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Kamal Mostafa <kamal@canonical.com>
Signed-off-by: Khalid Elmously <khalid.elmously@canonical.com>
drivers/infiniband/core/umem.c
include/rdma/ib_umem.h