Re: CVS commit: src

[Andrew Doran <ad%netbsd.org@localhost>]

On Sat, Apr 25, 2009 at 09:42:30PM +0100, Mindaugas Rasiukevicius wrote:

> > Module Name:    src
> > Committed By:   ad
> > Date:           Sun Apr 19 14:11:38 UTC 2009
> > 
> > <...>
> >
> > x86:
> > 
> > - Make intr_establish, intr_disestablish safe to use when !cold.
> > 
> > - Distribute hardware interrupts among the CPUs, instead of directing
> >   everything to the boot CPU.
> 
> This is cool!

Just a shame it doesn't work properly yet. The balancing code is OK I think,
something in the guts of APIC/SPL handling is busted.
 
> From sys/arch/x86/x86/intr.c:
> 
> /*
>  * A simple round-robin allocator to assign interrupts to CPUs.
>  */
> static int __noinline
> intr_allocate_slot(struct pic *pic, int pin, int level,
>                  struct cpu_info **cip, int *index, int *idt_slot)
> {
> 
> From what I read, it is no longer round-robin, right?

Right, the comment is stale.