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

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:

quoted

quoted

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.

-- 
Doug Ledford [off-list ref]
              GPG KeyID: 0E572FDD