On Wed, Jan 9, 2013 at 3:16 PM, David Woodhouse <dw...@infradead.org> wrote:
> On Wed, 2013-01-09 at 13:38 -0800, Jordan Justen wrote:
>> On Wed, Jan 9, 2013 at 1:18 PM, David Woodhouse <dw...@infradead.org>
>> wrote:
>> > On Wed, 2013-01-09 at 13:11 -0800, Jordan Justen wrote:
>> >> > +  // Clear all but the setup_header
>> >> > +  SetMem (KernelSetup, 0x1f1, 0);
>>
>> What about OFFSET_OF rather than 0x1f1?
>
> It's actually defined as 0x1f1, and the struct is built to match. So I
> think 0x1f1 is probably the better option.
>
>> Can you use ZeroMem for these?
>
> Done. Git tree with four patches (including the first one updated to use
> lea/push as discussed) at
>   http://git.infradead.org/users/dwmw2/edk2.git
>    git://git.infradead.org/users/dwmw2/edk2.git

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?

-Jordan

------------------------------------------------------------------------------
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

Reply via email to