mbox series

[for-next,0/3] Allocate MR cleanups

Message ID 20200706120343.10816-1-galpress@amazon.com (mailing list archive)
Headers show
Series Allocate MR cleanups | expand

Message

Gal Pressman July 6, 2020, 12:03 p.m. UTC 
The allocate MR functionality is limited to kernel users, there is no
reason to pass a redundant udata parameter.
In addition, a small cleanup was added to the MR allocation function to
keep the main flow unindented.

Gal Pressman (3):
  RDMA/core: Check for error instead of success in alloc MR function
  RDMA/core: Remove ib_alloc_mr_user function
  RDMA: Remove the udata parameter from alloc_mr callback

 drivers/infiniband/core/verbs.c               | 36 +++++++++----------
 drivers/infiniband/hw/bnxt_re/ib_verbs.c      |  2 +-
 drivers/infiniband/hw/bnxt_re/ib_verbs.h      |  2 +-
 drivers/infiniband/hw/cxgb4/iw_cxgb4.h        |  2 +-
 drivers/infiniband/hw/cxgb4/mem.c             |  2 +-
 drivers/infiniband/hw/hns/hns_roce_device.h   |  2 +-
 drivers/infiniband/hw/hns/hns_roce_mr.c       |  2 +-
 drivers/infiniband/hw/i40iw/i40iw_verbs.c     |  3 +-
 drivers/infiniband/hw/mlx4/mlx4_ib.h          |  2 +-
 drivers/infiniband/hw/mlx4/mr.c               |  2 +-
 drivers/infiniband/hw/mlx5/mlx5_ib.h          |  2 +-
 drivers/infiniband/hw/mlx5/mr.c               |  2 +-
 drivers/infiniband/hw/ocrdma/ocrdma_verbs.c   |  2 +-
 drivers/infiniband/hw/ocrdma/ocrdma_verbs.h   |  2 +-
 drivers/infiniband/hw/qedr/verbs.c            |  2 +-
 drivers/infiniband/hw/qedr/verbs.h            |  2 +-
 drivers/infiniband/hw/vmw_pvrdma/pvrdma_mr.c  |  2 +-
 .../infiniband/hw/vmw_pvrdma/pvrdma_verbs.h   |  2 +-
 drivers/infiniband/sw/rdmavt/mr.c             |  2 +-
 drivers/infiniband/sw/rdmavt/mr.h             |  2 +-
 drivers/infiniband/sw/rxe/rxe_verbs.c         |  2 +-
 drivers/infiniband/sw/siw/siw_verbs.c         |  2 +-
 drivers/infiniband/sw/siw/siw_verbs.h         |  2 +-
 include/rdma/ib_verbs.h                       | 12 ++-----
 24 files changed, 43 insertions(+), 50 deletions(-)

Comments

Jason Gunthorpe July 6, 2020, 10:26 p.m. UTC | #1 
On Mon, Jul 06, 2020 at 03:03:40PM +0300, Gal Pressman wrote:
> The allocate MR functionality is limited to kernel users, there is no
> reason to pass a redundant udata parameter.
> In addition, a small cleanup was added to the MR allocation function to
> keep the main flow unindented.
> 
> Gal Pressman (3):
>   RDMA/core: Check for error instead of success in alloc MR function
>   RDMA/core: Remove ib_alloc_mr_user function
>   RDMA: Remove the udata parameter from alloc_mr callback

Applied to for-next, thanks

Jason