On Thu, Oct 28, 2021 at 12:56:46AM -0700, Christoph Hellwig wrote: > On Thu, Oct 28, 2021 at 08:55:22AM +0300, Leon Romanovsky wrote: > > From: Aharon Landau <aharonl@xxxxxxxxxx> > > > > The vendors set the IOVA of newly created MRs in rereg_user_mr, so don't > > overwrite it. That ensures that this field is set only if IB_MR_REREG_TRANS > > flag is provided. > > How is a "vendor" involved with this? This should all be upstream code. "vendor" is wrong word here. I wanted to say that all drivers which support ".rereg_user_mr()" callback and return new_mr should set everything. In case of IB_MR_REREG_TRANS flow, it is IOVA which is not cmd.hca_va, but mr->iova. Thanks
