Hi Jason, Doug, > -----Original Message----- > From: linux-rdma-owner@xxxxxxxxxxxxxxx [mailto:linux-rdma- > owner@xxxxxxxxxxxxxxx] On Behalf Of Parav Pandit > Sent: Thursday, March 15, 2018 11:37 AM > To: Jason Gunthorpe <jgg@xxxxxxxx>; Leon Romanovsky <leon@xxxxxxxxxx> > Cc: Doug Ledford <dledford@xxxxxxxxxx>; RDMA mailing list <linux- > rdma@xxxxxxxxxxxxxxx>; Daniel Jurgens <danielj@xxxxxxxxxxxx>; Mark Bloch > <markb@xxxxxxxxxxxx> > Subject: RE: [PATCH rdma-next v1 09/13] IB/cm: Avoid AV ah_attr overwriting > during LAP message handling > > > > > -----Original Message----- > > From: Jason Gunthorpe [mailto:jgg@xxxxxxxx] > > Sent: Thursday, March 15, 2018 9:30 AM > > To: Leon Romanovsky <leon@xxxxxxxxxx> > > Cc: Parav Pandit <parav@xxxxxxxxxxxx>; Doug Ledford > > <dledford@xxxxxxxxxx>; RDMA mailing list <linux-rdma@xxxxxxxxxxxxxxx>; > > Daniel Jurgens <danielj@xxxxxxxxxxxx>; Mark Bloch <markb@xxxxxxxxxxxx> > > Subject: Re: [PATCH rdma-next v1 09/13] IB/cm: Avoid AV ah_attr > > overwriting during LAP message handling > > > > On Thu, Mar 15, 2018 at 10:00:17AM +0200, Leon Romanovsky wrote: > > > > > > But.. this patch is still complicated, it is hard to see if all > > > > the readers are safe or not.. > > > > > > It was tested, no new bugs were discovered, so it is safe to assume > > > that Parav's series are not breaking IB and RoCE and safe for > > > inclusion > > > > Are you kidding? The patch has obvious data races. > > > > Testing doesn't discover those. > > > In that case let's get ucm cleanup full plan in place where ucm is removed and > APIs only used by ucm also removed. > After that I attempt to fix this path if the code exist. So you can continue to review the rest of the series and drop this patch. In subsequent series too, I won't touch LAP code which doesn't have active user. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
