linux-kernel - Re: Galbraith patch

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Fri, 19 Nov 2010 19:05:03 -0500
From:	Gene Heskett <gene.heskett@...il.com>
To:	Mike Galbraith <efault@....de>
Cc:	Jordi Pujol <jordipujolp@...il.com>,
	LKML <linux-kernel@...r.kernel.org>
Subject: Re: Galbraith patch

On Friday, November 19, 2010, Mike Galbraith wrote:
>On Fri, 2010-11-19 at 10:05 -0500, Gene Heskett wrote:
>> 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.
>
>No idea what's going on with that.  Maybe you should wait until I get
>the /proc display/tweak interface cobbled together and post a checked
>over patch instead of hasty WIP snapshot.

I have come to that conclusion of too.  But because I am interested in an 
improvement in this 'interactivity' of the desktop, I was willing to give 
even the WIP a try.  If what I report helps you, then my testing effort is 
not wasted.  The more eyes and different hdwe theory. ;-)

Thanks Mike.

>> I wonder how long it will take to get work this into mainline?
>
>If it goes in at all, it won't be before .38.

I was hoping sooner.  Some of us could use the help this seems to promise.

>> Along the same line, how does this compare to CK's bfs patch?
>
>Like apple vs orange.
>
And they both contain too much sugar for me.  Dammit.

>	-Mike


-- 
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)
Deliberation, n.:
	The act of examining one's bread to determine which side it is
	buttered on.
		-- Ambrose Bierce, "The Devil's Dictionary"
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/