Re: [Xen-devel] [edk2] [PATCH RFC 0/7] OvmfPkg: make OVMF fully working with Xen

[Wei Liu <liuw@xxxxxxxxx>]

On Sat, Nov 16, 2013 at 3:31 AM, Jordan Justen <jljusten@xxxxxxxxx> wrote:
> When I try to boot this on qemu or kvm, it asserts because the video
> framebuffer address is NULL.
>

That's expected because of the hack. ;-)

> On Fri, Nov 15, 2013 at 7:58 AM, Wei Liu <wei.liu2@xxxxxxxxxx> wrote:
>> This is a quick hack I pulled together last night to make OVMF work with Xen.
>> I'm pretty sure there's many things that can be improved. ;-)
>>
>> Any advice is welcomed. I'm particular interested in comments such as how to
>> organize code, what to refactor, how to achieve wider benifit for other use
>> cases.
>>
>> Wei.
>>
>> Wei Liu (7):
>>   HACK: Use DeutPkg/PciRootBridgeNoEnumeration and
>>     PciBusNoEnumerationDxe
>
> These modules would need to move to PcAtChipsetPkg in the non-hack version.
>
> Actually, it would be even better if the generic version of these
> modules could be tweaked with a PCD to not do enumeration.
>

OK. I will look into it.

Just to make clear, you want to keep PcAtChipsetPkg (not moving
DuetPkg code in), and to make it able to skip enumeration with a PCD.

>>   OvmfPkg: introduce E820.h
>
> Can you copy the E820 structs from IntelFrameworkModulePkg/Csm/
> LegacyBiosDxe/LegacyBiosInterface.h instead?
>

OK, no problem.

Wei

>>   OvmfPkg: define EFI_XEN_OVMF_INFO and extend XenInfo
>>   OvmfPkg: extract OVMF info passed by Xen hvmloader
>>   OvmfPkg: detect Xen earlier
>>   OvmfPkg: introduce PublishPeiMemory
>>   OvmfPkg: introduce XenMemMapInitialization
>
> A quick glance makes me think the memmap stuff could tweaked, but I'll
> wait until it seems like this actually could work before I look at
> this too close. :)
>
> -Jordan
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxx
> http://lists.xen.org/xen-devel

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel