Re: Galbraith patch

[Gene Heskett]

On Friday, November 19, 2010, Jordi Pujol wrote:
>Hello,
>
>it requires a little modification,
>
>CONFIG_SCHED_DEBUG should not be a condition to create autogroup in
>structure task_group
>
>remove
>    && defined(CONFIG_SCHED_DEBUG)
>in the following conditional:
>
>+++ linux-2.6.36/kernel/sched.c
>@@ -78,6 +78,7 @@
>
> #include "sched_cpupri.h"
> #include "workqueue_sched.h"
>+#include "sched_autogroup.h"
>
> #define CREATE_TRACE_POINTS
> #include <trace/events/sched.h>
>@@ -268,6 +269,10 @@ struct task_group {
> 	struct task_group *parent;
> 	struct list_head siblings;
> 	struct list_head children;
>+
>+#if (defined(CONFIG_SCHED_AUTOGROUP) && defined(CONFIG_SCHED_DEBUG))
>+	struct autogroup *autogroup;
>+#endif
> };
>
>Kind,
>
>Jordi Pujol

Thanks Jordi, but my last attempted build, using only the last 2 patches 
that Mike posted on a fresh unpack of 2.6.36, couldn't get past stage 2 of 
the modules build, it kept squawking about all the IR control stuff that is 
not even configured to be built.

I wonder how long it will take to get work this into mainline?

Along the same line, how does this compare to CK's bfs patch?

>Live never ending Tale
>GNU/Linux Live forever!
>http://livenet.selfip.com


-- 
Cheers, Gene
"There are four boxes to be used in defense of liberty:
 soap, ballot, jury, and ammo. Please use in that order."
-Ed Howdershelt (Author)
"What's the use of a good quotation if you can't change it?"
		-- The Doctor
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/