Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

OvmfPkg/Bhyve: build platform info HOB #4676

Closed

Conversation

ckoehne
Copy link
Contributor

@ckoehne ckoehne commented Jul 24, 2023

Copy the function BuildPlatformInfoHob() from OvmfPkg/PlatformPei.

QemuFwCfgLib expect this HOB to be present, or fails to do anything. InternalQemuFwCfgIsAvailable() from QemuFwCfgPeiLib module will not check if the HOB is actually present for example and try to use a NULL pointer.

Fixes: cda98df ("OvmfPkg/QemuFwCfgLib: remove mQemuFwCfgSupported + mQemuFwCfgDmaSupported")

Cc: Ard Biesheuvel ardb+tianocore@kernel.org
Cc: Jiewen Yao jiewen.yao@intel.com
Cc: Jordan Justen jordan.l.justen@intel.com
Cc: Gerd Hoffmann kraxel@redhat.com
Cc: Rebecca Cran rebecca@bsdio.com
Cc: Peter Grehan grehan@freebsd.org

@ckoehne ckoehne force-pushed the phab/corvink/platforminfohob branch from aad2b91 to b178f3e Compare July 24, 2023 13:11
@ckoehne ckoehne force-pushed the phab/corvink/platforminfohob branch from b178f3e to 1a5e779 Compare August 7, 2023 10:41
Copy the function BuildPlatformInfoHob() from OvmfPkg/PlatformPei.

QemuFwCfgLib expects this HOB to be present, or fails to do anything.
InternalQemuFwCfgIsAvailable() from QemuFwCfgPeiLib module will not
check if the HOB is actually present for example and try to use a NULL
pointer.

Fixes: cda98df ("OvmfPkg/QemuFwCfgLib: remove mQemuFwCfgSupported + mQemuFwCfgDmaSupported")
Signed-off-by: Corvin Köhne <corvink@FreeBSD.org>
Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
Cc: Jiewen Yao <jiewen.yao@intel.com>
Cc: Jordan Justen <jordan.l.justen@intel.com>
Cc: Gerd Hoffmann <kraxel@redhat.com>
Cc: Rebecca Cran <rebecca@bsdio.com>
Cc: Peter Grehan <grehan@freebsd.org>
@ckoehne ckoehne force-pushed the phab/corvink/platforminfohob branch from 1a5e779 to 413339e Compare August 7, 2023 10:43
@mergify
Copy link

mergify bot commented Aug 25, 2023

PR can not be merged due to conflict. Please rebase and resubmit

@ckoehne ckoehne closed this Aug 25, 2023
@ckoehne ckoehne deleted the phab/corvink/platforminfohob branch August 25, 2023 05:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant