[Xen-devel] [ovmf test] 145476: regressions - FAIL

[osstest service owner <osstest-admin@xxxxxxxxxxxxxx>]

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

Regressions :-(

Tests which did not succeed and are blocking,
including tests which could not be run:
 test-amd64-amd64-xl-qemuu-ovmf-amd64  7 xen-boot         fail REGR. vs. 145179

version targeted for testing:
 ovmf                 fd30b0070773ac4ac5f49abca8f5b3afbeece158
baseline version:
 ovmf                 a5abd9cc2cebe7fac001f7bb7b647c47cf54af1a

Last test of basis   145179  2019-12-24 08:39:10 Z    8 days
Testing same since   145476  2020-01-02 03:39:12 Z    0 days    1 attempts

------------------------------------------------------------
People who touched revisions under test:
  Hao A Wu <hao.a.wu@xxxxxxxxx>

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                          pass    


------------------------------------------------------------
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 fd30b0070773ac4ac5f49abca8f5b3afbeece158
Author: Hao A Wu <hao.a.wu@xxxxxxxxx>
Date:   Tue Dec 24 15:02:23 2019 +0800

    UefiCpuPkg/MpInitLib: Remove redundant microcode fields in CPU_MP_DATA
    
    Previous commits have introduced below fields in structure CPU_AP_DATA:
    
      UINT32                         ProcessorSignature;
      UINT8                          PlatformId;
      UINT64                         MicrocodeEntryAddr;
    
    which store the information of:
    
    A. CPUID
    B. Platform ID
    C. Detected microcode patch entry address (including the microcode patch
       header)
    
    for each processor within system.
    
    Therefore, the below fields in structure CPU_MP_DATA:
    
      UINT32                         ProcessorSignature;
      UINT32                         ProcessorFlags;
      UINT64                         MicrocodeDataAddress;
      UINT32                         MicrocodeRevision;
    
    which store the BSP's information of:
    
    A. CPUID
    B. Platform ID
    C. The address and revision of detected microcode patch
    
    are redundant and can be removed.
    
    Cc: Eric Dong <eric.dong@xxxxxxxxx>
    Cc: Ray Ni <ray.ni@xxxxxxxxx>
    Cc: Laszlo Ersek <lersek@xxxxxxxxxx>
    Cc: Star Zeng <star.zeng@xxxxxxxxx>
    Cc: Siyuan Fu <siyuan.fu@xxxxxxxxx>
    Cc: Michael D Kinney <michael.d.kinney@xxxxxxxxx>
    Signed-off-by: Hao A Wu <hao.a.wu@xxxxxxxxx>
    Reviewed-by: Eric Dong <eric.dong@xxxxxxxxx>
    Reviewed-by: Ray Ni <ray.ni@xxxxxxxxx>

commit 88bd06616617ef2569f093f7b51893c11ad78e26
Author: Hao A Wu <hao.a.wu@xxxxxxxxx>
Date:   Wed Dec 25 10:50:19 2019 +0800

    UefiCpuPkg/MpInitLib: Relocate microcode patch fields in CPU_MP_DATA
    
    The below 2 microcode patch related fields in structure CPU_MP_DATA:
    
      UINT64                         MicrocodePatchAddress;
      UINT64                         MicrocodePatchRegionSize;
    
    They will be passed from PEI phase and be reused DXE phase.
    
    Previously, these 2 fields were placed after some fields with type
    'UINTN', this will lead to different field offset in different
    architecture for them.
    
    This commit will move them before the fields with different size in
    different architecture to ensure they can be properly used in DXE phase.
    
    Cc: Eric Dong <eric.dong@xxxxxxxxx>
    Cc: Ray Ni <ray.ni@xxxxxxxxx>
    Cc: Laszlo Ersek <lersek@xxxxxxxxxx>
    Cc: Star Zeng <star.zeng@xxxxxxxxx>
    Cc: Siyuan Fu <siyuan.fu@xxxxxxxxx>
    Cc: Michael D Kinney <michael.d.kinney@xxxxxxxxx>
    Signed-off-by: Hao A Wu <hao.a.wu@xxxxxxxxx>
    Reviewed-by: Ray Ni <ray.ni@xxxxxxxxx>
    Reviewed-by: Eric Dong <eric.dong@xxxxxxxxx>

commit e1ed55738ec30db364a00d0ddac50dbbf3671795
Author: Hao A Wu <hao.a.wu@xxxxxxxxx>
Date:   Mon Dec 23 14:32:49 2019 +0800

    UefiCpuPkg/MpInitLib: Produce EDKII microcode patch HOB
    
    REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2430
    
    This commit will update the MpInitLib to:
    
    A. Collect the base address and size information after microcode patches
       being loaded into memory;
    B. Collect the detected microcode patch for each processor within system;
    C. Based on the collected information, produce the EDKII microcode patch
       HOB.
    
    Cc: Eric Dong <eric.dong@xxxxxxxxx>
    Cc: Ray Ni <ray.ni@xxxxxxxxx>
    Cc: Laszlo Ersek <lersek@xxxxxxxxxx>
    Cc: Star Zeng <star.zeng@xxxxxxxxx>
    Cc: Siyuan Fu <siyuan.fu@xxxxxxxxx>
    Cc: Michael D Kinney <michael.d.kinney@xxxxxxxxx>
    Signed-off-by: Hao A Wu <hao.a.wu@xxxxxxxxx>
    Reviewed-by: Eric Dong <eric.dong@xxxxxxxxx>
    Reviewed-by: Ray Ni <ray.ni@xxxxxxxxx>

commit 253909974a0e2e9c60fa78e57c0c46fef2877332
Author: Hao A Wu <hao.a.wu@xxxxxxxxx>
Date:   Mon Dec 23 10:52:14 2019 +0800

    UefiCpuPkg: Add definitions for EDKII microcode patch HOB
    
    REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2430
    
    This commit will add the definitions for EDKII microcode patch HOB.
    
    The intention of adding this HOB is to provide a scheme to store the below
    information:
    
    A. The base address and size of the microcode patches that are being
       loaded (from flash) into memory;
    B. The information of detected microcode patch for each processor within
       the system.
    
    The producer of the HOB will be the UefiCpuPkg/MpInitLib (where the load,
    detect and apply of the microcode happen). The consumer of the HOB can be
    modules that want to detect/apply the microcode patch by themselves again
    later during the boot flow.
    
    Cc: Eric Dong <eric.dong@xxxxxxxxx>
    Cc: Ray Ni <ray.ni@xxxxxxxxx>
    Cc: Laszlo Ersek <lersek@xxxxxxxxxx>
    Cc: Star Zeng <star.zeng@xxxxxxxxx>
    Cc: Siyuan Fu <siyuan.fu@xxxxxxxxx>
    Cc: Michael D Kinney <michael.d.kinney@xxxxxxxxx>
    Signed-off-by: Hao A Wu <hao.a.wu@xxxxxxxxx>
    Reviewed-by: Ray Ni <ray.ni@xxxxxxxxx>
    Reviewed-by: Eric Dong <eric.dong@xxxxxxxxx>

commit d786a1723203e91c92615c58d217e35b0d894004
Author: Hao A Wu <hao.a.wu@xxxxxxxxx>
Date:   Thu Dec 19 14:33:44 2019 +0800

    UefiCpuPkg/MpInitLib: Reduce the size when loading microcode patches
    
    REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2429
    
    This commit will attempt to reduce the copy size when loading the
    microcode patches data from flash into memory.
    
    Such optimization is done by a pre-process of the microcode patch headers
    (on flash). A microcode patch will be loaded into memory only when the
    below 3 criteria are met:
    
    A. With a microcode patch header (which means the data is not padding data
       between microcode patches);
    B. The 'ProcessorSignature' & 'ProcessorFlags' fields in the header match
       at least one processor within system;
    C. If the Extended Signature Table exists in a microcode patch, the
       'ProcessorSignature' & 'ProcessorFlag' fields in the table entries
       match at least one processor within system.
    
    Criterion B and C will require all the processors to be woken up once to
    collect their CPUID and Platform ID information. Hence, this commit will
    move the copy, detect and apply of microcode patch on BSP and APs after
    all the processors have been woken up.
    
    Cc: Eric Dong <eric.dong@xxxxxxxxx>
    Cc: Ray Ni <ray.ni@xxxxxxxxx>
    Cc: Laszlo Ersek <lersek@xxxxxxxxxx>
    Cc: Star Zeng <star.zeng@xxxxxxxxx>
    Cc: Siyuan Fu <siyuan.fu@xxxxxxxxx>
    Cc: Michael D Kinney <michael.d.kinney@xxxxxxxxx>
    Signed-off-by: Hao A Wu <hao.a.wu@xxxxxxxxx>
    Reviewed-by: Eric Dong <eric.dong@xxxxxxxxx>
    Reviewed-by: Ray Ni <ray.ni@xxxxxxxxx>

commit 999463c865d3768a8432a89508096ae6a43873a5
Author: Hao A Wu <hao.a.wu@xxxxxxxxx>
Date:   Thu Dec 19 13:36:24 2019 +0800

    UefiCpuPkg/MpInitLib: Collect processors' CPUID & Platform ID info
    
    REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2429
    
    This commit will collect the CPUID and Platform ID information for each
    processor within system. They will be stored in the CPU_AP_DATA structure.
    
    These information will be used in the next commit to decide whether a
    microcode patch will be loaded into memory.
    
    Cc: Eric Dong <eric.dong@xxxxxxxxx>
    Cc: Ray Ni <ray.ni@xxxxxxxxx>
    Cc: Laszlo Ersek <lersek@xxxxxxxxxx>
    Cc: Star Zeng <star.zeng@xxxxxxxxx>
    Cc: Siyuan Fu <siyuan.fu@xxxxxxxxx>
    Cc: Michael D Kinney <michael.d.kinney@xxxxxxxxx>
    Signed-off-by: Hao A Wu <hao.a.wu@xxxxxxxxx>
    Reviewed-by: Ray Ni <ray.ni@xxxxxxxxx>
    Reviewed-by: Eric Dong <eric.dong@xxxxxxxxx>

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel