|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [ovmf test] 175273: regressions - FAIL
flight 175273 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/175273/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemuu-ovmf-amd64 12 debian-hvm-install fail REGR. vs. 175202 test-amd64-amd64-xl-qemuu-ovmf-amd64 12 debian-hvm-install fail REGR. vs. 175202 version targeted for testing: ovmf 3e3f5bb21c0a2c1368c43713cf7f4b51097259af baseline version: ovmf d103840cfb559c28831c2635b916d60118f671cc Last test of basis 175202 2022-12-14 13:42:59 Z 1 days Failing since 175214 2022-12-14 18:42:16 Z 1 days 8 attempts Testing same since 175263 2022-12-15 09:40:40 Z 0 days 2 attempts ------------------------------------------------------------ People who touched revisions under test: Adam Dunlap <acdunlap@xxxxxxxxxx> Ard Biesheuvel <ardb@xxxxxxxxxx> devel@xxxxxxxxxxxxxx <devel@xxxxxxxxxxxxxx> Gerd Hoffmann <kraxel@xxxxxxxxxx> Jeff Brasen <jbrasen@xxxxxxxxxx> Jeshua Smith <jeshuas@xxxxxxxxxx> Min M Xu <min.m.xu@xxxxxxxxx> Min Xu <min.m.xu@xxxxxxxxx> Tom Lendacky <thomas.lendacky@xxxxxxx> jobs: build-amd64-xsm pass build-i386-xsm pass build-amd64 pass build-i386 pass build-amd64-libvirt pass build-i386-libvirt pass build-amd64-pvops pass build-i386-pvops pass test-amd64-amd64-xl-qemuu-ovmf-amd64 fail test-amd64-i386-xl-qemuu-ovmf-amd64 fail ------------------------------------------------------------ sg-report-flight on osstest.test-lab.xenproject.org logs: /home/logs/logs images: /home/logs/images Logs, config files, etc. are available at http://logs.test-lab.xenproject.org/osstest/logs Explanation of these reports, and of osstest in general, is at http://xenbits.xen.org/gitweb/?p=osstest.git;a=blob;f=README.email;hb=master http://xenbits.xen.org/gitweb/?p=osstest.git;a=blob;f=README;hb=master Test harness code can be found at http://xenbits.xen.org/gitweb?p=osstest.git;a=summary Not pushing. ------------------------------------------------------------ commit 3e3f5bb21c0a2c1368c43713cf7f4b51097259af Author: Adam Dunlap <acdunlap@xxxxxxxxxx> Date: Sat Dec 10 05:04:16 2022 +0800 OvmfPkg/PlatformPei: Validate SEC's GHCB page When running under SEV-ES, a page of shared memory is allocated for the GHCB during the SEC phase at address 0x809000. This page of memory is eventually passed to the OS as EfiConventionalMemory. When running SEV-SNP, this page is not PVALIDATE'd in the RMP table, meaning that if the guest OS tries to access the page, it will think that the host has voilated the security guarantees and will likely crash. This patch validates this page immediately after EDK2 switches to using the GHCB page allocated for the PEI phase. This was tested by writing a UEFI application that reads to and writes from one byte of each page of memory and checks to see if a #VC exception is generated indicating that the page was not validated. Fixes: 6995a1b79bab ("OvmfPkg: Create a GHCB page for use during Sec phase") Signed-off-by: Adam Dunlap <acdunlap@xxxxxxxxxx> Reviewed-by: Tom Lendacky <thomas.lendacky@xxxxxxx> commit 01c0d3c0d508b8c1b41fd58e2ec565b40ea000ca Author: Min M Xu <min.m.xu@xxxxxxxxx> Date: Wed Dec 14 15:14:19 2022 +0800 OvmfPkg/SecTpmMeasurementLib: Fix the mapping error of PCR and RTMR index BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4179 TDVF has the feature to do RTMR measurement in SEC phase. In the same time it builds a GUID hob which carries the hash value of the measurement so that in DXE phase a td event can be created based on this GUID Hob. There is a mapping error between TPM PCR index and RTMR index according to UEFI 2.10. That PCR6 is missing in the mapping. This patch fixes this issue. Cc: Erdem Aktas <erdemaktas@xxxxxxxxxx> [ruleof2] Cc: James Bottomley <jejb@xxxxxxxxxxxxx> [jejb] Cc: Jiewen Yao <jiewen.yao@xxxxxxxxx> [jyao1] Cc: Tom Lendacky <thomas.lendacky@xxxxxxx> [tlendacky] Cc: Arti Gupta <ARGU@xxxxxxxxxxxxx> Signed-off-by: Min Xu <min.m.xu@xxxxxxxxx> Reviewed-by: Jiewen Yao <jiewen.yao@xxxxxxxxx> commit fb91d6cbd0cff704586c4cadbef870acef9c52ef Author: Min M Xu <min.m.xu@xxxxxxxxx> Date: Wed Dec 14 15:14:18 2022 +0800 OvmfPkg/TdTcg2Dxe: Fix the mapping error between PCR index and MR index BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4179 According to UEFI Spec 2.10 it is supposed to return the mapping from PCR index to CC MR index: // // In the current version, we use the below mapping for TDX: // // TPM PCR Index | CC Measurement Register Index | TDX-measurement register // ----------------------------------------------------------------------- // 0 | 0 | MRTD // 1, 7 | 1 | RTMR[0] // 2~6 | 2 | RTMR[1] // 8~15 | 3 | RTMR[2] In the current implementation TdMapPcrToMrIndex returns the index of RTMR, not the MR index. After fix the spec unconsistent, other related codes are updated accordingly. 1) The index of event log uses the input MrIndex. 2) MrIndex is decreated by 1 before it is sent for RTMR extending. Cc: Erdem Aktas <erdemaktas@xxxxxxxxxx> [ruleof2] Cc: James Bottomley <jejb@xxxxxxxxxxxxx> [jejb] Cc: Jiewen Yao <jiewen.yao@xxxxxxxxx> [jyao1] Cc: Tom Lendacky <thomas.lendacky@xxxxxxx> [tlendacky] Cc: Arti Gupta <ARGU@xxxxxxxxxxxxx> Reported-by: Arti Gupta <ARGU@xxxxxxxxxxxxx> Signed-off-by: Min Xu <min.m.xu@xxxxxxxxx> Reviewed-by: Jiewen Yao <jiewen.yao@xxxxxxxxx> commit 19f7c63ea978c5a4ef9345b9bf4995dcd48fa328 Author: Min M Xu <min.m.xu@xxxxxxxxx> Date: Wed Dec 14 15:14:17 2022 +0800 OvmfPkg/TdTcg2Dxe: Fix incorrect protocol and structure version BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4184 According to the Uefi spec 2.10 Section 38.2.2. EFI_CC_MEASUREMENT_PROTOCOL.GetCapability, the minor version of StructureVersion and ProtocolVersion should be 0. Cc: Erdem Aktas <erdemaktas@xxxxxxxxxx> [ruleof2] Cc: James Bottomley <jejb@xxxxxxxxxxxxx> [jejb] Cc: Jiewen Yao <jiewen.yao@xxxxxxxxx> [jyao1] Cc: Tom Lendacky <thomas.lendacky@xxxxxxx> [tlendacky] Cc: Arti Gupta <ARGU@xxxxxxxxxxxxx> Reported-by: Arti Gupta <ARGU@xxxxxxxxxxxxx> Signed-off-by: Min Xu <min.m.xu@xxxxxxxxx> Reviewed-by: Jiewen Yao <jiewen.yao@xxxxxxxxx> commit 44fc90eb0ea7299abc79577db55aa6257b46b7ae Author: devel@xxxxxxxxxxxxxx <devel@xxxxxxxxxxxxxx> Date: Wed Nov 30 15:02:15 2022 -0800 UnitTestFrameworkPkg/UnitTestLib: Print expected Status on ASSERT fail Update the UnitTestAssertStatusEqual error message to print out the expected value in addition to the seen value. Signed-off-by: Jeshua Smith <jeshuas@xxxxxxxxxx> Reviewed-by: Michael Kubacki <michael.kubacki@xxxxxxxxxxxxx> Reviewed-by: Michael D Kinney <michael.d.kinney@xxxxxxxxx> commit cda98df16228970dcf9a4ce2af5368219711b4b0 Author: Gerd Hoffmann <kraxel@xxxxxxxxxx> Date: Fri Dec 2 14:10:09 2022 +0100 OvmfPkg/QemuFwCfgLib: remove mQemuFwCfgSupported + mQemuFwCfgDmaSupported Remove global variables, store the state in PlatformInfoHob instead. Probing for fw_cfg happens on first use, at library initialization time the Hob might not be present yet. Signed-off-by: Gerd Hoffmann <kraxel@xxxxxxxxxx> Tested-by: Tom Lendacky <thomas.lendacky@xxxxxxx> Acked-by: Ard Biesheuvel <ardb@xxxxxxxxxx> commit 81bbc1452c972218f071cd4a8f5899df974b1dae Author: Gerd Hoffmann <kraxel@xxxxxxxxxx> Date: Fri Dec 2 14:10:08 2022 +0100 OvmfPkg/QemuFwCfgLib: rewrite fw_cfg probe Move the code to a new QemuFwCfgProbe() function. Use direct Io*() calls instead of indirect QemuFwCfg*() calls to make sure we don't get recursive calls. Also simplify CC guest detection. Signed-off-by: Gerd Hoffmann <kraxel@xxxxxxxxxx> Tested-by: Tom Lendacky <thomas.lendacky@xxxxxxx> Acked-by: Ard Biesheuvel <ardb@xxxxxxxxxx> commit e59747bd8246135faeecc18879d62db66a6acfc2 Author: Gerd Hoffmann <kraxel@xxxxxxxxxx> Date: Fri Dec 2 14:10:07 2022 +0100 OvmfPkg/DebugLibIoPort: use Rom version for PEI This variant does not use global variables. Signed-off-by: Gerd Hoffmann <kraxel@xxxxxxxxxx> Tested-by: Tom Lendacky <thomas.lendacky@xxxxxxx> Acked-by: Ard Biesheuvel <ardb@xxxxxxxxxx> commit f6a196c7eb34affff0cfe1864e126953096885e1 Author: Gerd Hoffmann <kraxel@xxxxxxxxxx> Date: Fri Dec 2 14:10:06 2022 +0100 OvmfPkg/PlatformPei: remove mFeatureControlValue Use PlatformInfoHob->FeatureControlValue instead. OnMpServicesAvailable() will find PlatformInfoHob using GetFirstGuidHob() and pass a pointer to the WriteFeatureControl callback. Signed-off-by: Gerd Hoffmann <kraxel@xxxxxxxxxx> Tested-by: Tom Lendacky <thomas.lendacky@xxxxxxx> Acked-by: Ard Biesheuvel <ardb@xxxxxxxxxx> commit 862614e2544997c848fab7388733774ae0ea92d8 Author: Gerd Hoffmann <kraxel@xxxxxxxxxx> Date: Fri Dec 2 14:10:05 2022 +0100 OvmfPkg/PlatformPei: remove mPlatformInfoHob Stop using the mPlatformInfoHob global variable. Let BuildPlatformInfoHob() allocate and return PlatformInfoHob instead. Signed-off-by: Gerd Hoffmann <kraxel@xxxxxxxxxx> Tested-by: Tom Lendacky <thomas.lendacky@xxxxxxx> Acked-by: Ard Biesheuvel <ardb@xxxxxxxxxx> commit 4bc2c748516e5c4a8bb86093cd5e1b80a9f35c0f Author: Gerd Hoffmann <kraxel@xxxxxxxxxx> Date: Fri Dec 2 14:10:04 2022 +0100 OvmfPkg/PlatformPei: Verification: stop using mPlatformInfoHob Stop using the mPlatformInfoHob global variable in S3Verification() and Q35BoardVerification() functions. Pass a pointer to the PlatformInfoHob instead. Signed-off-by: Gerd Hoffmann <kraxel@xxxxxxxxxx> Tested-by: Tom Lendacky <thomas.lendacky@xxxxxxx> Acked-by: Ard Biesheuvel <ardb@xxxxxxxxxx> commit 7dbb8a24d1a3403f85d959bc1234b9f4a92bfbf0 Author: Gerd Hoffmann <kraxel@xxxxxxxxxx> Date: Fri Dec 2 14:10:03 2022 +0100 OvmfPkg/PlatformPei: NoExec: stop using mPlatformInfoHob Stop using the mPlatformInfoHob global variable in NoexecDxeInitialization() function. Pass a pointer to the PlatformInfoHob instead. Signed-off-by: Gerd Hoffmann <kraxel@xxxxxxxxxx> Tested-by: Tom Lendacky <thomas.lendacky@xxxxxxx> Acked-by: Ard Biesheuvel <ardb@xxxxxxxxxx> commit cc6efda7770b8cabea3ae8c6054d47c4a8e229c3 Author: Gerd Hoffmann <kraxel@xxxxxxxxxx> Date: Fri Dec 2 14:10:02 2022 +0100 OvmfPkg/PlatformPei: MemTypeInfo: stop using mPlatformInfoHob Stop using the mPlatformInfoHob global variable in MemTypeInfoInitialization() function. Pass a pointer to the PlatformInfoHob instead. Signed-off-by: Gerd Hoffmann <kraxel@xxxxxxxxxx> Tested-by: Tom Lendacky <thomas.lendacky@xxxxxxx> Acked-by: Ard Biesheuvel <ardb@xxxxxxxxxx> commit 27874a382c38a95d3ab613eebd35c152ca9b3897 Author: Gerd Hoffmann <kraxel@xxxxxxxxxx> Date: Fri Dec 2 14:10:01 2022 +0100 OvmfPkg/PlatformPei: PeiMemory: stop using mPlatformInfoHob Stop using the mPlatformInfoHob global variable in PublishPeiMemory() and GetPeiMemoryCap() functions. Pass a pointer to the PlatformInfoHob instead. Signed-off-by: Gerd Hoffmann <kraxel@xxxxxxxxxx> Tested-by: Tom Lendacky <thomas.lendacky@xxxxxxx> Acked-by: Ard Biesheuvel <ardb@xxxxxxxxxx> commit 00743d144bc5b643e9323ad66f16cb48cf338705 Author: Gerd Hoffmann <kraxel@xxxxxxxxxx> Date: Fri Dec 2 14:10:00 2022 +0100 OvmfPkg/PlatformPei Q35 SMM helpers: stop using mPlatformInfoHob Stop using the mPlatformInfoHob global variable in Q35TsegMbytesInitialization() and Q35SmramAtDefaultSmbaseInitialization() ) functions. Pass a pointer to the PlatformInfoHob instead. Signed-off-by: Gerd Hoffmann <kraxel@xxxxxxxxxx> Tested-by: Tom Lendacky <thomas.lendacky@xxxxxxx> Acked-by: Ard Biesheuvel <ardb@xxxxxxxxxx> commit 9d9d15b42a5e13bb18729da0f608c629aa274e80 Author: Gerd Hoffmann <kraxel@xxxxxxxxxx> Date: Fri Dec 2 14:09:59 2022 +0100 OvmfPkg/PlatformPei: PeiFv: stop using mPlatformInfoHob Stop using the mPlatformInfoHob global variable in PeiFvInitialization() function. Pass a pointer to the PlatformInfoHob instead. Signed-off-by: Gerd Hoffmann <kraxel@xxxxxxxxxx> Tested-by: Tom Lendacky <thomas.lendacky@xxxxxxx> Acked-by: Ard Biesheuvel <ardb@xxxxxxxxxx> commit 78c373f2a5273af00b23b55d3e8c41583310cfb6 Author: Gerd Hoffmann <kraxel@xxxxxxxxxx> Date: Fri Dec 2 14:09:58 2022 +0100 OvmfPkg/PlatformPei: AmdSev: stop using mPlatformInfoHob Stop using the mPlatformInfoHob global variable in AmdSevInitialize() and AmdSevEsInitialize() functions. Pass a pointer to the PlatformInfoHob instead. Signed-off-by: Gerd Hoffmann <kraxel@xxxxxxxxxx> Tested-by: Tom Lendacky <thomas.lendacky@xxxxxxx> Acked-by: Ard Biesheuvel <ardb@xxxxxxxxxx> commit 916825b84f23b691dcce09a57625fb8fcb0cbb48 Author: Jeff Brasen <jbrasen@xxxxxxxxxx> Date: Tue Nov 15 11:01:06 2022 -0700 DynamicTablesPkg: SSDT _LPI revision is incorrect _LPI Revision should be 0 per the ACPI 6.5 specification. "The revision number of the _LPI object. Current revision is 0." Signed-off-by: Jeff Brasen <jbrasen@xxxxxxxxxx> Reviewed-by: Pierre Gondois <pierre.gondois@xxxxxxx> Reviewed-by: Sami Mujawar <sami.mujawar@xxxxxxx>
|
 |
Lists.xenproject.org is hosted with RackSpace, monitoring our |