lkml.org 
[lkml]   [2006]   [Feb]   [26]   [last100]   RSS Feed
Views: [more markup]  [less markup]  [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [patch 2.6.16-rc4-mm1] Task Throttling V14
FromMIke Galbraith <>
DateSun, 26 Feb 2006 14:19:35 +0100
On Sun, 2006-02-26 at 11:26 +0000, Daniel K. wrote:
> MIke Galbraith wrote:
> > On Fri, 2006-02-17 at 14:45 +0100, MIke Galbraith wrote: 
> > +/*
> > + * Masks for p->slice_info, formerly p->first_time_slice.
> > + * SLICE_FTS:   0x80000000  Task is in it's first ever timeslice.
> > + * SLICE_NEW:   0x40000000  Slice refreshed.
> > + * SLICE_SPA:   0x3FFF8000  Spare bits.
> > + * SLICE_LTS:   0x00007F80  Last time slice
> > + * SLICE_AVG:   0x0000007F  Task slice_avg stored as percentage.
> > + */
> > +#define SLICE_AVG_BITS    7
> > +#define SLICE_LTS_BITS   10
> > +#define SLICE_SPA_BITS   13
> > +#define SLICE_NEW_BITS    1
> > +#define SLICE_FTS_BITS    1> I count 8 and 15 bits in the documentation of LTS/SPA respectively, not 
> 10 and 13.
Dang, fixed the stupid bug, but forgot to wipe the evidence ;-)  Fixed.

> > >  	}
> > 
> >  	if (likely(sleep_time > 0)) {
> > +> > Extra line

Fixed.

> > > +	{> > +		.ctl_name	= KERN_SCHED_THROTTLE1,> > +		.procname	= "sched_g1",> > +		.data		= &sched_g1,> > +		.maxlen		= sizeof (int),> > +		.mode		= 0644,> > +		.proc_handler	= &proc_dointvec_minmax,> > +		.strategy	= &sysctl_intvec,> > +		.extra1		= &zero,> > +		.extra2		= &sched_g2_max,> > sched_g2_max is possibly badly named, as it is used in connection with 
> sched_g1 here.> > > +	},> > +	{> > +		.ctl_name	= KERN_SCHED_THROTTLE2,> > +		.procname	= "sched_g2",> > +		.data		= &sched_g2,> > +		.maxlen		= sizeof (int),> > +		.mode		= 0644,> > +		.proc_handler	= &proc_dointvec_minmax,> > +		.strategy	= &sysctl_intvec,> > +		.extra1		= &zero,> > +		.extra2		= &sched_g2_max,> > +	},

I suppose sched_grace_max would fit better.

Thanks for taking a look.

	-Mike

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2006-02-26 14:25    [from the cache]
©2003-2008