From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Paul E. McKenney" Subject: Re: [2.6 patch] kernel/: small cleanups Date: Sun, 30 Oct 2005 12:53:45 -0800 Message-ID: <20051030205345.GA32111@us.ibm.com> References: <20051030010200.GT4180@stusta.de> Reply-To: paulmck@us.ibm.com Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: linux-kernel@vger.kernel.org, dwmw2@infradead.org, linux-audit@redhat.com, faith@redhat.com, george@mvista.com, netdev@vger.kernel.org Return-path: To: Adrian Bunk Content-Disposition: inline In-Reply-To: <20051030010200.GT4180@stusta.de> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On Sun, Oct 30, 2005 at 02:02:00AM +0100, Adrian Bunk wrote: > This patch contains the following cleanups: > - make needlessly global functions static Good catch on rcu_torture_alloc()! Thanx, Paul > - every file should include the headers containing the prototypes for > it's global functions Acked-by: > Signed-off-by: Adrian Bunk > > --- > > kernel/audit.c | 2 +- > kernel/irq/proc.c | 2 ++ > kernel/rcutorture.c | 2 +- > kernel/timer.c | 1 + > 4 files changed, 5 insertions(+), 2 deletions(-) > > --- linux-2.6.14-rc5-mm1-full/kernel/timer.c.old 2005-10-30 02:27:36.000000000 +0200 > +++ linux-2.6.14-rc5-mm1-full/kernel/timer.c 2005-10-30 02:27:56.000000000 +0200 > @@ -33,6 +33,7 @@ > #include > #include > #include > +#include > > #include > #include > --- linux-2.6.14-rc5-mm1-full/kernel/irq/proc.c.old 2005-10-30 02:31:31.000000000 +0200 > +++ linux-2.6.14-rc5-mm1-full/kernel/irq/proc.c 2005-10-30 02:31:48.000000000 +0200 > @@ -10,6 +10,8 @@ > #include > #include > > +#include "internals.h" > + > static struct proc_dir_entry *root_irq_dir, *irq_dir[NR_IRQS]; > > #ifdef CONFIG_SMP > --- linux-2.6.14-rc5-mm1-full/kernel/audit.c.old 2005-10-30 02:33:08.000000000 +0200 > +++ linux-2.6.14-rc5-mm1-full/kernel/audit.c 2005-10-30 02:33:15.000000000 +0200 > @@ -272,7 +272,7 @@ > return old; > } > > -int kauditd_thread(void *dummy) > +static int kauditd_thread(void *dummy) > { > struct sk_buff *skb; > > --- linux-2.6.14-rc5-mm1-full/kernel/rcutorture.c.old 2005-10-30 02:33:35.000000000 +0200 > +++ linux-2.6.14-rc5-mm1-full/kernel/rcutorture.c 2005-10-30 02:33:53.000000000 +0200 > @@ -99,7 +99,7 @@ > /* > * Allocate an element from the rcu_tortures pool. > */ > -struct rcu_torture * > +static struct rcu_torture * > rcu_torture_alloc(void) > { > struct list_head *p; > >