On 5/24/22 06:57, Jason Gunthorpe wrote: > On Tue, May 24, 2022 at 03:53:30AM +0000, yangx.jy@xxxxxxxxxxx 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