On Thu, Sep 03, 2020 at 02:12:10PM +0000, Saleem, Shiraz wrote: > > Subject: [PATCH 06/14] RDMA/umem: Split ib_umem_num_pages() into > > ib_umem_num_dma_blocks() > > > > ib_num_pages() should only be used by things working with the SGL in CPU > > pages directly. > > > > Drivers building DMA lists should use the new ib_num_dma_blocks() which returns > > the number of blocks rdma_umem_for_each_block() will return. > > > > Signed-off-by: Jason Gunthorpe <jgg@xxxxxxxxxx> > > drivers/infiniband/hw/cxgb4/mem.c | 2 +- > > drivers/infiniband/hw/mlx5/mem.c | 5 +++-- > > drivers/infiniband/hw/mthca/mthca_provider.c | 2 +- > > drivers/infiniband/hw/vmw_pvrdma/pvrdma_mr.c | 2 +- > > include/rdma/ib_umem.h | 14 +++++++++++--- > > 5 files changed, 17 insertions(+), 8 deletions(-) > > > > Perhaps the one in the bnxt_re too? > > https://elixir.bootlin.com/linux/v5.9-rc3/source/drivers/infiniband/hw/bnxt_re/qplib_res.c#L94 Yes, that needs fixing, but it is not so easy. This patch is just for the easy drivers.. Planning to do it separately as it is the 'easiest' of the remaining 'hard' conversions Jason