|
Re: [PATCH 1/1] OvmfPkg PlatformBootManagerLib: Move TryRunningQemuKernel()
Thanks for your interest Gerd,
I added "-serial mon:stdio" to the qemu invocation and "console=ttyS0"
to the -append line to capture all boot output for both good and bad
cases and saved the results
Thanks for your interest Gerd,
I added "-serial mon:stdio" to the qemu invocation and "console=ttyS0"
to the -append line to capture all boot output for both good and bad
cases and saved the results
|
By
Christoph Willing
·
#79027
·
|
|
Re: [PATCH v6 6/6] OvmfPkg/AmdSevDxe: Add support for SEV live migration.
Hello Tom,
I don't think we should do an assert here and abort booting, failure
here will simply disable live migration support but i don't think that
it is such a fatal error that we should abort
Hello Tom,
I don't think we should do an assert here and abort booting, failure
here will simply disable live migration support but i don't think that
it is such a fatal error that we should abort
|
By
Ashish Kalra
·
#79026
·
|
|
Re: [PATCH v2 1/2] MdePkg/BaseLib: Fix unaligned API prototypes
They do contain UB, inherently from the combination of their prototype and their usage.
Please refer to the new BZ added for V2: https://bugzilla.tianocore.org/show_bug.cgi?id=3542
I could only
They do contain UB, inherently from the combination of their prototype and their usage.
Please refer to the new BZ added for V2: https://bugzilla.tianocore.org/show_bug.cgi?id=3542
I could only
|
By
Marvin Häuser <mhaeuser@...>
·
#79025
·
|
|
Re: [PATCH v2 2/2] UefiCpuPkg/BaseUefiCpuLib: Use toolchain-specific rodata section name
The latter part is true only for Xcode-based toolchains, as far as I am aware now, and this is in fact the way it works right now.
You are correct, but the NASM section name semantically translates
The latter part is true only for Xcode-based toolchains, as far as I am aware now, and this is in fact the way it works right now.
You are correct, but the NASM section name semantically translates
|
By
Marvin Häuser <mhaeuser@...>
·
#79024
·
|
|
Re: [PATCH v2 1/2] BaseTools: Define the read-only data section name per toolchain
Yes, and ".rodata" is almost a synonym for "__DATA,__const", with a small exception [1]. Maybe it'd be clearer if the macro was renamed to "NASM_RODATA_SECTION_NAME", to indicate this is not just a
Yes, and ".rodata" is almost a synonym for "__DATA,__const", with a small exception [1]. Maybe it'd be clearer if the macro was renamed to "NASM_RODATA_SECTION_NAME", to indicate this is not just a
|
By
Marvin Häuser <mhaeuser@...>
·
#79023
·
|
|
Re: [PATCH 0/3] BaseTools: fix some python DeprecationWarnings
Hi Robinson,
When we doing the internal test, the issue is found that py27 is blocked by this patch.
You can reproduce the error with below steps:
For Linux:
[Error Reproduce steps]
$ export
Hi Robinson,
When we doing the internal test, the issue is found that py27 is blocked by this patch.
You can reproduce the error with below steps:
For Linux:
[Error Reproduce steps]
$ export
|
By
Yuwei Chen
·
#79022
·
|
|
[edk2-platforms:PATCH V6] Platform/Intel: Correct CPU APIC IDs
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3365
BIOS cannot find correct AcpiProcId in mApicIdMap because of
there is no suitable map, that causes ACPI_BIOS_ERROR.
Remove mApicIdMap for
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3365
BIOS cannot find correct AcpiProcId in mApicIdMap because of
there is no suitable map, that causes ACPI_BIOS_ERROR.
Remove mApicIdMap for
|
By
JackX Lin
·
#79021
·
|
|
Re: [PATCH] SecurityPkg: Debug code to audit BIOS TPM extend operations.
Hi Jiewen,
The intention of such API would be to ease debugging and auditing PCR attestation along the boot; it has been a common task while debugging several issues and TPM configurations.
a)
Hi Jiewen,
The intention of such API would be to ease debugging and auditing PCR attestation along the boot; it has been a common task while debugging several issues and TPM configurations.
a)
|
By
Rodrigo Gonzalez del Cueto
·
#79020
·
|
|
Re: [PATCH] Reallocate TPM Active PCRs based on platform support.
Hi Jiewen,
Indeed, this bug has existed for a long time in this code. What recently changed are the TPM configurations we are testing and exposed the issue; this can be reproduced when theBIOS
Hi Jiewen,
Indeed, this bug has existed for a long time in this code. What recently changed are the TPM configurations we are testing and exposed the issue; this can be reproduced when theBIOS
|
By
Rodrigo Gonzalez del Cueto
·
#79019
·
|
|
Re: [PATCH v6 1/6] OvmfPkg/BaseMemEncryptLib: Detect SEV live migration feature.
Hi,
Nope. When you enable hyper-v emulation features you'll go find the kvm
cpuid @ 0x40000000 and the hyper-v cpuid @ 0x40000100 (or the other way
around, not sure).
take care,
Gerd
Hi,
Nope. When you enable hyper-v emulation features you'll go find the kvm
cpuid @ 0x40000000 and the hyper-v cpuid @ 0x40000100 (or the other way
around, not sure).
take care,
Gerd
|
By
Gerd Hoffmann
·
#79018
·
|
|
Re: [PATCH 1/1] OvmfPkg PlatformBootManagerLib: Move TryRunningQemuKernel()
Hi,
Root cause is not clear. Grab the kernel log ('dmesg') from a good and
a bad boot and comparing them should give a clue.
take care,
Gerd
Hi,
Root cause is not clear. Grab the kernel log ('dmesg') from a good and
a bad boot and comparing them should give a clue.
take care,
Gerd
|
By
Gerd Hoffmann
·
#79017
·
|
|
Re: [PATCH v2 2/2] UefiCpuPkg/BaseUefiCpuLib: Use toolchain-specific rodata section name
Ray,
I made a detailed response about Mach-O with Xcode/clang and I don’t think patch works. Not sure if it breaks anything, but it puts things in the .data PE/COFF section.
I’m also worried it
Ray,
I made a detailed response about Mach-O with Xcode/clang and I don’t think patch works. Not sure if it breaks anything, but it puts things in the .data PE/COFF section.
I’m also worried it
|
By
Andrew Fish
·
#79016
·
|
|
[edk2-platforms] [PATCH V1] MinPlatformPkg: Cleanup PeiFspWrapperHobProcessLib dependencies
PeiFspWrapperHobProcessLib is currently set to depens on
FspWrapperPlatformLib, but it does not use any of the
functions implemented by that LibraryClass. This change
removes that unneeded
PeiFspWrapperHobProcessLib is currently set to depens on
FspWrapperPlatformLib, but it does not use any of the
functions implemented by that LibraryClass. This change
removes that unneeded
|
By
Nate DeSimone
·
#79015
·
|
|
[edk2-platforms] [PATCH V1] KabylakeSiliconPkg: Update SA_MISC_PEI_PREMEM_CONFIG
Updates SA_MISC_PEI_PREMEM_CONFIG from revision 1
to revision 3. Add initialization of the policy values.
Cc: Chasel Chiu <chasel.chiu@...>
Cc: Michael Kubacki
Updates SA_MISC_PEI_PREMEM_CONFIG from revision 1
to revision 3. Add initialization of the policy values.
Cc: Chasel Chiu <chasel.chiu@...>
Cc: Michael Kubacki
|
By
Nate DeSimone
·
#79014
·
|
|
Re: [PATCH v2 1/2] BaseTools: Define the read-only data section name per toolchain
An EFI Mach-O file does not contain a .rodata section. A Mach-O contains a __DATA segment that is broken up into sections. For a typical EFI image there are __const, __data, __bss sections in the
An EFI Mach-O file does not contain a .rodata section. A Mach-O contains a __DATA segment that is broken up into sections. For a typical EFI image there are __const, __data, __bss sections in the
|
By
Andrew Fish
·
#79013
·
|
|
Re: [PATCH EDK2 v2 1/1] MdeModulePkg/UefiSortLib:Add UefiSortLib unit test
Sorry Mike,
Do you have advice on how to deal with ECC reporting function naming issue on the 'main' function for unit test cases?
So far, I think Wenyi has tried following the same pattern in file
Sorry Mike,
Do you have advice on how to deal with ECC reporting function naming issue on the 'main' function for unit test cases?
So far, I think Wenyi has tried following the same pattern in file
|
By
Wu, Hao A
·
#79012
·
|
|
[PATCH v2 6/6] Platform/RaspberryPi: Enable NVMe boot on CM4
The CM4 has a number of carrier boards with PCIe
slots. With the PCIe changes in place its quite
possible to utilize a NVMe root device. Lets allow
people to boot from it.
Reviewed-by: Andrei
The CM4 has a number of carrier boards with PCIe
slots. With the PCIe changes in place its quite
possible to utilize a NVMe root device. Lets allow
people to boot from it.
Reviewed-by: Andrei
|
By
Jeremy Linton
·
#79011
·
|
|
[PATCH v2 5/6] Silicon/Broadcom/Bcm27xx: Move linkup check into the cfg accessor
The existing code fails to create/finish configuring the
pcie subsystem if it fails to get a linkup. This is reasonable
on the RPi4 because it generally won't happen, and the OS
could not see the root
The existing code fails to create/finish configuring the
pcie subsystem if it fails to get a linkup. This is reasonable
on the RPi4 because it generally won't happen, and the OS
could not see the root
|
By
Jeremy Linton
·
#79010
·
|
|
[PATCH v2 4/6] Silicon/Broadcom/Bcm27xx: Relax PCIe device restriction
The CM4 has an actual PCIe slot, so the device filtering
need to be a little less restrictive WRT busses with more
than 1 device given that switches can now appear in the
topology. Since it is
The CM4 has an actual PCIe slot, so the device filtering
need to be a little less restrictive WRT busses with more
than 1 device given that switches can now appear in the
topology. Since it is
|
By
Jeremy Linton
·
#79009
·
|
|
[PATCH v2 3/6] Platform/RaspberryPi: Add PCIe SSDT
Since we plan on toggling between XHCI and PCI the PCI
root needs to be in its own SSDT. This is all thats needed
of UEFI. The SMC conduit is provided directly to the running
OS. When the OS detects
Since we plan on toggling between XHCI and PCI the PCI
root needs to be in its own SSDT. This is all thats needed
of UEFI. The SMC conduit is provided directly to the running
OS. When the OS detects
|
By
Jeremy Linton
·
#79008
·
|