From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758870AbaCTQ0P (ORCPT ); Thu, 20 Mar 2014 12:26:15 -0400 Received: from mail-gw1-out.broadcom.com ([216.31.210.62]:58057 "EHLO mail-gw1-out.broadcom.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756925AbaCTQ0N (ORCPT ); Thu, 20 Mar 2014 12:26:13 -0400 X-IronPort-AV: E=Sophos;i="4.97,695,1389772800"; d="scan'208";a="20716303" Message-ID: <532B169F.9030707@broadcom.com> Date: Thu, 20 Mar 2014 17:26:07 +0100 From: Arend van Spriel User-Agent: Mozilla/5.0 (X11; U; Linux i686 (x86_64); en-US; rv:1.9.2.24) Gecko/20111103 Lightning/1.0b2 Thunderbird/3.1.16 MIME-Version: 1.0 To: CC: Monam Agarwal , , , Subject: Re: [PATCH 0/9] Replace rcu_assign_pointer(x, NULL) with RCU_INIT_POINTER(x,NULL) References: <532B022C.8010505@broadcom.com> <20140320150601.GK4405@linux.vnet.ibm.com> In-Reply-To: <20140320150601.GK4405@linux.vnet.ibm.com> Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 03/20/14 16:06, Paul E. McKenney wrote: > On Thu, Mar 20, 2014 at 03:58:52PM +0100, Arend van Spriel wrote: >> On 20/03/14 15:37, Monam Agarwal wrote: >>> This patchset uses following coccinelle script to replace >>> rcu_assign_pointer(x, NULL) with RCU_INIT_POINTER(x,NULL) >>> >>> @@ >>> expression E; >>> @@ >>> - rcu_assign_pointer( >>> + RCU_INIT_POINTER( >>> E, NULL) >> >> I am not very familiar with rcu functionality nor coccinelle, but it >> looks too generic. I would think only NULL assignments during >> initialization should use RCU_INIT_POINTER(). The first patch in the >> series does it in a function called unregister_...(). Is it really >> ok to do there? > > Hello, Arend, > > The thing that rcu_assign_pointer() is doing is ensuring that the > initialization of a structure is carried out before storing a pointer > to that structure. In the case of the NULL pointer, there is no > structure to initialize, so nothing need be ordered. > > So, yes, rcu_assign_pointer(p, NULL) can always safely be converted to > RCU_INIT_POINTER(p, NULL). ;-) And so I learned something today. Thanks, Paul. I really should pick up reading rcu material again. It ended up in a dusty corner upstairs. Thanks, Arend