Messages in this thread | | | Subject | Re: [PATCH 03/16] sched: SCHED_DEADLINE data structures. | From | Peter Zijlstra <> | Date | Mon, 23 Apr 2012 11:49:24 +0200 |
| |
On Mon, 2012-04-23 at 11:47 +0200, Juri Lelli wrote: > On 04/23/2012 11:08 AM, Peter Zijlstra wrote: > > On Fri, 2012-04-06 at 09:14 +0200, Juri Lelli wrote: > >> +struct sched_dl_entity { > >> + struct rb_node rb_node; > >> + int nr_cpus_allowed; > > > > I think it would be all-round best to move > > sched_rt_entity::nr_cpus_allowed out next to cpus_allowed. > > You mean unify them: a single nr_cpus_allowed after > task_struct::cpus_allowed, right?
Yes, no point in keeping that one value twice and in fact there's a usage of p->rt.nr_cpus_allowed in sched/fair.c so its past time its moved out of that rt specific thing.
| |