On Tue, Aug 09, 2016 at 04:20:08PM +0000, Marciniszyn, Mike wrote: > > > > -----Original Message----- > > From: Wei Yongjun [mailto:weiyj.lk@xxxxxxxxx] > > Sent: Monday, August 8, 2016 5:50 AM > > To: Marciniszyn, Mike <mike.marciniszyn@xxxxxxxxx>; Dalessandro, Dennis > > <dennis.dalessandro@xxxxxxxxx>; Doug Ledford <dledford@xxxxxxxxxx>; > > Hefty, Sean <sean.hefty@xxxxxxxxx>; Hal Rosenstock > > <hal.rosenstock@xxxxxxxxx> > > Cc: Wei Yongjun <weiyj.lk@xxxxxxxxx>; linux-rdma@xxxxxxxxxxxxxxx > > Subject: [PATCH] IB/hfi1: Using kfree_rcu() to simplify the code > > > > The callback function of call_rcu() just calls a kfree(), so we can use > > kfree_rcu() instead of call_rcu() + callback function. > > > > Signed-off-by: Wei Yongjun <weiyj.lk@xxxxxxxxx> > > --- > > drivers/infiniband/hw/hfi1/mad.c | 9 +-------- > > 1 file changed, 1 insertion(+), 8 deletions(-) > > > > diff --git a/drivers/infiniband/hw/hfi1/mad.c > > b/drivers/infiniband/hw/hfi1/mad.c > > index 1263abe..95c43e1 100644 > > --- a/drivers/infiniband/hw/hfi1/mad.c > > +++ b/drivers/infiniband/hw/hfi1/mad.c > > @@ -3398,7 +3398,7 @@ static void apply_cc_state(struct hfi1_pportdata > > *ppd) > > > > spin_unlock(&ppd->cc_state_lock); > > > > - call_rcu(&old_cc_state->rcu, cc_state_reclaim); > > + kfree_rcu(old_cc_state, rcu); > > } > > > > This patch misses another call_rcu() that needs to be replaced in cleanup_device_data() > and that seems to be causing 0day issues. > > Doug, have you put this anywhere to elicit the 0day issues? 0day bot runs on patches in mailing list too. > > Nacked-by: Mike Marciniszyn <mike.marciniszyn@xxxxxxxxx> > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html
Attachment:
signature.asc
Description: Digital signature