Re: [patch/backport] CFS scheduler, -v24, for v2.6.24-rc3, v2.6.23.8,v2.6.22.13, v2.6.21.7

[David <david@livux.com>]

El Martes, 20 de Noviembre de 2007, Ingo Molnar escribió:
> * David <david@livux.com> wrote:
> > El Lunes, 19 de Noviembre de 2007, Ingo Molnar escribió:
> > > * David <david@livux.com> wrote:
> > > > I have removed all other patches, and applied only cfs v24 above
> > > > 2.6.23.8, and the compiler ran into (with CONFIG_FAIR_GROUP_SCHED
> > > > enabled):
> > >
> > > does the patch below help?
> > >
> > > 	Ingo
> >
> > Yes, now sched.c compile without errors, but linking fails at:
> >
> >
> >   LD      init/built-in.o
> >   LD      .tmp_vmlinux1
> > kernel/built-in.o:(.data+0x4a8): undefined reference to
> > `sysctl_sched_min_bal_int_shares' kernel/built-in.o:(.data+0x4d4):
> > undefined reference to `sysctl_sched_max_bal_int_shares' make: ***
> > [.tmp_vmlinux1] Error 1
>
> does the patch below do the trick?
>
> 	Ingo
>
> Index: linux/kernel/sysctl.c
> ===================================================================
> --- linux.orig/kernel/sysctl.c
> +++ linux/kernel/sysctl.c
> @@ -309,7 +309,7 @@ static struct ctl_table kern_table[] = {
>  		.mode		= 644,
>  		.proc_handler	= &proc_dointvec,
>  	},
> -#ifdef CONFIG_FAIR_GROUP_SCHED
> +#if defined(CONFIG_FAIR_GROUP_SCHED) && defined(CONFIG_SMP)
>  	{
>  		.ctl_name	= CTL_UNNUMBERED,
>  		.procname	= "sched_min_bal_int_shares",

Yes, the kernel compile correctly.

Thanks Ingo



-- 
David Rodríguez García
Director Técnico
LiVux I+D S.L.