Re: [PATCH] IB/iser: Remove support for FMR memory registration

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 5/14/2020 6:50 AM, Max Gurtovoy wrote:

On 5/14/2020 1:09 PM, Alexey Lyashkov wrote:
CX3 with fast registration isn’t stable enough.
I was make this change for Lustre for year or two ago, but it was reverted by serious bugs.

I'm not aware of any issues there.

You can always report it on the mailing list.

Just as a data point, NFS/RDMA and SMB Direct both use FRWR exclusively,
and have been working well at >>100K IOPS over CX3 and all others.

Tom.


Alex
13 мая 2020 г., в 11:43, Max Gurtovoy <maxg@xxxxxxxxxxxx <mailto:maxg@xxxxxxxxxxxx>> написал(а):


On 5/12/2020 11:09 PM, Bart Van Assche wrote:
On 2020-05-12 10:16, Leon Romanovsky wrote:
On Tue, May 12, 2020 at 07:08:59PM +0300, Israel Rukshin wrote:
FMR is not supported on most recent RDMA devices (that use fast memory
registration mechanism). Also, FMR was recently removed from NFS/RDMA
ULP.

Signed-off-by: Israel Rukshin <israelr@xxxxxxxxxxxx <mailto:israelr@xxxxxxxxxxxx>> Signed-off-by: Max Gurtovoy <maxg@xxxxxxxxxxxx <mailto:maxg@xxxxxxxxxxxx>>
---
 drivers/infiniband/ulp/iser/iscsi_iser.h     |  79 +----------
 drivers/infiniband/ulp/iser/iser_initiator.c |  19 ++-
 drivers/infiniband/ulp/iser/iser_memory.c    | 188 ++-------------------------  drivers/infiniband/ulp/iser/iser_verbs.c     | 126 +++---------------
 4 files changed, 40 insertions(+), 372 deletions(-)
Can we do an extra step and remove FMR from srp too?
Which HCA's will be affected by removing FMR support? Or in other words, when did (Mellanox) HCA's start supporting fast memory registration? I'm
asking this because there is a tradition in the Linux kernel not to
remove support for old hardware unless it is pretty sure that nobody is
using that hardware anymore.

ConnectX-3 and above supports fast memory registrations.



Thanks,

Bart.






[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux