Re: [PATCH] RDMA/rxe: Remove unused 'iova' parameter from rxe_mr_init_user

[Guoqing Jiang <guoqing.jiang@xxxxxxxxx>]

On 2/2/24 19:49, Zhu Yanjun wrote:
> On Thu, Feb 1, 2024 at 8:58 PM Guoqing Jiang <guoqing.jiang@xxxxxxxxx> wrote:
> > This one is not needed since commit 954afc5a8fd8 ("RDMA/rxe:
> > Use members of generic struct in rxe_mr").
> >
> > Signed-off-by: Guoqing Jiang <guoqing.jiang@xxxxxxxxx>
> > ---
> >   drivers/infiniband/sw/rxe/rxe_loc.h | 2 +-
> >   drivers/infiniband/sw/rxe/rxe_mr.c  | 2 +-
> >   2 files changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/infiniband/sw/rxe/rxe_loc.h b/drivers/infiniband/sw/rxe/rxe_loc.h
> > index 4d2a8ef52c85..746110898a0e 100644
> > --- a/drivers/infiniband/sw/rxe/rxe_loc.h
> > +++ b/drivers/infiniband/sw/rxe/rxe_loc.h
> > @@ -59,7 +59,7 @@ int rxe_mmap(struct ib_ucontext *context, struct vm_area_struct *vma);
> >   /* rxe_mr.c */
> >   u8 rxe_get_next_key(u32 last_key);
> >   void rxe_mr_init_dma(int access, struct rxe_mr *mr);
> > -int rxe_mr_init_user(struct rxe_dev *rxe, u64 start, u64 length, u64 iova,
> > +int rxe_mr_init_user(struct rxe_dev *rxe, u64 start, u64 length,
> >                       int access, struct rxe_mr *mr);
> >   int rxe_mr_init_fast(int max_pages, struct rxe_mr *mr);
> >   int rxe_flush_pmem_iova(struct rxe_mr *mr, u64 iova, unsigned int length);
> > diff --git a/drivers/infiniband/sw/rxe/rxe_mr.c b/drivers/infiniband/sw/rxe/rxe_mr.c
> > index bc81fde696ee..da3dee520876 100644
> > --- a/drivers/infiniband/sw/rxe/rxe_mr.c
> > +++ b/drivers/infiniband/sw/rxe/rxe_mr.c
> > @@ -126,7 +126,7 @@ static int rxe_mr_fill_pages_from_sgt(struct rxe_mr *mr, struct sg_table *sgt)
> >          return xas_error(&xas);
> >   }
> >
> > -int rxe_mr_init_user(struct rxe_dev *rxe, u64 start, u64 length, u64 iova,
> > +int rxe_mr_init_user(struct rxe_dev *rxe, u64 start, u64 length,
> >                       int access, struct rxe_mr *mr)
> >   {
> >          struct ib_umem *umem;
> diff --git a/drivers/infiniband/sw/rxe/rxe_verbs.c
> b/drivers/infiniband/sw/rxe/rxe_verbs.c
> index 48f86839d36a..04427238fcab 100644
> --- a/drivers/infiniband/sw/rxe/rxe_verbs.c
> +++ b/drivers/infiniband/sw/rxe/rxe_verbs.c
> @@ -1278,7 +1278,7 @@ static struct ib_mr *rxe_reg_user_mr(struct
> ib_pd *ibpd, u64 start,
>          mr->ibmr.pd = ibpd;
>          mr->ibmr.device = ibpd->device;
>
> -       err = rxe_mr_init_user(rxe, start, length, iova, access, mr);
> +       err = rxe_mr_init_user(rxe, start, length, access, mr);
>          if (err) {
>                  rxe_dbg_mr(mr, "reg_user_mr failed, err = %d", err);
>                  goto err_cleanup;

Oops, my mistake, will add above in v2.

Thanks,
Guoqing