[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[ovmf test] 175934: regressions - FAIL



flight 175934 ovmf real [real]
http://logs.test-lab.xenproject.org/osstest/logs/175934/

Regressions :-(

Tests which did not succeed and are blocking,
including tests which could not be run:
 build-amd64-xsm               6 xen-build                fail REGR. vs. 175747
 build-i386                    6 xen-build                fail REGR. vs. 175747
 build-amd64                   6 xen-build                fail REGR. vs. 175747
 build-i386-xsm                6 xen-build                fail REGR. vs. 175747

Tests which did not succeed, but are not blocking:
 build-amd64-libvirt           1 build-check(1)               blocked  n/a
 build-i386-libvirt            1 build-check(1)               blocked  n/a
 test-amd64-amd64-xl-qemuu-ovmf-amd64  1 build-check(1)             blocked n/a
 test-amd64-i386-xl-qemuu-ovmf-amd64  1 build-check(1)              blocked n/a

version targeted for testing:
 ovmf                 d05739a3ff88457ae3ce90db3e91e9d2a11949c8
baseline version:
 ovmf                 9d70d8f20d0feee1d232cbf86fc87147ce92c2cb

Last test of basis   175747  2023-01-12 16:10:44 Z    4 days
Failing since        175860  2023-01-15 07:11:07 Z    2 days   27 attempts
Testing same since   175934  2023-01-17 13:40:59 Z    0 days    1 attempts

------------------------------------------------------------
People who touched revisions under test:
  Abner Chang <abner.chang@xxxxxxx>
  Gerd Hoffmann <kraxel@xxxxxxxxxx>
  Jiangang He <jiangang.he@xxxxxxx>
  Konstantin Aladyshev <aladyshev22@xxxxxxxxx>
  Min M Xu <min.m.xu@xxxxxxxxx>
  Min Xu <min.m.xu@xxxxxxxxx>

jobs:
 build-amd64-xsm                                              fail    
 build-i386-xsm                                               fail    
 build-amd64                                                  fail    
 build-i386                                                   fail    
 build-amd64-libvirt                                          blocked 
 build-i386-libvirt                                           blocked 
 build-amd64-pvops                                            pass    
 build-i386-pvops                                             pass    
 test-amd64-amd64-xl-qemuu-ovmf-amd64                         blocked 
 test-amd64-i386-xl-qemuu-ovmf-amd64                          blocked 


------------------------------------------------------------
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 d05739a3ff88457ae3ce90db3e91e9d2a11949c8
Author: Konstantin Aladyshev <aladyshev22@xxxxxxxxx>
Date:   Wed Dec 14 00:22:22 2022 +0800

    Fix cyclic dependency error on OptionROM build
    
    EDKII build system supports OptionROM generation if particular PCI_*
    defines are present in the module INF file:
    ```
    [Defines]
      ...
      PCI_VENDOR_ID                  = <...>
      PCI_DEVICE_ID                  = <...>
      PCI_CLASS_CODE                 = <...>
      PCI_REVISION                   = <...>
    ```
    Although after the commit d372ab585a2cdc5348af5f701c56c631235fe698
    ("BaseTools/Conf: Fix Dynamic-Library-File template") it is no longer
    possible.
    The build system fails with the error:
    ```
    Cyclic dependency detected while generating rule for
    "<...>/DEBUG/<...>.efi" file
    ```
    Remove "$(DEBUG_DIR)(+)$(MODULE_NAME).efi" from the 'dll' output files
    to fix the cyclic dependency.
    
    Signed-off-by: Konstantin Aladyshev <aladyshev22@xxxxxxxxx>
    Reviewed-by: Bob Feng <bob.c.feng@xxxxxxxxx>

commit 987cc09c7cf38d628063062483e2341fba679b0e
Author: Gerd Hoffmann <kraxel@xxxxxxxxxx>
Date:   Mon Jan 16 10:46:39 2023 +0100

    ArmVirt: don't use unaligned CopyMem () on NOR flash
    
    Commit 789a72328553 reclassified the NOR flash region as EFI_MEMORY_WC
    in the OS visible EFI memory map, and dropped the explicit aligned
    CopyMem() implementation, in the assumption that EFI_MEMORY_WC will be
    honored by the OS, and that the region will be mapped in a way that
    tolerates misaligned accesseses. However, Linux today uses device
    attributes for all EFI MMIO regions, in spite of the memory type
    attributes, and so using misaligned accesses is never safe.
    
    So instead, switch to the generic CopyMem() implementation entirely,
    just like we already did for VariableRuntimeDxe.
    
    Fixes: 789a72328553 ("OvmfPkg/VirtNorFlashDxe: use EFI_MEMORY_WC and drop 
AlignedCopyMem()")
    Signed-off-by: Gerd Hoffmann <kraxel@xxxxxxxxxx>
    Reviewed-by: Ard Biesheuvel <ardb@xxxxxxxxxx>

commit 47ab397011b6d1ce4d5805117dc87d9e35f378db
Author: Abner Chang <abner.chang@xxxxxxx>
Date:   Wed Jan 11 11:10:08 2023 +0800

    MdeModulePkg/XhciPei: Unlinked XhciPei memory block
    
    Unlink the XhciPei memory block when it has been freed.
    
    Signed-off-by: Jiangang He <jiangang.he@xxxxxxx>
    Cc: Hao A Wu <hao.a.wu@xxxxxxxxx>
    Cc: Ray Ni <ray.ni@xxxxxxxxx>
    Cc: Garrett Kirkendall <garrett.kirkendall@xxxxxxx>
    Cc: Abner Chang <abner.chang@xxxxxxx>
    Cc: Kuei-Hung Lin <Kuei-Hung.Lin@xxxxxxx>
    Reviewed-by: Hao A Wu <hao.a.wu@xxxxxxxxx>

commit be8d6ef3856fac2e64e23847a8f05d37822b1f14
Author: Abner Chang <abner.chang@xxxxxxx>
Date:   Wed Jan 11 11:10:07 2023 +0800

    MdeModulePkg/Usb: Read a large number of blocks
    
    Changes to allow reading blocks that greater than 65535 sectors.
    
    Signed-off-by: Jiangang He <jiangang.he@xxxxxxx>
    Cc: Hao A Wu <hao.a.wu@xxxxxxxxx>
    Cc: Ray Ni <ray.ni@xxxxxxxxx>
    Cc: Garrett Kirkendall <garrett.kirkendall@xxxxxxx>
    Cc: Abner Chang <abner.chang@xxxxxxx>
    Cc: Kuei-Hung Lin <Kuei-Hung.Lin@xxxxxxx>
    Reviewed-by: Hao A Wu <hao.a.wu@xxxxxxxxx>

commit 8147fe090fb566f9a1ed8fde24098bbe425026be
Author: Abner Chang <abner.chang@xxxxxxx>
Date:   Wed Jan 11 11:10:06 2023 +0800

    MdeModulePkg/Xhci: Initial XHCI DCI slot's Context value
    
    Initialize XHCI DCI slot's context entries value.
    
    Signed-off-by: Jiangang He <jiangang.he@xxxxxxx>
    Cc: Hao A Wu <hao.a.wu@xxxxxxxxx>
    Cc: Ray Ni <ray.ni@xxxxxxxxx>
    Cc: Garrett Kirkendall <garrett.kirkendall@xxxxxxx>
    Cc: Abner Chang <abner.chang@xxxxxxx>
    Cc: Kuei-Hung Lin <Kuei-Hung.Lin@xxxxxxx>
    Reviewed-by: Hao A Wu <hao.a.wu@xxxxxxxxx>

commit 7cd55f300915af8759bdf1687af7e3a7f4d4f13c
Author: Min M Xu <min.m.xu@xxxxxxxxx>
Date:   Wed Jan 11 09:22:35 2023 +0800

    OvmfPkg/AcpiPlatformDxe: Return error if installing NotifyProtocol failed
    
    BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4237
    
    Installation of gQemuAcpiTableNotifyProtocol may fail. The error code
    should be returned so that the caller can handle it.
    
    Cc: Erdem Aktas <erdemaktas@xxxxxxxxxx>
    Cc: James Bottomley <jejb@xxxxxxxxxxxxx>
    Cc: Jiewen Yao <jiewen.yao@xxxxxxxxx>
    Cc: Gerd Hoffmann <kraxel@xxxxxxxxxx>
    Cc: Tom Lendacky <thomas.lendacky@xxxxxxx>
    Cc: Sebastien Boeuf <sebastien.boeuf@xxxxxxxxx>
    Signed-off-by: Min Xu <min.m.xu@xxxxxxxxx>
    Message-Id: <20230111012235.189-7-min.m.xu@xxxxxxxxx>
    Reviewed-by: Sebastien Boeuf <sebastien.boeuf@xxxxxxxxx>

commit 66f18fde49c7fe65818db0801cdaf63015e875e5
Author: Min M Xu <min.m.xu@xxxxxxxxx>
Date:   Wed Jan 11 09:22:34 2023 +0800

    OvmfPkg/AcpiPlatformDxe: Refactor QemuAcpiTableNotifyProtocol
    
    BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4237
    
    Commit 9fdc70af6ba8 install the QemuAcpiTableNotifyProtocol at a
    wrong positioin. It should be called before TransferS3ContextToBootScript
    because TransferS3ContextToBootScript is the last operation in
    InstallQemuFwCfgTables(). Another error is that we should check the
    returned value after installing the QemuAcpiTableNotifyProtocol.
    
    This patch refactors the installation and error handling of
    QemuAcpiTableNotifyProtocol in InstallQemuFwCfgTables ().
    
    Cc: Laszlo Ersek <lersek@xxxxxxxxxx>
    Cc: Erdem Aktas <erdemaktas@xxxxxxxxxx>
    Cc: James Bottomley <jejb@xxxxxxxxxxxxx>
    Cc: Jiewen Yao <jiewen.yao@xxxxxxxxx>
    Cc: Gerd Hoffmann <kraxel@xxxxxxxxxx>
    Cc: Tom Lendacky <thomas.lendacky@xxxxxxx>
    Reported-by: Laszlo Ersek <lersek@xxxxxxxxxx>
    Signed-off-by: Min Xu <min.m.xu@xxxxxxxxx>
    Message-Id: <20230111012235.189-6-min.m.xu@xxxxxxxxx>
    Reviewed-by: Laszlo Ersek <lersek@xxxxxxxxxx>

commit 2ef0ff39e53d2d2af3859b783882eea6f0beda64
Author: Min M Xu <min.m.xu@xxxxxxxxx>
Date:   Wed Jan 11 09:22:33 2023 +0800

    OvmfPkg/AcpiPlatformDxe: Add log to show the installed tables
    
    BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4237
    
    Commit 9fdc70af6ba8 wrongly removed the log from InstallQemuFwCfgTables
    after ACPI tables are successfully installed. This patch add the log
    back after all operations succeed.
    
    Cc: Laszlo Ersek <lersek@xxxxxxxxxx>
    Cc: Erdem Aktas <erdemaktas@xxxxxxxxxx>
    Cc: James Bottomley <jejb@xxxxxxxxxxxxx>
    Cc: Jiewen Yao <jiewen.yao@xxxxxxxxx>
    Cc: Gerd Hoffmann <kraxel@xxxxxxxxxx>
    Cc: Tom Lendacky <thomas.lendacky@xxxxxxx>
    Reported-by: Laszlo Ersek <lersek@xxxxxxxxxx>
    Reviewed-by: Laszlo Ersek <lersek@xxxxxxxxxx>
    Signed-off-by: Min Xu <min.m.xu@xxxxxxxxx>
    Message-Id: <20230111012235.189-5-min.m.xu@xxxxxxxxx>

commit 165f1e49361a9a5f5936f2d582641096d0d7a2a2
Author: Min M Xu <min.m.xu@xxxxxxxxx>
Date:   Wed Jan 11 09:22:32 2023 +0800

    OvmfPkg/AcpiPlatformDxe: Use local variable in QemuFwCfgAcpi.c
    
    BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4237
    
    The handle of mQemuAcpiHandle is not needed for anything, beyond the
    scope of the InstallQemuFwCfgTables(). So a local variable will
    suffice for storing the handle.
    
    Cc: Laszlo Ersek <lersek@xxxxxxxxxx>
    Cc: Erdem Aktas <erdemaktas@xxxxxxxxxx>
    Cc: James Bottomley <jejb@xxxxxxxxxxxxx>
    Cc: Jiewen Yao <jiewen.yao@xxxxxxxxx>
    Cc: Gerd Hoffmann <kraxel@xxxxxxxxxx>
    Cc: Tom Lendacky <thomas.lendacky@xxxxxxx>
    Reported-by: Laszlo Ersek <lersek@xxxxxxxxxx>
    Reviewed-by: Laszlo Ersek <lersek@xxxxxxxxxx>
    Signed-off-by: Min Xu <min.m.xu@xxxxxxxxx>
    Message-Id: <20230111012235.189-4-min.m.xu@xxxxxxxxx>

commit f81273f7fbb3defbef43313ada8397bbc202a1d0
Author: Min M Xu <min.m.xu@xxxxxxxxx>
Date:   Wed Jan 11 09:22:31 2023 +0800

    OvmfPkg/AcpiPlatformDxe: Use local variable in CloudHvAcpi.c
    
    BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4237
    
    The handle of mChAcpiHandle is not needed for anything, beyond the
    scope of the InstallCloudHvTablesTdx (). A local variable (ChAcpiHandle)
    suffices for storing the handle.
    
    Cc: Laszlo Ersek <lersek@xxxxxxxxxx>
    Cc: Erdem Aktas <erdemaktas@xxxxxxxxxx>
    Cc: James Bottomley <jejb@xxxxxxxxxxxxx>
    Cc: Jiewen Yao <jiewen.yao@xxxxxxxxx>
    Cc: Gerd Hoffmann <kraxel@xxxxxxxxxx>
    Cc: Tom Lendacky <thomas.lendacky@xxxxxxx>
    Cc: Sebastien Boeuf <sebastien.boeuf@xxxxxxxxx>
    Reported-by: Laszlo Ersek <lersek@xxxxxxxxxx>
    Signed-off-by: Min Xu <min.m.xu@xxxxxxxxx>
    Message-Id: <20230111012235.189-3-min.m.xu@xxxxxxxxx>
    Reviewed-by: Sebastien Boeuf <sebastien.boeuf@xxxxxxxxx>

commit 43b3ca6b7f626c6dcdc1a347ad8a42d8cf9ea575
Author: Min M Xu <min.m.xu@xxxxxxxxx>
Date:   Wed Jan 11 09:22:30 2023 +0800

    OvmfPkg/AcpiPlatformDxe: Remove QEMU_ACPI_TABLE_NOTIFY_PROTOCOL
    
    BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4237
    
    The QEMU_ACPI_TABLE_NOTIFY_PROTOCOL structure is superfluous because NULL
    protocol interfaces have been used in edk2 repeatedly. A protocol instance
    can exist in the protocol database with a NULL associated interface.
    Therefore the QEMU_ACPI_TABLE_NOTIFY_PROTOCOL type, the
    "QemuAcpiTableNotify.h" header, and the "mAcpiNotifyProtocol" global
    variable can be removed.
    
    Cc: Laszlo Ersek <lersek@xxxxxxxxxx>
    Cc: Erdem Aktas <erdemaktas@xxxxxxxxxx>
    Cc: James Bottomley <jejb@xxxxxxxxxxxxx>
    Cc: Jiewen Yao <jiewen.yao@xxxxxxxxx>
    Cc: Gerd Hoffmann <kraxel@xxxxxxxxxx>
    Cc: Tom Lendacky <thomas.lendacky@xxxxxxx>
    Cc: Sebastien Boeuf <sebastien.boeuf@xxxxxxxxx>
    Reported-by: Laszlo Ersek <lersek@xxxxxxxxxx>
    Reviewed-by: Laszlo Ersek <lersek@xxxxxxxxxx>
    Signed-off-by: Min Xu <min.m.xu@xxxxxxxxx>
    Message-Id: <20230111012235.189-2-min.m.xu@xxxxxxxxx>
    Reviewed-by: Sebastien Boeuf <sebastien.boeuf@xxxxxxxxx>

commit ba08910df1071bf5ade987529d9becb38d14a14a
Author: Gerd Hoffmann <kraxel@xxxxxxxxxx>
Date:   Thu Jan 12 23:41:02 2023 +0800

    OvmfPkg: fix OvmfTpmSecurityStub.dsc.inc include
    
    TPM support is independent from secure boot support.  Move the TPM
    include snipped out of the secure boot !if block.
    
    Fixes: b47575801e19 ("OvmfPkg: move tcg configuration to dsc and fdf 
include files")
    Bugzilla: https://bugzilla.tianocore.org//show_bug.cgi?id=4290
    Signed-off-by: Gerd Hoffmann <kraxel@xxxxxxxxxx>
    Reviewed-by: Jiewen Yao <jiewen.yao@xxxxxxxxx>



 


Rackspace

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