mbox series

[rdma-next,v1,0/5] Cleanup of CONFIG_INFINIBAND_ON_DEMAND_PAGING usage

Message ID 20181223091418.26218-1-leon@kernel.org
Headers show
Series Cleanup of CONFIG_INFINIBAND_ON_DEMAND_PAGING usage | expand

Message

Leon Romanovsky Dec. 23, 2018, 9:14 a.m. UTC
From: Leon Romanovsky <leonro@mellanox.com>

Changelog v0->v1:
 * Don't set ODP caps always (patch #2)
 * Set invalidate_range callback based on device_cap only

-------------------------------------------------------------------------------

Hi,

As a followup to Jason's request to rethink CONFIG_INFINIBAND_ON_DEMAND_PAGING
usage, this series cleans mlx5_ib and RDMA/core code and it is based on accepted
patch https://patchwork.kernel.org/patch/10735547/

I think that we will see more patches in that area, I would like to
remove CONFIG_INFINIBAND_ON_DEMAND_PAGING completely.

Thanks

Leon Romanovsky (5):
  RDMA: Clean structures from CONFIG_INFINIBAND_ON_DEMAND_PAGING
  RDMA/core: Don't depend device ODP capabilities on kconfig option
  RDMA/mlx5: Introduce and reuse helper to identify ODP MR
  RDMA/mlx5: Embed into the code flow the ODP config option
  RDMA/mlx5: Delete declaration of already removed function

 drivers/infiniband/core/uverbs_cmd.c |  5 ---
 drivers/infiniband/hw/mlx5/main.c    | 51 ++++++++++++++--------------
 drivers/infiniband/hw/mlx5/mem.c     |  5 +--
 drivers/infiniband/hw/mlx5/mlx5_ib.h | 12 +++++--
 drivers/infiniband/hw/mlx5/mr.c      | 45 +++++++-----------------
 drivers/infiniband/hw/mlx5/odp.c     |  6 ++--
 include/linux/mlx5/driver.h          |  4 ---
 include/rdma/ib_umem_odp.h           | 26 +++++++-------
 include/rdma/ib_verbs.h              |  2 --
 9 files changed, 65 insertions(+), 91 deletions(-)

--
2.19.1

Comments

Jason Gunthorpe Jan. 7, 2019, 6:31 p.m. UTC | #1
On Sun, Dec 23, 2018 at 11:14:13AM +0200, Leon Romanovsky wrote:
> From: Leon Romanovsky <leonro@mellanox.com>
> 
> Changelog v0->v1:
>  * Don't set ODP caps always (patch #2)
>  * Set invalidate_range callback based on device_cap only
> 
> -------------------------------------------------------------------------------
> 
> Hi,
> 
> As a followup to Jason's request to rethink CONFIG_INFINIBAND_ON_DEMAND_PAGING
> usage, this series cleans mlx5_ib and RDMA/core code and it is based on accepted
> patch https://patchwork.kernel.org/patch/10735547/
> 
> I think that we will see more patches in that area, I would like to
> remove CONFIG_INFINIBAND_ON_DEMAND_PAGING completely.
> 
> Thanks
> 
> Leon Romanovsky (5):
>   RDMA: Clean structures from CONFIG_INFINIBAND_ON_DEMAND_PAGING
>   RDMA/core: Don't depend device ODP capabilities on kconfig option
>   RDMA/mlx5: Introduce and reuse helper to identify ODP MR
>   RDMA/mlx5: Embed into the code flow the ODP config option
>   RDMA/mlx5: Delete declaration of already removed function

This doesn't apply anymore, can you resend it please?

Jason
Leon Romanovsky Jan. 8, 2019, 6:18 a.m. UTC | #2
On Mon, Jan 07, 2019 at 11:31:28AM -0700, Jason Gunthorpe wrote:
> On Sun, Dec 23, 2018 at 11:14:13AM +0200, Leon Romanovsky wrote:
> > From: Leon Romanovsky <leonro@mellanox.com>
> >
> > Changelog v0->v1:
> >  * Don't set ODP caps always (patch #2)
> >  * Set invalidate_range callback based on device_cap only
> >
> > -------------------------------------------------------------------------------
> >
> > Hi,
> >
> > As a followup to Jason's request to rethink CONFIG_INFINIBAND_ON_DEMAND_PAGING
> > usage, this series cleans mlx5_ib and RDMA/core code and it is based on accepted
> > patch https://patchwork.kernel.org/patch/10735547/
> >
> > I think that we will see more patches in that area, I would like to
> > remove CONFIG_INFINIBAND_ON_DEMAND_PAGING completely.
> >
> > Thanks
> >
> > Leon Romanovsky (5):
> >   RDMA: Clean structures from CONFIG_INFINIBAND_ON_DEMAND_PAGING
> >   RDMA/core: Don't depend device ODP capabilities on kconfig option
> >   RDMA/mlx5: Introduce and reuse helper to identify ODP MR
> >   RDMA/mlx5: Embed into the code flow the ODP config option
> >   RDMA/mlx5: Delete declaration of already removed function
>
> This doesn't apply anymore, can you resend it please?

Done, thanks

>
> Jason