i386: clean up oops/bug reports

Typically the oops first lines look like this:

BUG: unable to handle kernel NULL pointer dereference at virtual address 00000000
 printing eip:
c049dfbd
*pde = 00000000
Oops: 0002 [#1]
PREEMPT SMP
...

Such output is gained with some ugly if (!nl) printk("\n"); code and
besides being a waste of lines, this is also annoying to read. The
following output looks better (and it is how it looks on x86_64):

BUG: unable to handle kernel NULL pointer dereference at virtual address 00000000
printing eip: c049dfbd *pde = 00000000
Oops: 0002 [#1] PREEMPT SMP
...

[ tglx: arch/x86 adaptation ]

Signed-off-by: Pavel Emelyanov <xemul@openvz.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Andi Kleen <ak@suse.de>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>

1 files changed, 7 insertions, 6 deletions

diff --git a/arch/x86/mm/fault_32.c b/arch/x86/mm/fault_32.c
index c686ae20fd6b..d84314ce22bb 100644
--- a/arch/x86/mm/fault_32.c
+++ b/arch/x86/mm/fault_32.c
@@ -539,23 +539,22 @@ no_context:
                         printk(KERN_ALERT "BUG: unable to handle kernel paging"
                                         " request");
                 printk(" at virtual address %08lx\n",address);
-                printk(KERN_ALERT " printing eip:\n");
-                printk("%08lx\n", regs->eip);
+                printk(KERN_ALERT "printing eip: %08lx ", regs->eip);
 
                 page = read_cr3();
                 page = ((__typeof__(page) *) __va(page))[address >> PGDIR_SHIFT];
 #ifdef CONFIG_X86_PAE
-                printk(KERN_ALERT "*pdpt = %016Lx\n", page);
+                printk("*pdpt = %016Lx ", page);
                 if ((page >> PAGE_SHIFT) < max_low_pfn
                     && page & _PAGE_PRESENT) {
                         page &= PAGE_MASK;
                         page = ((__typeof__(page) *) __va(page))[(address >> PMD_SHIFT)
                                                                  & (PTRS_PER_PMD - 1)];
-                        printk(KERN_ALERT "*pde = %016Lx\n", page);
+                        printk(KERN_ALERT "*pde = %016Lx ", page);
                         page &= ~_PAGE_NX;
                 }
 #else
-                printk(KERN_ALERT "*pde = %08lx\n", page);
+                printk("*pde = %08lx ", page);
 #endif
 
                 /*
@@ -569,8 +568,10 @@ no_context:
                         page &= PAGE_MASK;
                         page = ((__typeof__(page) *) __va(page))[(address >> PAGE_SHIFT)
                                                                  & (PTRS_PER_PTE - 1)];
-                        printk(KERN_ALERT "*pte = %0*Lx\n", sizeof(page)*2, (u64)page);
+                        printk("*pte = %0*Lx ", sizeof(page)*2, (u64)page);
                 }
+
+                printk("\n");
         }
 
         tsk->thread.cr2 = address;