Re: [edk2] OVMF Xen ACPI tables

Subject: Re: [edk2] OVMF Xen ACPI tables

From: Bei Guan <gbtju85@gmail.com>

To: Andrei Warkentin <andreiw@motorola.com>

Date: 2011-07-01 19:30:16



2011/7/1 Andrei Warkentin <andreiw@motorola.com>
Bei,

Looks alright. Of course, you had to modify the Universal/SmBiosDxe,
right? So it doesn't install more tables if it's already installed?
Where is that change?
The new patch just adds every Xen SMBIOS structure to the previous SMBIOS installed byUniversal/SmBiosDxe. If we need to add more Xen SMBIOS structure in the future, we just add a new methodInstallSmbiosType[type of structure]() to do this. It's very easy to maintain the code.

About the next step.
1) Modify the patch OvmfPkg/SmbiosPlatformDxe to fixthe bug about "dmidecode" inside UEFI OS I mentioned before. (There may be some error with adding Xen BIOS information structure to UEFI SMBIOS, which is done in methodInstallSmbiosType0())
2) If the patch OvmfPkg/AcpiPlatformDxe can also be applied, I will move on to the optimization tasks "implementation ParaVirtualization module in OVMF" quickly.

Before starting 2), I think we need to make Xen libxl supporting "hvmbios=[ovmf-ia32|ovmf-x64]" in HVM config file. Because xm may not run well in Xen-4.2-unstable. What about your opinion?

Meanwhile, we need to prepare for mid-term valuation. As mentioned in my proposal, the files we need to submit are
1) Binary release of OVMF for Xen,
2) Patches for EDK2 and Xen to support OVMF running on Xen,
3) A simple README file for how to use.
4)Hypervisor support driver in OVMF if I finish that before 10th, July. (This is optional before mid-term)

Is there anything I havemissed?


Thanks,
Bei Guan


A

On Thu, Jun 30, 2011 at 1:55 PM, Jordan Justen <jljusten@gmail.com> wrote:
> On Thu, Jun 30, 2011 at 10:47, Bei Guan <gbtju85@gmail.com> wrote:
>> I have updated the Xen SMBIOS publish patch just now.
>> https://github.com/GavinGuan/gavinguan/blob/master/xen/ovmf-support/ovmf_edk2_OvmfPkg_XenSmbiosDxe.patch
>> https://github.com/GavinGuan/gavinguan/blob/master/xen/ovmf-support/ovmf_edk2_OvmfPkg_configuration.patch
>
> It looks pretty good.
>
> Can you rename OvmfPkg/XenBiosDxe to OvmfPkg/SmbiosPlatformDxe?
>
> I would rather add a OVMF SMBIOS driver with Xen support than a Xen
> SMBIOS driver.
>
> Thanks,
>
> -Jordan
>