Messages in this thread | | | Date | Mon, 23 Apr 2012 11:55:45 +0200 | From | Juri Lelli <> | Subject | Re: [PATCH 03/16] sched: SCHED_DEADLINE data structures. |
| |
On 04/23/2012 11:49 AM, Peter Zijlstra wrote: > 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.
Sure. Since this is a small change, probably not strictly related to this patchset, may I wait to see the change in mainline?
Thanks,
- Juri
| |