lkml.org 
[lkml]   [2002]   [Mar]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Subject[PATCH] 2.4: BUG_ON (2/2)
From
Date
Marcelo,

This patch, which requires the previous BUG_ON part 1 patch, changes a
few uses of BUG -> BUG_ON in fast paths in the kernel.

Patch is against 2.4.19-pre4. Marcelo, please apply.

Robert Love

diff -urN linux-2.4.19-pre4/arch/i386/kernel/smp.c linux/arch/i386/kernel/smp.c
--- linux-2.4.19-pre4/arch/i386/kernel/smp.c Wed Mar 20 17:44:53 2002
+++ linux/arch/i386/kernel/smp.c Wed Mar 20 18:01:09 2002
@@ -301,8 +301,7 @@
*/
static void inline leave_mm (unsigned long cpu)
{
- if (cpu_tlbstate[cpu].state == TLBSTATE_OK)
- BUG();
+ BUG_ON(cpu_tlbstate[cpu].state == TLBSTATE_OK);
clear_bit(cpu, &cpu_tlbstate[cpu].active_mm->cpu_vm_mask);
}

diff -urN linux-2.4.19-pre4/kernel/exit.c linux/kernel/exit.c
--- linux-2.4.19-pre4/kernel/exit.c Wed Mar 20 17:43:58 2002
+++ linux/kernel/exit.c Wed Mar 20 18:02:26 2002
@@ -316,7 +316,7 @@
mm_release();
if (mm) {
atomic_inc(&mm->mm_count);
- if (mm != tsk->active_mm) BUG();
+ BUG_ON(mm != tsk->active_mm);
/* more a memory barrier than a real lock */
task_lock(tsk);
tsk->mm = NULL;
diff -urN linux-2.4.19-pre4/kernel/fork.c linux/kernel/fork.c
--- linux-2.4.19-pre4/kernel/fork.c Wed Mar 20 17:43:58 2002
+++ linux/kernel/fork.c Wed Mar 20 18:02:44 2002
@@ -251,7 +251,7 @@
*/
inline void __mmdrop(struct mm_struct *mm)
{
- if (mm == &init_mm) BUG();
+ BUG_ON(mm == &init_mm);
pgd_free(mm->pgd);
destroy_context(mm);
free_mm(mm);
diff -urN linux-2.4.19-pre4/kernel/sched.c linux/kernel/sched.c
--- linux-2.4.19-pre4/kernel/sched.c Wed Mar 20 17:43:58 2002
+++ linux/kernel/sched.c Wed Mar 20 18:03:19 2002
@@ -556,7 +556,7 @@

spin_lock_prefetch(&runqueue_lock);

- if (!current->active_mm) BUG();
+ BUG_ON(!current->active_mm);
need_resched_back:
prev = current;
this_cpu = prev->processor;
@@ -675,12 +675,12 @@
struct mm_struct *mm = next->mm;
struct mm_struct *oldmm = prev->active_mm;
if (!mm) {
- if (next->active_mm) BUG();
+ BUG_ON(next->active_mm);
next->active_mm = oldmm;
atomic_inc(&oldmm->mm_count);
enter_lazy_tlb(oldmm, next, this_cpu);
} else {
- if (next->active_mm != mm) BUG();
+ BUG_ON(next->active_mm != mm);
switch_mm(oldmm, mm, next, this_cpu);
}

diff -urN linux-2.4.19-pre4/mm/slab.c linux/mm/slab.c
--- linux-2.4.19-pre4/mm/slab.c Wed Mar 20 17:43:58 2002
+++ linux/mm/slab.c Wed Mar 20 18:05:13 2002
@@ -666,8 +666,7 @@
* Always checks flags, a caller might be expecting debug
* support which isn't available.
*/
- if (flags & ~CREATE_MASK)
- BUG();
+ BUG_ON(flags & ~CREATE_MASK);

/* Get cache's description obj. */
cachep = (kmem_cache_t *) kmem_cache_alloc(&cache_cache, SLAB_KERNEL);
diff -urN linux-2.4.19-pre4/mm/vmscan.c linux/mm/vmscan.c
--- linux-2.4.19-pre4/mm/vmscan.c Wed Mar 20 17:43:58 2002
+++ linux/mm/vmscan.c Wed Mar 20 18:05:55 2002
@@ -234,8 +234,7 @@
pgdir = pgd_offset(mm, address);

end = vma->vm_end;
- if (address >= end)
- BUG();
+ BUG_ON(address >= end);
do {
count = swap_out_pgd(mm, vma, pgdir, address, end, count, classzone);
if (!count)
@@ -354,10 +353,8 @@

page = list_entry(entry, struct page, lru);

- if (unlikely(!PageLRU(page)))
- BUG();
- if (unlikely(PageActive(page)))
- BUG();
+ BUG_ON(!PageLRU(page));
+ BUG_ON(PageActive(page));

list_del(entry);
list_add(entry, &inactive_list);
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 13:25    [W:0.042 / U:1.352 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site