On Fri, 2013-01-11 at 00:18 -0800, Jordan Justen wrote: > > Looking at the first patch > "OvmfPkg: LoadLinuxLib: Fix kernel entry for 64-bit OVMF" > > It only works for me if I drop the change to LinuxGdt.c. It is > required in your testing?
Hm. No, it's not required. Without KVM, qemu works either with or without it. And if I enable KVM, I get the same results as you. Further investigation suggests that the GDT change was indeed wrong. It's actually setting the long-mode flag when I wanted compatibility mode. I'd misread the code in EdkCompatibilityPkg/Compatibility/BootScriptSaveOnS3SaveStateThunk/ I'm not sure why it works in non-KVM qemu, but that's probably a qemu bug (or legitimate undefined behaviour). I'll drop that part of the patch; thanks. -- dwmw2
smime.p7s
Description: S/MIME cryptographic signature
------------------------------------------------------------------------------ Master HTML5, CSS3, ASP.NET, MVC, AJAX, Knockout.js, Web API and much more. Get web development skills now with LearnDevNow - 350+ hours of step-by-step video tutorials by Microsoft MVPs and experts. SALE $99.99 this month only -- learn more at: http://p.sf.net/sfu/learnmore_122812
_______________________________________________ edk2-devel mailing list edk2-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/edk2-devel
