Re: [Intel-gfx] [PATCH rdma-next v3 0/3] SG fix together with update to RDMA umem

2021-08-20 Thread Jason Gunthorpe
On Thu, Jul 29, 2021 at 12:39:10PM +0300, Leon Romanovsky wrote:
> From: Leon Romanovsky 
> 
> Changelog:
> v3:
>  * Rewrote to new API suggestion
>  * Split for more patches
> v2: https://lore.kernel.org/lkml/cover.1626605893.git.leo...@nvidia.com
>  * Changed implementation of first patch, based on our discussion with 
> Christoph.
>https://lore.kernel.org/lkml/ynwavtt0qmqdx...@infradead.org/
> v1: https://lore.kernel.org/lkml/cover.1624955710.git.leo...@nvidia.com/
>  * Fixed sg_page with a _dma_ API in the umem.c
> v0: https://lore.kernel.org/lkml/cover.1624361199.git.leo...@nvidia.com
> 
> 
> Maor Gottlieb (3):
>   lib/scatterlist: Provide a dedicated function to support table append
>   lib/scatterlist: Fix wrong update of orig_nents
>   RDMA: Use the sg_table directly and remove the opencoded version from
> umem

I'm going to send this into linux-next, last time that triggered some
bug reports.

But overall it looks okay, though some of the sg_append_table is bit
odd. Certainly using the sg_table throughout the RDMA code is big
improvement.

Lets see a v4, reviews/etc and I'll update it.

Jason


[Intel-gfx] [PATCH rdma-next v3 0/3] SG fix together with update to RDMA umem

2021-07-29 Thread Leon Romanovsky
From: Leon Romanovsky 

Changelog:
v3:
 * Rewrote to new API suggestion
 * Split for more patches
v2: https://lore.kernel.org/lkml/cover.1626605893.git.leo...@nvidia.com
 * Changed implementation of first patch, based on our discussion with 
Christoph.
   https://lore.kernel.org/lkml/ynwavtt0qmqdx...@infradead.org/
v1: https://lore.kernel.org/lkml/cover.1624955710.git.leo...@nvidia.com/
 * Fixed sg_page with a _dma_ API in the umem.c
v0: https://lore.kernel.org/lkml/cover.1624361199.git.leo...@nvidia.com


Maor Gottlieb (3):
  lib/scatterlist: Provide a dedicated function to support table append
  lib/scatterlist: Fix wrong update of orig_nents
  RDMA: Use the sg_table directly and remove the opencoded version from
umem

 drivers/gpu/drm/drm_prime.c |  13 +-
 drivers/gpu/drm/i915/gem/i915_gem_userptr.c |  11 +-
 drivers/gpu/drm/vmwgfx/vmwgfx_ttm_buffer.c  |  14 +-
 drivers/infiniband/core/umem.c  |  56 +++---
 drivers/infiniband/core/umem_dmabuf.c   |   5 +-
 drivers/infiniband/hw/hns/hns_roce_db.c |   4 +-
 drivers/infiniband/hw/irdma/verbs.c |   2 +-
 drivers/infiniband/hw/mlx4/doorbell.c   |   3 +-
 drivers/infiniband/hw/mlx4/mr.c |   4 +-
 drivers/infiniband/hw/mlx5/doorbell.c   |   3 +-
 drivers/infiniband/hw/mlx5/mr.c |   3 +-
 drivers/infiniband/hw/qedr/verbs.c  |   2 +-
 drivers/infiniband/sw/rdmavt/mr.c   |   2 +-
 drivers/infiniband/sw/rxe/rxe_mr.c  |   2 +-
 include/linux/scatterlist.h |  54 +-
 include/rdma/ib_umem.h  |  11 +-
 include/rdma/ib_verbs.h |  28 +++
 lib/scatterlist.c   | 189 
 tools/testing/scatterlist/main.c|  38 ++--
 19 files changed, 275 insertions(+), 169 deletions(-)

-- 
2.31.1

___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx