[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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

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.


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



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.