On Thu, Jan 27, 2022 at 03:37:41PM -0600, Bob Pearson wrote: > > struct rxe_mcg { > struct rxe_pool_elem elem; > + struct rb_node node; > struct rxe_dev *rxe; > struct list_head qp_list; > atomic_t qp_num; > @@ -397,6 +398,8 @@ struct rxe_dev { > struct rxe_pool mc_grp_pool; > > spinlock_t mcg_lock; /* guard multicast groups */ I think you should probably just use a mutex here and simplify things.. Jason