Re: [PATCH] RDMA/irdma: Initialize iwmr->access during MR registration

[Jason Gunthorpe <jgg@nvidia.com>]

On Thu, Jun 04, 2026 at 03:41:04PM +0000, Jacob Moroni wrote:
> Initialize iwmr->access during initial user mem registration so
> that it contains a valid value during a subsequent rereg_mr.
> 
> Otherwise, a rereg_mr that doesn't set IB_MR_REREG_ACCESS (for
> example, one that only changes the PD) ends up clearing the
> access flags in HW since iwmr->access is zero-initialized, which
> is not intended.
> 
> Fixes: 5ac388db27c4 ("RDMA/irdma: Add support to re-register a memory region")
> Signed-off-by: Jacob Moroni <jmoroni@google.com>
> ---
>  drivers/infiniband/hw/irdma/verbs.c | 1 +
>  1 file changed, 1 insertion(+)

Applied to for-next

Still more preexisting Sashiko issues:

https://sashiko.dev/#/patchset/20260604154104.4035581-1-jmoroni%40google.com

This one sounds bad:

 This is a pre-existing issue, but could this sequence allow arbitrary
 physical memory access when an MR re-registration fails?

Sigh, rereg_mr is a trainwreck in every driver.

I fixed mlx5 a long time ago, there is a new flow now for rereg where
the driver can create a new mr from scratch and return that. irdma
should do that for any destructive change like replacing the
umem. Otherwise it is impossible to get the error semantics correct.

Jason