public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [2.6 patch] make kernel/sched.c:account_guest_time() static
@ 2007-10-24 16:25 Adrian Bunk
  2007-10-24 16:50 ` Ingo Molnar
  0 siblings, 1 reply; 4+ messages in thread
From: Adrian Bunk @ 2007-10-24 16:25 UTC (permalink / raw)
  To: Laurent Vivier, Avi Kivity, Ingo Molnar; +Cc: linux-kernel

account_guest_time() can become static.

Signed-off-by: Adrian Bunk <bunk@kernel.org>

---
2779ceb45e015ce32451d20c3724a8ad2cd26462 
diff --git a/kernel/sched.c b/kernel/sched.c
index 2810e56..c306a4a 100644
--- a/kernel/sched.c
+++ b/kernel/sched.c
@@ -3336,7 +3336,7 @@ void account_user_time(struct task_struct *p, cputime_t cputime)
  * @p: the process that the cpu time gets accounted to
  * @cputime: the cpu time spent in virtual machine since the last update
  */
-void account_guest_time(struct task_struct *p, cputime_t cputime)
+static void account_guest_time(struct task_struct *p, cputime_t cputime)
 {
 	cputime64_t tmp;
 	struct cpu_usage_stat *cpustat = &kstat_this_cpu.cpustat;


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [2.6 patch] make kernel/sched.c:account_guest_time() static
  2007-10-24 16:25 [2.6 patch] make kernel/sched.c:account_guest_time() static Adrian Bunk
@ 2007-10-24 16:50 ` Ingo Molnar
  2007-10-24 17:00   ` Adrian Bunk
  0 siblings, 1 reply; 4+ messages in thread
From: Ingo Molnar @ 2007-10-24 16:50 UTC (permalink / raw)
  To: Adrian Bunk; +Cc: Laurent Vivier, Avi Kivity, linux-kernel


* Adrian Bunk <bunk@kernel.org> wrote:

> account_guest_time() can become static.

thanks, applied.

btw., how about adding your 'check static functions' scripts to the 
kernel, via a "make checkstatic" pass or something like that? That way 
people could check this for themselves and could avoid letting them into 
the kernel.

	Ingo

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [2.6 patch] make kernel/sched.c:account_guest_time() static
  2007-10-24 16:50 ` Ingo Molnar
@ 2007-10-24 17:00   ` Adrian Bunk
  2007-10-24 17:20     ` Ingo Molnar
  0 siblings, 1 reply; 4+ messages in thread
From: Adrian Bunk @ 2007-10-24 17:00 UTC (permalink / raw)
  To: Ingo Molnar; +Cc: Laurent Vivier, Avi Kivity, linux-kernel

On Wed, Oct 24, 2007 at 06:50:27PM +0200, Ingo Molnar wrote:
> 
> * Adrian Bunk <bunk@kernel.org> wrote:
> 
> > account_guest_time() can become static.
> 
> thanks, applied.
> 
> btw., how about adding your 'check static functions' scripts to the 
> kernel, via a "make checkstatic" pass or something like that? That way 
> people could check this for themselves and could avoid letting them into 
> the kernel.

It is not my script, it was written by Keith Owens.

It is already available as "make namespacecheck" since kernel 2.6.9.

And yes, it is already documented besides checkstack in
Documentation/SubmitChecklist point 10.

> 	Ingo

cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [2.6 patch] make kernel/sched.c:account_guest_time() static
  2007-10-24 17:00   ` Adrian Bunk
@ 2007-10-24 17:20     ` Ingo Molnar
  0 siblings, 0 replies; 4+ messages in thread
From: Ingo Molnar @ 2007-10-24 17:20 UTC (permalink / raw)
  To: Adrian Bunk; +Cc: Laurent Vivier, Avi Kivity, linux-kernel


* Adrian Bunk <bunk@kernel.org> wrote:

> > btw., how about adding your 'check static functions' scripts to the 
> > kernel, via a "make checkstatic" pass or something like that? That 
> > way people could check this for themselves and could avoid letting 
> > them into the kernel.
> 
> It is not my script, it was written by Keith Owens.
> 
> It is already available as "make namespacecheck" since kernel 2.6.9.

heh, *blush* :-) I'll start using it :)

	Ingo

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2007-10-24 17:21 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-10-24 16:25 [2.6 patch] make kernel/sched.c:account_guest_time() static Adrian Bunk
2007-10-24 16:50 ` Ingo Molnar
2007-10-24 17:00   ` Adrian Bunk
2007-10-24 17:20     ` Ingo Molnar

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox