From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934930AbcA0Xj7 (ORCPT ); Wed, 27 Jan 2016 18:39:59 -0500 Received: from kirsty.vergenet.net ([202.4.237.240]:46390 "EHLO kirsty.vergenet.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934577AbcA0Xj5 (ORCPT ); Wed, 27 Jan 2016 18:39:57 -0500 Date: Thu, 28 Jan 2016 08:39:53 +0900 From: Simon Horman To: Julian Anastasov Cc: Arnd Bergmann , Wensong Zhang , linux-arm-kernel@lists.infradead.org, Pablo Neira Ayuso , Patrick McHardy , Jozsef Kadlecsik , "David S. Miller" , netdev@vger.kernel.org, lvs-devel@vger.kernel.org, netfilter-devel@vger.kernel.org, coreteam@netfilter.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/2] netfilter: ipvs: avoid unused variable warnings Message-ID: <20160127233953.GB5183@verge.net.au> References: <1453902749-3422685-1-git-send-email-arnd@arndb.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Organisation: Horms Solutions Ltd. User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jan 27, 2016 at 10:01:42PM +0200, Julian Anastasov wrote: > > Hello, > > On Wed, 27 Jan 2016, Arnd Bergmann wrote: > > > The proc_create() and remove_proc_entry() functions do not reference > > their arguments when CONFIG_PROC_FS is disabled, so we get a couple > > of warnings about unused variables in IPVS: > > > > ipvs/ip_vs_app.c:608:14: warning: unused variable 'net' [-Wunused-variable] > > ipvs/ip_vs_ctl.c:3950:14: warning: unused variable 'net' [-Wunused-variable] > > ipvs/ip_vs_ctl.c:3994:14: warning: unused variable 'net' [-Wunused-variable] > > > > This removes the local variables and instead looks them up separately > > for each use, which obviously avoids the warning. > > > > Signed-off-by: Arnd Bergmann > > Fixes: 4c50a8ce2b63 ("netfilter: ipvs: avoid unused variable warning") > > Looks like your previous patch for ip_vs_app_net_cleanup > was delayed in ipvs-next tree. I guess, Simon should drop it and > use this one instead when net-next opens: > > Acked-by: Julian Anastasov Thanks, and sorry about not pushing the other patch to net-next. I have dropped it and queued up this one in its place.