From: Jason Gunthorpe <jgg@ziepe.ca>
To: linux-rdma@vger.kernel.org, netdev@vger.kernel.org
Cc: Ariel Elior <aelior@marvell.com>,
aron.silverton@oracle.com, Bernard Metzler <bmt@zurich.ibm.com>,
Bart Van Assche <bvanassche@acm.org>,
Dennis Dalessandro <dennis.dalessandro@intel.com>,
Devesh Sharma <devesh.sharma@broadcom.com>,
Faisal Latif <faisal.latif@intel.com>,
Gal Pressman <galpress@amazon.com>,
Israel Rukshin <israelr@mellanox.com>,
Leon Romanovsky <leonro@mellanox.com>,
Max Gurtovoy <maxg@mellanox.com>,
Mike Marciniszyn <mike.marciniszyn@intel.com>,
Michal Kalderon <mkalderon@marvell.com>,
oren@mellanox.com, Sagi Grimberg <sagi@grimberg.me>,
santosh.shilimkar@oracle.com,
Selvin Xavier <selvin.xavier@broadcom.com>,
Shiraz Saleem <shiraz.saleem@intel.com>,
shlomin@mellanox.com, Somnath Kotur <somnath.kotur@broadcom.com>,
Sriharsha Basavapatna <sriharsha.basavapatna@broadcom.com>,
vladimirk@mellanox.com, Yishai Hadas <yishaih@mellanox.com>
Subject: Re: [PATCH v3 00/13] Remove FMR support from RDMA drivers
Date: Fri, 29 May 2020 16:17:57 -0300 [thread overview]
Message-ID: <20200529191757.GA12325@ziepe.ca> (raw)
In-Reply-To: <0-v3-f58e6669d5d3+2cf-fmr_removal_jgg@mellanox.com>
On Thu, May 28, 2020 at 04:45:42PM -0300, Jason Gunthorpe wrote:
> From: Jason Gunthorpe <jgg@mellanox.com>
>
> This series removes the support for FMR mode to register memory. This
> ancient mode is unsafe (rkeys that are usually exposed for caching
> purposes and the API is limited to page granularity mappings) and not
> maintained/tested in the last few years. It also doesn't have any
> reasonable advantage over other memory registration methods such as
> FRWR (that is implemented in all the recent RDMA adapters). This series
> should be reviewed and approved by the maintainer of the effected drivers
> and I suggest to test it as well.
>
> Changes from V2:
> - Removed more occurances of _fmr
> - Remove max_map_per_fmr device attribute
> - Remove max_fmr device attribute
> - Remove additional dead code from bnxt_re and i40iw
> - Revised RDS to not use ib_fmr_attr or other fmr things
> - Rebased on RDMA for-next
> Changes from V1:
> https://lore.kernel.org/linux-rdma/20200527094634.24240-1-maxg@mellanox.com/
> - added "RDMA/mlx5: Remove FMR leftovers" (from GalP)
> - rebased on top of "Linux 5.7-rc7"
> - added "Reviewed-by" Bart signature for SRP
>
> Cc: shlomin@mellanox.com
> Cc: vladimirk@mellanox.com
> Cc: oren@mellanox.com
>
> Gal Pressman (1):
> RDMA/mlx5: Remove FMR leftovers
>
> Israel Rukshin (1):
> RDMA/iser: Remove support for FMR memory registration
>
> Jason Gunthorpe (4):
> RDMA/bnxt_re: Remove FMR leftovers
> RDMA/i40iw: Remove FMR leftovers
> RDMA: Remove 'max_fmr'
> RDMA: Remove 'max_map_per_fmr'
>
> Max Gurtovoy (7):
> RDMA/srp: Remove support for FMR memory registration
> RDMA/rds: Remove FMR support for memory registration
> RDMA/core: Remove FMR pool API
> RDMA/mlx4: Remove FMR support for memory registration
> RDMA/mthca: Remove FMR support for memory registration
> RDMA/rdmavt: Remove FMR memory registration
> RDMA/core: Remove FMR device ops
Applied to for-next
Jason
prev parent reply other threads:[~2020-05-29 19:18 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-05-28 19:45 [PATCH v3 00/13] Remove FMR support from RDMA drivers Jason Gunthorpe
2020-05-28 19:45 ` [PATCH v3 01/13] RDMA/iser: Remove support for FMR memory registration Jason Gunthorpe
2020-05-28 19:45 ` [PATCH v3 02/13] RDMA/srp: " Jason Gunthorpe
2020-05-28 19:45 ` [PATCH v3 03/13] RDMA/rds: Remove FMR support for " Jason Gunthorpe
2020-05-28 20:21 ` santosh.shilimkar
2020-05-29 19:12 ` Jason Gunthorpe
2020-06-16 2:33 ` santosh.shilimkar
2020-05-28 19:45 ` [PATCH v3 04/13] RDMA/core: Remove FMR pool API Jason Gunthorpe
2020-05-28 19:45 ` [PATCH v3 05/13] RDMA/mlx5: Remove FMR leftovers Jason Gunthorpe
2020-05-28 19:45 ` [PATCH v3 06/13] RDMA/bnxt_re: " Jason Gunthorpe
2020-05-28 19:45 ` [PATCH v3 07/13] RDMA/i40iw: " Jason Gunthorpe
2020-05-29 15:20 ` Saleem, Shiraz
2020-05-28 19:45 ` [PATCH v3 08/13] RDMA/mlx4: Remove FMR support for memory registration Jason Gunthorpe
2020-05-28 19:45 ` [PATCH v3 09/13] RDMA/mthca: " Jason Gunthorpe
2020-05-28 19:45 ` [PATCH v3 10/13] RDMA/rdmavt: Remove FMR " Jason Gunthorpe
2020-05-28 19:45 ` [PATCH v3 11/13] RDMA/core: Remove FMR device ops Jason Gunthorpe
2020-05-28 19:45 ` [PATCH v3 12/13] RDMA: Remove 'max_fmr' Jason Gunthorpe
2020-05-29 14:54 ` Bernard Metzler
2020-05-28 19:45 ` [PATCH v3 13/13] RDMA: Remove 'max_map_per_fmr' Jason Gunthorpe
2020-05-29 19:17 ` Jason Gunthorpe [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200529191757.GA12325@ziepe.ca \
--to=jgg@ziepe.ca \
--cc=aelior@marvell.com \
--cc=aron.silverton@oracle.com \
--cc=bmt@zurich.ibm.com \
--cc=bvanassche@acm.org \
--cc=dennis.dalessandro@intel.com \
--cc=devesh.sharma@broadcom.com \
--cc=faisal.latif@intel.com \
--cc=galpress@amazon.com \
--cc=israelr@mellanox.com \
--cc=leonro@mellanox.com \
--cc=linux-rdma@vger.kernel.org \
--cc=maxg@mellanox.com \
--cc=mike.marciniszyn@intel.com \
--cc=mkalderon@marvell.com \
--cc=netdev@vger.kernel.org \
--cc=oren@mellanox.com \
--cc=sagi@grimberg.me \
--cc=santosh.shilimkar@oracle.com \
--cc=selvin.xavier@broadcom.com \
--cc=shiraz.saleem@intel.com \
--cc=shlomin@mellanox.com \
--cc=somnath.kotur@broadcom.com \
--cc=sriharsha.basavapatna@broadcom.com \
--cc=vladimirk@mellanox.com \
--cc=yishaih@mellanox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).