On Thu, Apr 8, 2021 at 4:51 PM Jason Gunthorpe <jgg@xxxxxxxxxx> wrote: > > On Thu, Apr 08, 2021 at 04:44:02PM +0200, Gioh Kim wrote: > > > I think it might be a solution to change the > > rtrs_clt_remove_path_from_sysfs as below. It changes the order: > > first remove the session from list and then destroy sess->stat > > memory. > > Freeing memory used under a RCU lock before doing a RCU > synchronization is an error, so at least this could make sense to me > > Jason Hi Jason, I just sent a patch "RDMA/rtrs-clt: destroy sysfs after removing session from active list". Thank you for the review.