On 03/05/20 22:26, Ard Biesheuvel wrote:
> Commit 859b55443a4253ba ("OvmfPkg/PlatformBootManagerLib: switch to
> QemuLoadImageLib") replaced a dependency on LoadLinuxLib with one on
> QemuLoadImageLib in the PlatformBootManagerLib implementation that is
> shared between all OVMF builds, without taking into account that even
> the Xen targeted builds incorporate this code, which is only used to
> load kernels passed via the QEMU command line.
>
> Since this is dead code on Xen, we can satisfy the dependency using
> the generic version of QemuLoadImageLib, which does not rely on
> LoadLinuxLib, which we can therefore drop from OvmfXen.dsc.
>
> Fixes: 859b55443a4253bad8bb618d04a51b2ded67f24b
> Signed-off-by: Ard Biesheuvel <[email protected]>
> ---
> OvmfPkg/OvmfXen.dsc | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/OvmfPkg/OvmfXen.dsc b/OvmfPkg/OvmfXen.dsc
> index d476694b18d5..85fe39f7896c 100644
> --- a/OvmfPkg/OvmfXen.dsc
> +++ b/OvmfPkg/OvmfXen.dsc
> @@ -151,7 +151,7 @@ [LibraryClasses]
> UefiUsbLib|MdePkg/Library/UefiUsbLib/UefiUsbLib.inf
>
> SerializeVariablesLib|OvmfPkg/Library/SerializeVariablesLib/SerializeVariablesLib.inf
> QemuFwCfgLib|OvmfPkg/Library/QemuFwCfgLib/QemuFwCfgDxeLib.inf
> - LoadLinuxLib|OvmfPkg/Library/LoadLinuxLib/LoadLinuxLib.inf
> +
> QemuLoadImageLib|OvmfPkg/Library/GenericQemuLoadImageLib/GenericQemuLoadImageLib.inf
>
> MemEncryptSevLib|OvmfPkg/Library/BaseMemEncryptSevLib/BaseMemEncryptSevLib.inf
> LockBoxLib|OvmfPkg/Library/LockBoxLib/LockBoxBaseLib.inf
>
> CustomizedDisplayLib|MdeModulePkg/Library/CustomizedDisplayLib/CustomizedDisplayLib.inf
>
Reviewed-by: Laszlo Ersek <[email protected]>
Thanks for the quick fix!
Laszlo
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#55561): https://edk2.groups.io/g/devel/message/55561
Mute This Topic: https://groups.io/mt/71759233/21656
Group Owner: [email protected]
Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-
