mirror of
https://github.com/opnsense/src.git
synced 2026-03-26 12:43:12 -04:00
MFC 196745:
Don't attempt to bind the current thread to the CPU an IRQ is bound to when removing an interrupt handler from an IRQ during shutdown. During shutdown we are already bound to CPU 0 and this was triggering a panic. Approved by: re (kib)
This commit is contained in:
parent
3c7562c77e
commit
bae950f3b3
2 changed files with 24 additions and 18 deletions
|
|
@ -990,18 +990,21 @@ apic_free_vector(u_int apic_id, u_int vector, u_int irq)
|
|||
* we don't lose an interrupt delivery race.
|
||||
*/
|
||||
td = curthread;
|
||||
thread_lock(td);
|
||||
if (sched_is_bound(td))
|
||||
panic("apic_free_vector: Thread already bound.\n");
|
||||
sched_bind(td, apic_cpuid(apic_id));
|
||||
thread_unlock(td);
|
||||
if (!rebooting) {
|
||||
thread_lock(td);
|
||||
if (sched_is_bound(td))
|
||||
panic("apic_free_vector: Thread already bound.\n");
|
||||
sched_bind(td, apic_cpuid(apic_id));
|
||||
thread_unlock(td);
|
||||
}
|
||||
mtx_lock_spin(&icu_lock);
|
||||
lapics[apic_id].la_ioint_irqs[vector - APIC_IO_INTS] = -1;
|
||||
mtx_unlock_spin(&icu_lock);
|
||||
thread_lock(td);
|
||||
sched_unbind(td);
|
||||
thread_unlock(td);
|
||||
|
||||
if (!rebooting) {
|
||||
thread_lock(td);
|
||||
sched_unbind(td);
|
||||
thread_unlock(td);
|
||||
}
|
||||
}
|
||||
|
||||
/* Map an IDT vector (APIC) to an IRQ (interrupt source). */
|
||||
|
|
|
|||
|
|
@ -994,18 +994,21 @@ apic_free_vector(u_int apic_id, u_int vector, u_int irq)
|
|||
* we don't lose an interrupt delivery race.
|
||||
*/
|
||||
td = curthread;
|
||||
thread_lock(td);
|
||||
if (sched_is_bound(td))
|
||||
panic("apic_free_vector: Thread already bound.\n");
|
||||
sched_bind(td, apic_cpuid(apic_id));
|
||||
thread_unlock(td);
|
||||
if (!rebooting) {
|
||||
thread_lock(td);
|
||||
if (sched_is_bound(td))
|
||||
panic("apic_free_vector: Thread already bound.\n");
|
||||
sched_bind(td, apic_cpuid(apic_id));
|
||||
thread_unlock(td);
|
||||
}
|
||||
mtx_lock_spin(&icu_lock);
|
||||
lapics[apic_id].la_ioint_irqs[vector - APIC_IO_INTS] = -1;
|
||||
mtx_unlock_spin(&icu_lock);
|
||||
thread_lock(td);
|
||||
sched_unbind(td);
|
||||
thread_unlock(td);
|
||||
|
||||
if (!rebooting) {
|
||||
thread_lock(td);
|
||||
sched_unbind(td);
|
||||
thread_unlock(td);
|
||||
}
|
||||
}
|
||||
|
||||
/* Map an IDT vector (APIC) to an IRQ (interrupt source). */
|
||||
|
|
|
|||
Loading…
Reference in a new issue