Commit c84f0f4f authored by Kalesh AP's avatar Kalesh AP Committed by Leon Romanovsky
Browse files

RDMA/bnxt_re: Fix to drop reference to the mmap entry in case of error 



In the error handling path of bnxt_re_mmap(), driver should invoke
rdma_user_mmap_entry_put() to free the reference of mmap entry in case
the error happens after rdma_user_mmap_entry_get was called.

Fixes: ea222485 ("RDMA/bnxt_re: Update alloc_page uapi for pacing")
Reviewed-by: default avatarSaravanan Vajravel <saravanan.vajravel@broadcom.com>
Reviewed-by: default avatarKashyap Desai <kashyap.desai@broadcom.com>
Signed-off-by: default avatarKalesh AP <kalesh-anakkur.purayil@broadcom.com>
Signed-off-by: default avatarSelvin Xavier <selvin.xavier@broadcom.com>
Link: https://patch.msgid.link/20250104061519.2540178-1-kalesh-anakkur.purayil@broadcom.com


Signed-off-by: default avatarLeon Romanovsky <leon@kernel.org>
parent 220043b0

drivers/infiniband/hw/bnxt_re/ib_verbs.c

+4 −3
Original line number Diff line number Diff line
@@ -4465,7 +4465,8 @@ int bnxt_re_mmap(struct ib_ucontext *ib_uctx, struct vm_area_struct *vma) 
	case BNXT_RE_MMAP_TOGGLE_PAGE:

		/* Driver doesn't expect write access for user space */

		if (vma->vm_flags & VM_WRITE)

			return -EFAULT;

			ret = -EFAULT;

		else

			ret = vm_insert_page(vma, vma->vm_start,

					     virt_to_page((void *)bnxt_entry->mem_offset));

		break;