RE: [PATCH 1/2] IB/core: correct issue with sge copyin corrupting wr

2015-11-30 Thread Marciniszyn, Mike
> Being a non native English speaker I am not fully sure, but "copyin > corrupting" sounds like slang / street talking... proper English please > > > Commit e622f2f4ad21 ("IB: split struct ib_send_wr") introduced a > > regression for HCAs whose user mode post sends go through > > ib_uverbs_post_se

Re: [PATCH 1/2] IB/core: correct issue with sge copyin corrupting wr

2015-11-30 Thread Or Gerlitz
On Mon, Nov 30, 2015 at 4:34 PM, Mike Marciniszyn wrote: Being a non native English speaker I am not fully sure, but "copyin corrupting" sounds like slang / street talking... proper English please > Commit e622f2f4ad21 ("IB: split struct ib_send_wr") > introduced a regression for HCAs whose user

[PATCH 1/2] IB/core: correct issue with sge copyin corrupting wr

2015-11-30 Thread Mike Marciniszyn
Commit e622f2f4ad21 ("IB: split struct ib_send_wr") introduced a regression for HCAs whose user mode post sends go through ib_uverbs_post_send(). The code didn't account for the fact that the first sge is offset by an operation dependent length. The allocation did, but the pointer to the sge list