[edk2-devel] RELEASE build error with QEMU_PV_VARS

[edk2-devel] RELEASE build error with QEMU_PV_VARS

[wanglimin]

[-- Attachment #1: Type: text/plain, Size: 914 bytes --]

Hello,
I met undefined reference to `MmioWrite32' error when building AARCH64 with -D QEMU_PV_VARS=TRUE on RELEASE  mode,

build -n 128 -p ArmVirtPkg/ArmVirtQemu.dsc -a AARCH64 -b RELEASE  -D QEMU_PV_VARS=TRUE  -t GCC5

…
OvmfPkg/VirtMmCommunicationDxe/QemuMmio.c:159: undefined reference to `MmioWrite32'
OvmfPkg/VirtMmCommunicationDxe/QemuMmio.c:161: undefined reference to `MmioRead32'
….

It is built successfully without the option QEMU_PV_VARS, or build with DEBUG mode.
Not sure if I am missing anything?

Best Regards
Limin Wang.


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#121351): https://edk2.groups.io/g/devel/message/121351
Mute This Topic: https://groups.io/mt/113204735/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



[-- Attachment #2: Type: text/html, Size: 5693 bytes --]

Re: [edk2-devel] RELEASE build error with QEMU_PV_VARS

[wanglimin]

[-- Attachment #1: Type: text/plain, Size: 1408 bytes --]

It can build successfully when I add IoLib, is this the correct fix ?

--- a/OvmfPkg/VirtMmCommunicationDxe/VirtMmCommunication.inf
+++ b/OvmfPkg/VirtMmCommunicationDxe/VirtMmCommunication.inf
@@ -39,6 +39,7 @@
   DebugLib
   DxeServicesTableLib
   HobLib
+  IoLib
   MemoryAllocationLib
   UefiDriverEntryPoint

Best Regards
LiminWang

发件人: wanglimin
发送时间: 2025年5月20日 10:10
收件人: devel@edk2.groups.io
主题: RELEASE build error with QEMU_PV_VARS

Hello,
I met undefined reference to `MmioWrite32' error when building AARCH64 with -D QEMU_PV_VARS=TRUE on RELEASE  mode,

build -n 128 -p ArmVirtPkg/ArmVirtQemu.dsc -a AARCH64 -b RELEASE  -D QEMU_PV_VARS=TRUE  -t GCC5

…
OvmfPkg/VirtMmCommunicationDxe/QemuMmio.c:159: undefined reference to `MmioWrite32'
OvmfPkg/VirtMmCommunicationDxe/QemuMmio.c:161: undefined reference to `MmioRead32'
….

It is built successfully without the option QEMU_PV_VARS, or build with DEBUG mode.
Not sure if I am missing anything?

Best Regards
Limin Wang.


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#121352): https://edk2.groups.io/g/devel/message/121352
Mute This Topic: https://groups.io/mt/113204735/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



[-- Attachment #2: Type: text/html, Size: 8484 bytes --]

Re: [edk2-devel] RELEASE build error with QEMU_PV_VARS

[Gerd Hoffmann via groups.io]

On Tue, May 20, 2025 at 02:24:12AM +0000, wanglimin wrote:
> It can build successfully when I add IoLib, is this the correct fix ?
> 
> --- a/OvmfPkg/VirtMmCommunicationDxe/VirtMmCommunication.inf
> +++ b/OvmfPkg/VirtMmCommunicationDxe/VirtMmCommunication.inf
> @@ -39,6 +39,7 @@
>    DebugLib
>    DxeServicesTableLib
>    HobLib
> +  IoLib
>    MemoryAllocationLib
>    UefiDriverEntryPoint

Yes.  Can you open a PR with this?

thanks,
  Gerd



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#121354): https://edk2.groups.io/g/devel/message/121354
Mute This Topic: https://groups.io/mt/113204735/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] RELEASE build error with QEMU_PV_VARS

[wanglimin]

Sure, I will open the PR.

-----邮件原件-----
发件人: Gerd Hoffmann <kraxel@redhat.com> 
发送时间: 2025年5月20日 14:37
收件人: devel@edk2.groups.io; wanglimin <wanglimin@xfusion.com>
主题: Re: [edk2-devel] RELEASE build error with QEMU_PV_VARS

On Tue, May 20, 2025 at 02:24:12AM +0000, wanglimin wrote:
> It can build successfully when I add IoLib, is this the correct fix ?
> 
> --- a/OvmfPkg/VirtMmCommunicationDxe/VirtMmCommunication.inf
> +++ b/OvmfPkg/VirtMmCommunicationDxe/VirtMmCommunication.inf
> @@ -39,6 +39,7 @@
>    DebugLib
>    DxeServicesTableLib
>    HobLib
> +  IoLib
>    MemoryAllocationLib
>    UefiDriverEntryPoint

Yes.  Can you open a PR with this?

thanks,
  Gerd



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#121355): https://edk2.groups.io/g/devel/message/121355
Mute This Topic: https://groups.io/mt/113204735/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] RELEASE build error with QEMU_PV_VARS

[wanglimin]

Hi Gerd, 
 The PR opened: https://github.com/tianocore/edk2/pull/11091
  Please have a review

Thanks
LiminWang

-----邮件原件-----
发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 wanglimin
发送时间: 2025年5月20日 15:00
收件人: Gerd Hoffmann <kraxel@redhat.com>; devel@edk2.groups.io
主题: Re: [edk2-devel] RELEASE build error with QEMU_PV_VARS

Sure, I will open the PR.

-----邮件原件-----
发件人: Gerd Hoffmann <kraxel@redhat.com> 
发送时间: 2025年5月20日 14:37
收件人: devel@edk2.groups.io; wanglimin <wanglimin@xfusion.com>
主题: Re: [edk2-devel] RELEASE build error with QEMU_PV_VARS

On Tue, May 20, 2025 at 02:24:12AM +0000, wanglimin wrote:
> It can build successfully when I add IoLib, is this the correct fix ?
> 
> --- a/OvmfPkg/VirtMmCommunicationDxe/VirtMmCommunication.inf
> +++ b/OvmfPkg/VirtMmCommunicationDxe/VirtMmCommunication.inf
> @@ -39,6 +39,7 @@
>    DebugLib
>    DxeServicesTableLib
>    HobLib
> +  IoLib
>    MemoryAllocationLib
>    UefiDriverEntryPoint

Yes.  Can you open a PR with this?

thanks,
  Gerd








-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#121357): https://edk2.groups.io/g/devel/message/121357
Mute This Topic: https://groups.io/mt/113204735/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-