>Rechberger, Markus wrote: >> >> This patch is a sync with the latest apic code in the qemu repository, >> this fixes a system crash after linux tries to set up apic after the >> first reboot. >> >> >> > >I'm worried that a partial sync can introduce bugs. Did you test this >with a Windows guest as well? (we can now use the ACPI HAL on Windows, >which (ab)uses the APIC). >
I tested it on windows with APIC and it works. Also the code shifts looks logical. (One is an even simple run over bug). Why did you change the version_id? You also did not register id == 2. Will it survive save/restore? Because the apic timer is not restarted on id < 2. > >-- >error compiling committee.c: too many arguments to function > > >----------------------------------------------------------------------- -- >Take Surveys. Earn Cash. Influence the Future of IT >Join SourceForge.net's Techsay panel and you'll get the chance to share >your >opinions on IT & business topics through brief surveys - and earn cash >http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVD EV >_______________________________________________ >kvm-devel mailing list >kvm-devel@lists.sourceforge.net >https://lists.sourceforge.net/lists/listinfo/kvm-devel ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ kvm-devel mailing list kvm-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/kvm-devel
