From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Paul E. McKenney" Subject: Re: [PATCH 2/9] rcu: Replace rcu_assign_pointer(x, NULL) with RCU_INIT_POINTER(x, NULL) in drivers/net/ethernet/chelsio/cxgb3/cxgb3_offload.c Date: Tue, 18 Mar 2014 16:29:06 -0700 Message-ID: <20140318232906.GB4420@linux.vnet.ibm.com> References: <23ae65fa8bad7f9092739a52e35fdf9d4e8576a3.1395182132.git.monamagarwal123@gmail.com> Reply-To: paulmck@linux.vnet.ibm.com Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, eddie.wai@broadcom.com To: Monam Agarwal Return-path: Content-Disposition: inline In-Reply-To: <23ae65fa8bad7f9092739a52e35fdf9d4e8576a3.1395182132.git.monamagarwal123@gmail.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On Wed, Mar 19, 2014 at 04:12:16AM +0530, Monam Agarwal wrote: > Signed-off-by: Monam Agarwal Reviewed-by: Paul E. McKenney > --- > drivers/net/ethernet/chelsio/cxgb3/cxgb3_offload.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/net/ethernet/chelsio/cxgb3/cxgb3_offload.c b/drivers/net/ethernet/chelsio/cxgb3/cxgb3_offload.c > index da5a5d9..a7ec886 100644 > --- a/drivers/net/ethernet/chelsio/cxgb3/cxgb3_offload.c > +++ b/drivers/net/ethernet/chelsio/cxgb3/cxgb3_offload.c > @@ -1300,7 +1300,7 @@ int cxgb3_offload_activate(struct adapter *adapter) > > out_free_l2t: > t3_free_l2t(L2DATA(dev)); > - rcu_assign_pointer(dev->l2opt, NULL); > + RCU_INIT_POINTER(dev->l2opt, NULL); > out_free: > kfree(t); > return err; > @@ -1328,7 +1328,7 @@ void cxgb3_offload_deactivate(struct adapter *adapter) > rcu_read_lock(); > d = L2DATA(tdev); > rcu_read_unlock(); > - rcu_assign_pointer(tdev->l2opt, NULL); > + RCU_INIT_POINTER(tdev->l2opt, NULL); > call_rcu(&d->rcu_head, clean_l2_data); > if (t->nofail_skb) > kfree_skb(t->nofail_skb); > -- > 1.7.9.5 >