From patchwork Sun May 16 13:02:12 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sven Eckelmann X-Patchwork-Id: 149 Return-Path: Received: from mail.gmx.net (mail.gmx.net [213.165.64.20]) by open-mesh.net (Postfix) with SMTP id 4B5BE1543A8 for ; Sun, 16 May 2010 09:38:31 +0200 (CEST) Received: (qmail invoked by alias); 16 May 2010 13:02:16 -0000 Received: from p4FCDFB80.dip.t-dialin.net (EHLO localhost) [79.205.251.128] by mail.gmx.net (mp069) with SMTP; 16 May 2010 15:02:16 +0200 X-Authenticated: #15668376 X-Provags-ID: V01U2FsdGVkX18NuxlmlzwNL6JYTnHf9j02VBfMvrmrxxA+QGbrbD /wX6WjN/XEGild From: Sven Eckelmann To: b.a.t.m.a.n@lists.open-mesh.org Date: Sun, 16 May 2010 15:02:12 +0200 Message-Id: <1274014932-2792-1-git-send-email-sven.eckelmann@gmx.de> X-Mailer: git-send-email 1.7.1 In-Reply-To: <1273868969293@kroah.org> References: <1273868969293@kroah.org> X-Y-GMX-Trusted: 0 Subject: [B.A.T.M.A.N.] [PATCH] batman-adv: remove all rcu head initializations X-BeenThere: b.a.t.m.a.n@lists.open-mesh.org X-Mailman-Version: 2.1.11 Precedence: list Reply-To: The list for a Better Approach To Mobile Ad-hoc Networking List-Id: The list for a Better Approach To Mobile Ad-hoc Networking List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 16 May 2010 07:38:31 -0000 From: Paul E. McKenney Remove all rcu head inits. We don't care about the RCU head state before passing it to call_rcu() anyway. Only leave the "on_stack" variants so debugobjects can keep track of objects on stack. Signed-off-by: Mathieu Desnoyers Signed-off-by: Paul E. McKenney Cc: Andrew Lunn Cc: Simon Wunderlich Cc: Marek Lindner Cc: Sven Eckelmann Signed-off-by: Greg Kroah-Hartman Signed-off-by: Sven Eckelmann --- I've tried to check if the initialisation was always done in call_rcu in all implementations since 2.6.21. This assumption seems to be correct. I've already applied it to the maint branch and it must only be applied to the trunk. hard-interface.c | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/hard-interface.c b/hard-interface.c index c17ad85..bc2bd12 100644 --- a/hard-interface.c +++ b/hard-interface.c @@ -341,7 +341,6 @@ static struct batman_if *hardif_add_interface(struct net_device *net_dev) batman_if->if_num = -1; batman_if->net_dev = net_dev; batman_if->if_status = IF_NOT_IN_USE; - INIT_RCU_HEAD(&batman_if->rcu); INIT_LIST_HEAD(&batman_if->list); check_known_mac_addr(batman_if->net_dev->dev_addr);