Re: [PATCH for-next v14 00/10] Fix race conditions in rxe_pool

[Bob Pearson <rpearsonhpe@gmail.com>]

On 5/24/22 06:57, Jason Gunthorpe wrote:
> On Tue, May 24, 2022 at 03:53:30AM +0000, yangx.jy@fujitsu.com wrote:
>> On 2022/5/10 2:23, Jason Gunthorpe wrote:
>>> On Wed, Apr 20, 2022 at 08:40:33PM -0500, Bob Pearson wrote:
>>>
>>>> Bob Pearson (10):
>>>>    RDMA/rxe: Remove IB_SRQ_INIT_MASK
>>>>    RDMA/rxe: Add rxe_srq_cleanup()
>>>>    RDMA/rxe: Check rxe_get() return value
>>>>    RDMA/rxe: Move qp cleanup code to rxe_qp_do_cleanup()
>>>>    RDMA/rxe: Move mr cleanup code to rxe_mr_cleanup()
>>>>    RDMA/rxe: Move mw cleanup code to rxe_mw_cleanup()
>>>>    RDMA/rxe: Enforce IBA C11-17
>>>
>>> I took these patches with the small edits I noted
>>>
>>>>    RDMA/rxe: Stop lookup of partially built objects
>>>>    RDMA/rxe: Convert read side locking to rcu
>>>>    RDMA/rxe: Cleanup rxe_pool.c
>>>
>>> It seems OK, but we need to fix the AH problem at least in the destroy
>>> path first - lets try to fix it in alloc as well?
>> Hi Jason, Bob
>>
>> Could you tell me what the AH problem is? Thanks a lot.
> 
> rxe doesn't implement RDMA_CREATE_AH_SLEEPABLE /
> RDMA_DESTROY_AH_SLEEPABLE
> 
> Jason

First I have heard of those. Should we implement them?

Bob