From mboxrd@z Thu Jan 1 00:00:00 1970 From: Johannes Berg Subject: Re: [PATCH] mac80211: scan: Replace rcu_assign_pointer() with RCU_INIT_POINTER() Date: Tue, 26 Aug 2014 11:16:43 +0200 Message-ID: <1409044603.2489.7.camel@jlt4.sipsolutions.net> References: <20140822131449.GA19959@ada> (sfid-20140822_151532_626084_CD0D5F37) Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Cc: linville@tuxdriver.com, davem@davemloft.net, linux-wireless@vger.kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, paulmck@linux.vnet.ibm.com To: Andreea-Cristina Bernat Return-path: Received: from s3.sipsolutions.net ([5.9.151.49]:38987 "EHLO sipsolutions.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933975AbaHZJSc (ORCPT ); Tue, 26 Aug 2014 05:18:32 -0400 In-Reply-To: <20140822131449.GA19959@ada> (sfid-20140822_151532_626084_CD0D5F37) Sender: netdev-owner@vger.kernel.org List-ID: On Fri, 2014-08-22 at 16:14 +0300, Andreea-Cristina Bernat wrote: > The use of "rcu_assign_pointer()" is NULLing out the pointer. > According to RCU_INIT_POINTER()'s block comment: > "1. This use of RCU_INIT_POINTER() is NULLing out the pointer" > it is better to use it instead of rcu_assign_pointer() because it has a > smaller overhead. Applied, thanks. johannes