2014-02-24 05:42:52

by majianpeng

[permalink] [raw]
Subject: [TRIVIAL][PATCH] x86/ioapic: Using printk in func print_entries avoiding print incomplete infos

Commit(afcc8a40a090f): Introduce x86_io_apic_ops.print_entries for
debugging. Using kernel parameter apci=debug/verbose, kernel print
the incomplete infomation if we don't enable CONFIG_DYNAMIC_DEBUG.
So using printk to replace.

Signed-off-by: Jianpeng Ma <[email protected]>
---
arch/x86/kernel/apic/io_apic.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/arch/x86/kernel/apic/io_apic.c b/arch/x86/kernel/apic/io_apic.c
index 6ad4658..26996fc 100644
--- a/arch/x86/kernel/apic/io_apic.c
+++ b/arch/x86/kernel/apic/io_apic.c
@@ -1480,14 +1480,14 @@ void native_io_apic_print_entries(unsigned int apic, unsigned int nr_entries)
{
int i;

- pr_debug(" NR Dst Mask Trig IRR Pol Stat Dmod Deli Vect:\n");
+ printk(KERN_DEBUG" NR Dst Mask Trig IRR Pol Stat Dmod Deli Vect:\n");

for (i = 0; i <= nr_entries; i++) {
struct IO_APIC_route_entry entry;

entry = ioapic_read_entry(apic, i);

- pr_debug(" %02x %02X ", i, entry.dest);
+ printk(KERN_DEBUG" %02x %02X ", i, entry.dest);
pr_cont("%1d %1d %1d %1d %1d "
"%1d %1d %02X\n",
entry.mask,
@@ -1506,7 +1506,7 @@ void intel_ir_io_apic_print_entries(unsigned int apic,
{
int i;

- pr_debug(" NR Indx Fmt Mask Trig IRR Pol Stat Indx2 Zero Vect:\n");
+ printk(KERN_DEBUG" NR Indx Fmt Mask Trig IRR Pol Stat Indx2 Zero Vect:\n");

for (i = 0; i <= nr_entries; i++) {
struct IR_IO_APIC_route_entry *ir_entry;
@@ -1516,7 +1516,7 @@ void intel_ir_io_apic_print_entries(unsigned int apic,

ir_entry = (struct IR_IO_APIC_route_entry *)&entry;

- pr_debug(" %02x %04X ", i, ir_entry->index);
+ printk(KERN_DEBUG " %02x %04X ", i, ir_entry->index);
pr_cont("%1d %1d %1d %1d %1d "
"%1d %1d %X %02X\n",
ir_entry->format,
--
1.7.10.4
????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????{ay?ʇڙ?,j??f???h?????????z_??(?階?ݢj"???m??????G????????????&???~???iO???z??v?^?m???? ????????I?