[lkml]   [2007]   [Mar]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
Patch in this message
SubjectRe: 2.6.21-rc5-mm1
On Thu, 29 Mar 2007 16:32:19 -0400
"Ryan Hope" <> wrote:

> I was not able to boot 2.6.21-rc3-mm* so I can not tell if this problem
> exists in that series but with 2.6.21-rc4-mm* and 2.6.21-rc5-mm* I can not
> get into Xorg. I disabled frame buffer support but when X loads normally
> the display goes blank for a moment then the greeter gets started but with
> the mentioned kernels the screen just stays blank. Everyting worked fine in
> 2.6.21-rc2-mm* no I am not sure where the problem is. I am using the fglrx
> display driver for X if that help anyone.

hm, don't know, sorry.

Are you able to get control back so you can see if there's anything
interesting in the kernel logs? Maybe ssh in from another machine?

It's conceivable that reverting fix-sysfs-reclaim-crash.patch will fix it,
or applying the below.

If that doesn't help, or of you don't have the time to try it, please be
sure to test the next -mm kernels, let us know if the bug is still there?

It could of course be that the memory management changes in -mm broke the
fglrx driver.

Begin forwarded message:

Date: Wed, 28 Mar 2007 23:38:16 -0600
From: (Eric W. Biederman)
To: Linus Torvalds <>
Cc: Andrew Morton <>, Sid Boyce <>, Adrian Bunk <>, <>
Subject: [PATCH] pid: Properly detect orphaned process groups in exit_notify

In commit 0475ac0845f9295bc5f69af45f58dff2c104c8d1 when converting
the converting the orphaned process group handling to use struct pid
I made a small mistake. I accidentally replaced an == with a !=.

Besides just being a dumb thing to do apparently this has a bad side
effect. The improper orphaned process group detection causes kwin to
die after a suspend/resume cycle.

I'm amazed this patch has been around as long as it has without anyone
else noticing something funny going on.

And the following people deserve credit for spotting and helping
to reproduce this.

Thanks to: Sid Boyce <>
Thanks to: "Michael Wu"

Signed-off-by: "Eric W. Biederman" <>

diff --git a/kernel/exit.c b/kernel/exit.c
index f132349..b55ed4c 100644
--- a/kernel/exit.c
+++ b/kernel/exit.c
@@ -790,7 +790,7 @@ static void exit_notify(struct task_struct *tsk)

pgrp = task_pgrp(tsk);
if ((task_pgrp(t) != pgrp) &&
- (task_session(t) != task_session(tsk)) &&
+ (task_session(t) == task_session(tsk)) &&
will_become_orphaned_pgrp(pgrp, tsk) &&
has_stopped_jobs(pgrp)) {
__kill_pgrp_info(SIGHUP, SEND_SIG_PRIV, pgrp);
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at
 \ /
  Last update: 2007-03-29 23:37    [W:0.143 / U:1.228 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site