[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>



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.