Re: suspicious code in ioapic locking

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

On Wed, Apr 23, 2008 at 01:20:40AM -0400, Stephen Degler wrote:

> The apic handling routines embedded in the macros in vector.S come from 
> i82093reg.h, and operate on struct pic (sys/x86/include/pic.h).  The 
> __cpu_simple_lock_t in the struct pic is now a byte, so the code is 
> wrong.  This doesn't seem to break anything, probably due to structure 
> padding.  The following patch corrects the ioapic code.  Another 
> approach would be to rearrange the register usage so that the simple 
> lock routines could be called directly, but this would be more 
> involved.  In the patch below, I used %bl because %rbx is blown away in 
> the macros anyway.

Hmm, my immediate reaction on reading it was that it was dangerous to use
%bl since the register conventions there are really kooky. I've checked 
the stubs and it seems to be OK.
 
> Let me know if you think I should commit this.

Yes please.

Thanks,
Andrew