[PATCH 5/8] replacing/fixing printk with pr_debug/pr_info inarch/i386 - nmi.c

From: Daniele Pizzoni
Date: Sun Oct 17 2004 - 11:45:30 EST


Replace custom macro Dprintk (defined in apic.h included via smp.h) with pr_debug from kernel.h.
Use pr_info when appropriate.
Fix consistency of printks.

Signed-off-by: Daniele Pizzoni <auouo@xxxxxx>

Index: linux-2.6.9-rc4/arch/i386/kernel/nmi.c
===================================================================
--- linux-2.6.9-rc4.orig/arch/i386/kernel/nmi.c 2004-10-17 16:57:00.404604432 +0200
+++ linux-2.6.9-rc4/arch/i386/kernel/nmi.c 2004-10-17 17:01:18.114426584 +0200
@@ -13,6 +13,7 @@
* Mikael Pettersson : PM converted to driver model. Disable/enable API.
*/

+#include <linux/kernel.h>
#include <linux/config.h>
#include <linux/mm.h>
#include <linux/irq.h>
@@ -107,7 +108,7 @@ int __init check_nmi_watchdog (void)
unsigned int prev_nmi_count[NR_CPUS];
int cpu;

- printk(KERN_INFO "testing NMI watchdog ... ");
+ pr_info("testing NMI watchdog ... ");

for (cpu = 0; cpu < NR_CPUS; cpu++)
prev_nmi_count[cpu] = irq_stat[cpu].__nmi_count;
@@ -120,13 +121,13 @@ int __init check_nmi_watchdog (void)
if (!cpu_online(cpu))
continue;
if (nmi_count(cpu) - prev_nmi_count[cpu] <= 5) {
- printk("CPU#%d: NMI appears to be stuck!\n", cpu);
+ printk(KERN_WARNING "CPU#%d: NMI appears to be stuck!\n", cpu);
nmi_active = 0;
lapic_nmi_owner &= ~LAPIC_NMI_WATCHDOG;
return -1;
}
}
- printk("OK.\n");
+ pr_info("OK.\n");

/* now that we know it works we can reduce NMI frequency to
something more reasonable; makes a difference in some configs */
@@ -332,7 +333,7 @@ static void setup_k7_watchdog(void)
| K7_NMI_EVENT;

wrmsr(MSR_K7_EVNTSEL0, evntsel, 0);
- Dprintk("setting K7_PERFCTR0 to %08lx\n", -(cpu_khz/nmi_hz*1000));
+ pr_debug("setting K7_PERFCTR0 to %08lx\n", -(cpu_khz/nmi_hz*1000));
wrmsr(MSR_K7_PERFCTR0, -(cpu_khz/nmi_hz*1000), -1);
apic_write(APIC_LVTPC, APIC_DM_NMI);
evntsel |= K7_EVNTSEL_ENABLE;
@@ -354,7 +355,7 @@ static void setup_p6_watchdog(void)
| P6_NMI_EVENT;

wrmsr(MSR_P6_EVNTSEL0, evntsel, 0);
- Dprintk("setting P6_PERFCTR0 to %08lx\n", -(cpu_khz/nmi_hz*1000));
+ pr_debug("setting P6_PERFCTR0 to %08lx\n", -(cpu_khz/nmi_hz*1000));
wrmsr(MSR_P6_PERFCTR0, -(cpu_khz/nmi_hz*1000), 0);
apic_write(APIC_LVTPC, APIC_DM_NMI);
evntsel |= P6_EVNTSEL0_ENABLE;
@@ -395,7 +396,7 @@ static int setup_p4_watchdog(void)

wrmsr(MSR_P4_CRU_ESCR0, P4_NMI_CRU_ESCR0, 0);
wrmsr(MSR_P4_IQ_CCCR0, P4_NMI_IQ_CCCR0 & ~P4_CCCR_ENABLE, 0);
- Dprintk("setting P4_IQ_COUNTER0 to 0x%08lx\n", -(cpu_khz/nmi_hz*1000));
+ pr_debug("setting P4_IQ_COUNTER0 to 0x%08lx\n", -(cpu_khz/nmi_hz*1000));
wrmsr(MSR_P4_IQ_COUNTER0, -(cpu_khz/nmi_hz*1000), -1);
apic_write(APIC_LVTPC, APIC_DM_NMI);
wrmsr(MSR_P4_IQ_CCCR0, nmi_p4_cccr_val, 0);


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/