Deprecate useless bug

PATCH In review Zachary Amsden 1 participant 1 message Nov 10, 2005

Share & tools

Zachary Amsden

author

2005-11-10 00:47 UTC #0 lore raw

PATCH 10/10 Deprecate useless bug

Remove the "temporary debugging check" which has managed to live for quite some time, and is clearly unneeded. The mm can never be live at this point, so clearly checking the LDT in the mm->context is redundant as well.

Signed-off-by Zachary Amsden <[email protected]>

arch/i386/kernel/process.c +1 ~~-11~~

Index: linux-2.6.14/arch/i386/kernel/process.c
===================================================================
@@ -415,17 +415,7 @@
 
 void release_thread(struct task_struct *dead_task)
 {
-	if (dead_task->mm) {
-		// temporary debugging check
-		if (dead_task->mm->context.size) {
-			printk("WARNING: dead process %8s still has LDT? <%p/%d>\n",
-					dead_task->comm,
-					dead_task->mm->context.ldt,
-					dead_task->mm->context.size);
-			BUG();
-		}
-	}
-
+	BUG_ON(dead_task->mm);
 	release_vm86_irqs(dead_task);
 }