BUG_ON() Conversion in mm/mmap.c
Subsystems (3)
PATCH 4/7 BUG_ON() Conversion in mm/mmap.c
hi,
this changes if() BUG(); constructs to BUG_ON() which is cleaner, contains unlikely() and can better optimized away.
Signed-off-by Eric Sesterhenn <[email protected]>
mm/mmap.c renamed from mm/mmap.c.orig +3 -6
@@ -294,8 +294,7 @@ void validate_mm(struct mm_struct *mm) i =browse_rb (&mm->mm_rb);if (i != mm->map_count)printk ("map_count %d rb %d\n" , mm->map_count, i), bug =1 ; -if (bug) - BUG(); + BUG_ON(bug); }#else #define validate_mm (mm) do { } while (0)
@@ -432,8 +431,7 @@ __insert_vm_struct(struct mm_struct * mmstruct rb_node ** rb_link, * rb_parent; __vma =find_vma_prepare (mm, vma->vm_start,&prev, &rb_link, &rb_parent); -if (__vma && __vma->vm_start < vma->vm_end) -BUG (); +BUG_ON (__vma && __vma->vm_start < vma->vm_end);__vma_link (mm, vma, prev, rb_link, rb_parent); mm->map_count++; }
@@ -813,8 +811,7 @@ try_prev:* (e.g. stash info in next's anon_vma_node when assigning * an anon_vma, or when trying vma_merge). Another time. */ -if (find_vma_prev (vma->vm_mm, vma->vm_start, &near) != vma) -BUG (); +BUG_ON (find_vma_prev (vma->vm_mm, vma->vm_start, &near) != vma);if (!near) goto none;
_______________________________________________
Kernel-janitors mailing list
[email protected]
https://lists.osdl.org/mailman/listinfo/kernel-janitors