Control: reassign -1 seabios 1.7.2-1 Control: tag -1 + upstream patch pending
13.05.2013 18:56, Edward J. Shornock wrote: > As a final test I did > > cd pc-bios > cp vgabios-vmware.bin vgabios-stdvga.bin vgabios-qxl.bin \ > vgabios-cirrus.bin q35-acpi-dsdt.aml acpi-dsdt.aml bios.bin \ > /usr/share/seabios/ > cp multiboot.bin linuxboot.bin kvmvapic.bin > /usr/share/seabios/optionrom/ Blah. The only really interesting file is bios.bin. The rest should not make any difference. The bug in question has been most likely fixed by this seabios commit: commit f64a472a481784231fbf8541825501df411b11d1 Author: Paolo Bonzini <[email protected]> Date: Thu Dec 13 08:42:02 2012 +0100 acpi: reintroduce LNKS Commit 4540409 (acpi: LNKS is not needed, 2012-08-07) removed LNKS because it basically worked by chance: _CRS returns something else then one of the possible resources from _PRS, _DIS would not really disable the interrupt, and there was no _SRS method. It just happened to work because all OSes have some kind of special-casing for SCI. Unfortunately, the code after the patch is also against the spec, and it breaks FreeBSD because it treats IRQ 9 polarity as active low without the Interrupt() entry. Actually, numeric _PRT entries are handled the same in Linux and FreeBSD (as active-low). However, under Linux it just happens to trigger another special casing of SCI which sets SCI up from its override entry in the MADT, ignoring the DSDT completely. This patch adds back the LNKS, but without using the PIIX register for LNKA in its methods. Tested-by: Luigi Rizzo <[email protected]> Signed-off-by: Paolo Bonzini <[email protected]> Reassigning and re-tagging :) Thanks, /mjt -- To UNSUBSCRIBE, email to [email protected] with a subject of "unsubscribe". Trouble? Contact [email protected]
