Hi, On Wed, Jan 7, 2009 at 12:51 AM, Peter Stuge <[email protected]> wrote: > Hi, I found what seems to be a bug in the current yabel/biosemu.c. > You seem to have addressed it in a patch already, though.. > > > Pattrick Hueper wrote: > .. >> +u32 >> +biosemu(u8 *biosmem, u32 biosmem_size, struct device * dev) >> +{ > .. >> + // setup default Interrupt Vectors >> + // some expansion ROMs seem to check for these addresses.. >> + // each handler is only an IRET (0xCF) instruction >> + // ROM BIOS Int 10 Handler F000:F065 >> + my_wrl(0x10 * 4, 0xf000f065); >> + my_wrb(0x000ff065, 0xcf); > > Has this gotten some feedback? The fix is correct! >
This code is currently in svn, isnt it? It should be correct, all it does is let the INT10 int handler point to F000:F065 (the default location) and the code at that location is 0xCF (IRET). Patty -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
