|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [PATCH v2 1/2] x86/APIC: include full string with error_interrupt() error messages
Rather than adding ", " with each printf(), simply include them in the
string initially.
Signed-off-by: Elliott Mitchell <ehem+xen@xxxxxxx>
---
xen/arch/x86/apic.c | 18 +++++++++---------
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/xen/arch/x86/apic.c b/xen/arch/x86/apic.c
index f71474d47d..8cfb8cd71c 100644
--- a/xen/arch/x86/apic.c
+++ b/xen/arch/x86/apic.c
@@ -1401,14 +1401,14 @@ static void cf_check spurious_interrupt(struct
cpu_user_regs *regs)
static void cf_check error_interrupt(struct cpu_user_regs *regs)
{
static const char *const esr_fields[] = {
- "Send CS error",
- "Receive CS error",
- "Send accept error",
- "Receive accept error",
- "Redirectable IPI",
- "Send illegal vector",
- "Received illegal vector",
- "Illegal register address",
+ ", Send CS error",
+ ", Receive CS error",
+ ", Send accept error",
+ ", Receive accept error",
+ ", Redirectable IPI",
+ ", Send illegal vector",
+ ", Received illegal vector",
+ ", Illegal register address",
};
unsigned int v, v1;
int i;
@@ -1423,7 +1423,7 @@ static void cf_check error_interrupt(struct cpu_user_regs
*regs)
smp_processor_id(), v , v1);
for ( i = 7; i >= 0; --i )
if ( v1 & (1 << i) )
- printk(", %s", esr_fields[i]);
+ printk("%s", esr_fields[i]);
printk("\n");
}
--
(\___(\___(\______ --=> 8-) EHM <=-- ______/)___/)___/)
\BS ( | ehem+sigmsg@xxxxxxxxxxxx PGP 87145445 | ) /
\_CS\ | _____ -O #include <stddisclaimer.h> O- _____ | / _/
8A19\___\_|_/58D2 7E3D DDF4 7BA6 <-PGP-> 41D1 B375 37D0 8714\_|_/___/5445
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |