Re: [PATCH for-next 1/2] IB/uverbs: Fix race between uverbs_close and remove_one

[Leon Romanovsky <leon-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>]

[-- Attachment #1: Type: text/plain, Size: 1278 bytes --]

On Tue, Aug 02, 2016 at 10:43:14PM -0400, Doug Ledford wrote:
> On Tue, 2016-08-02 at 13:26 -0600, Jason Gunthorpe wrote:
> > On Tue, Aug 02, 2016 at 09:50:14PM +0300, Leon Romanovsky wrote:
> > > 
> > > > 
> > > > The only reason I hadn't taken this patch before is because Jason
> > > > said
> > > > it was totally untested and someone (Devesh in this case) needed
> > > > to
> > > > test it to make sure it resolved their problem.  I don't see a
> > > > test-by
> > > > line here, so has this happened?
> > > 
> > > I can't say about Devesh, but we verified it as part of our
> > > verification phase.
> > > Do you need anything special except Yishai's SOB?
> > 
> > I was hoping to see Devesh confirm that the race he was able to
> > reproduce is in fact closed, unless Mellanox did the reproduction?
> > 
> > At least your testing shows it doesn't make things worse, so it is
> > probably OK to go ahead if Devesh does not respond soon.
> > 
> > Jason
> 
> I'm not sure what Devesh is off doing now a days, so I went ahead and
> took this.  With the fact that Mellanox tested it, I'd rather have it
> in than out.

Thanks

> 
> -- 
> Doug Ledford <dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
>               GPG KeyID: 0E572FDD



[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]