Messages in this thread | | | Date | Fri, 7 Nov 2008 10:30:50 +0100 | From | Ingo Molnar <> | Subject | Re: [PATCH] sched: rewrite SCHED_CPUMASK_ALLOC |
| |
* Li Zefan <lizf@cn.fujitsu.com> wrote:
> Sorry, you are right - keep the SCHED_CPUMASK_DECLARE() macro please. > > > > But the other macros can become an inline function just fine, correct? > > > > Yes, but sched_cpumask_alloc() should be declared this way: > void sched_cpumask_alloc(struct **allmasks) > but not: > struct *allmasks sched_cpumask_alloc(void) > > Because the latter is not workable for <128 CPUs case. > > (patch is based on [PATCH] sched: fix memory leak in a failing path)
applied to tip/sched/core, thanks!
Ingo
| |