lkml.org 
[lkml]   [2008]   [Sep]   [16]   [last100]   RSS Feed
Views: [more markup]  [less markup]  [headers]  [forward] 
 
Messages in this thread
/
DateTue, 16 Sep 2008 10:39:15 +0200
FromIngo Molnar <>
SubjectRe: [PATCH 2.6.27-rc5 resubmit] Fix itimer/many thread hang.
* Frank Mayhar <fmayhar@google.com> wrote:

> On Mon, 2008-09-15 at 08:41 +0200, Ingo Molnar wrote:
> > * Roland McGrath <roland@redhat.com> wrote:
> > 
> > > > please uninline these functions.
> > > 
> > > They are short and have only one caller.
> > 
> > then they should be defined in the .c file which uses them.
> 
> Well, they are actually used in sched.h, in the inline routines 
> account_group_user_time() and friends, which are in turn used by 
> routines in sched.c.  The point here is that the 
> thread_group_cputime_account_xxx() routines are defined differently 
> depending upon whether we're building for UP or SMP and the 
> account_group_xxx() routines use the appropriate version accordingly.
> 
> > really, sched.h is large enough already and has lots of unnecessary 
> > stuff in it.
> 
> Agreed, but Roland and I were trying to make life easier for folks who 
> have to maintain SMP and UP versions of this stuff, keeping it all in 
> one place rather than scattering it about.
> 
> If you insist, I'll un-inline the routines but this will mean moving 
> all of this (thread_group_cputime_account_xxx() and 
> account_group_xxx() routines) to sched.c.  Maybe that makes sense.  
> It's your call; I'll hold off on the changes (and resubmitting the 
> patch) while I wait for you to let me know what you prefer.  Thanks.

it's already in tip/sched/posix-cpu-timers, so please send a delta patch 
against tip/master:

  http://people.redhat.com/mingo/tip.git/README

i'd suggest to move all the scheduler-internal accounting functions into 
kernel/sched_stats.h. AFAICS there's no scheduler-external user of these 
APIs. The most widely used external APIs is thread_group_cputime_smp(), 
and that is out of line already. (except on UP where it's trivial)

	Ingo


\
 
 \ /
  Last update: 2008-09-16 10:43    [from the cache]
©2003-2008