Skip to content

Commit

Permalink
[PATCH] x86-64: use safe_apic_wait_icr_idle in smpboot.c - x86_64
Browse files Browse the repository at this point in the history
inquire_remote_apic is used for APIC debugging, so use
safe_apic_wait_icr_idle  instead of apic_wait_icr_idle to avoid possible
lockups when APIC delivery fails.

Signed-off-by: Fernando Luis Vazquez Cao <fernando@oss.ntt.co.jp>
Signed-off-by: Andi Kleen <ak@suse.de>
  • Loading branch information
Fernando Luis VazquezCao authored and Andi Kleen committed May 2, 2007
1 parent 4312fa8 commit 3144c33
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions arch/x86_64/kernel/smpboot.c
Original file line number Diff line number Diff line change
Expand Up @@ -391,7 +391,8 @@ static void inquire_remote_apic(int apicid)
{
unsigned i, regs[] = { APIC_ID >> 4, APIC_LVR >> 4, APIC_SPIV >> 4 };
char *names[] = { "ID", "VERSION", "SPIV" };
int timeout, status;
int timeout;
unsigned int status;

printk(KERN_INFO "Inquiring remote APIC #%d...\n", apicid);

Expand All @@ -401,7 +402,9 @@ static void inquire_remote_apic(int apicid)
/*
* Wait for idle.
*/
apic_wait_icr_idle();
status = safe_apic_wait_icr_idle();
if (status)
printk("a previous APIC delivery may have failed\n");

apic_write(APIC_ICR2, SET_APIC_DEST_FIELD(apicid));
apic_write(APIC_ICR, APIC_DM_REMRD | regs[i]);
Expand Down

0 comments on commit 3144c33

Please sign in to comment.