Keith Mannthey <kmannth@us.ibm.com> wrote:
>
> This should be better. Thanks for the comments.
Remind me again what the patch actually does? It seems to be purely adding
debug checks?
Won't it just go BUG if someone boots the kernel and then tries to manually
set affinity?
Seems a bit racy too. setup_ioapic_dest() does:
pending_irq_balance_apicid[irq] = mask;
==> window here
set_ioapic_affinity(irq, mask);
ioapic_lock is not held, so there is a window where
pending_irq_balance_apicid[irq] can be set to some other value and
io_apic_write_affinity() will accidentally go BUG.
Is it not possible to fix set_ioapic_affinity() for real for clustered APIC
mode? What is involved in that?
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
This archive was generated by hypermail 2b29 : Wed Apr 30 2003 - 22:00:37 EST