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

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

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

Regressions :-(

Tests which did not succeed and are blocking,
including tests which could not be run:
 build-i386-libvirt            6 libvirt-build            fail REGR. vs. 124920
 test-amd64-i386-xl-qemuu-ovmf-amd64 16 guest-localmigrate/x10 fail REGR. vs. 
124920

version targeted for testing:
 ovmf                 f88290964fe528ffeb67ff108e8174fc2ce52741
baseline version:
 ovmf                 aa4240edff41034d709938a15b42cf4fd3214386

Last test of basis   124920  2018-07-02 15:41:35 Z    2 days
Failing since        124944  2018-07-03 19:02:15 Z    1 days    2 attempts
Testing same since   124974  2018-07-04 16:22:54 Z    0 days    1 attempts

------------------------------------------------------------
People who touched revisions under test:
  Brijesh Singh <brijesh.singh@xxxxxxx>
  Chris Co <Christopher.Co@xxxxxxxxxxxxx>
  Christopher Co <christopher.co@xxxxxxxxxxxxx>
  Dandan Bi <dandan.bi@xxxxxxxxx>
  Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
  Star Zeng <star.zeng@xxxxxxxxx>

jobs:
 build-amd64-xsm                                              pass    
 build-i386-xsm                                               pass    
 build-amd64                                                  pass    
 build-i386                                                   pass    
 build-amd64-libvirt                                          pass    
 build-i386-libvirt                                           fail    
 build-amd64-pvops                                            pass    
 build-i386-pvops                                             pass    
 test-amd64-amd64-xl-qemuu-ovmf-amd64                         pass    
 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 f88290964fe528ffeb67ff108e8174fc2ce52741
Author: Brijesh Singh <brijesh.singh@xxxxxxx>
Date:   Wed Jul 4 10:02:16 2018 +0800

    MdeModulePkg/Variable: Check EFI_MEMORY_RUNTIME attribute before setting it
    
    Set the EFI_MEMORY_RUNTIME attribute in FtwNotificationEvent() only if
    the attribute is not already present. This will ensure that the attributes
    set by the platform drivers (e.g Ovmf pflash) is not lost.
    
    Cc: Dong Eric <eric.dong@xxxxxxxxx>
    Cc: Justen Jordan L <jordan.l.justen@xxxxxxxxx>
    Cc: Zeng Star <star.zeng@xxxxxxxxx>
    Cc: Laszlo Ersek <lersek@xxxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Reviewed-by: Star Zeng <star.zeng@xxxxxxxxx>
    Signed-off-by: Brijesh Singh <brijesh.singh@xxxxxxx>

commit dd3907f221fd9b774a5734cb7a000377acdfd5ca
Author: Dandan Bi <dandan.bi@xxxxxxxxx>
Date:   Mon Jul 2 13:29:52 2018 +0800

    MdeModulePkg/SmmCorePerformanceLib: Update mPerformanceLength
    
    In mSmmBootPerformanceTable there are two parts,
    one is the FPDT table header and the other is FPDT records.
    Currently:
    mPerformanceLength: The length of existing FPDT records.
    mMaxPerformanceLength: The maximum length allocated for
    mSmmBootPerformanceTable(including FPDT table header length
    and existing FPDT records length)
    But when compare mPerformanceLength with mMaxPerformanceLength
    to check whether need to allocate new buffer for new FPDT
    record, we miss to add mPerformanceLength with header length,
    which will cause pool allocation behavior is not correct.
    
    Now update the mPerformanceLength to fix this issue:
    updated mPerformanceLength = FPDT table header length
    + existing FPDT records length.
    
    Cc: Liming Gao <liming.gao@xxxxxxxxx>
    Cc: Star Zeng <star.zeng@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Dandan Bi <dandan.bi@xxxxxxxxx>
    Reviewed-by: Liming Gao <liming.gao@xxxxxxxxx>

commit 5895956dd955714c0e578a413d0e289153cc9eea
Author: Chris Co <Christopher.Co@xxxxxxxxxxxxx>
Date:   Fri Jun 29 03:31:21 2018 +0800

    BaseTools/Trim: Normalize filepaths to fix comparisons on Windows
    
    When using Linaro GCC5+ arm-eabi toolchain on Windows, the generated
    DSDT.iii contains a canonicalized ("\.\" removed and lower case)
    filepath for the preprocessed DSDT.i file in the first line.
    Trim.exe is called on DSDT.iii to generate DSDT.iiii, which does a
    line for line comparison of filepaths encountered to the preprocessed
    DSDT.i filepath found in the first line to determine what lines to
    place in DSDT.iiii. Since the DSDT.i filepath is canonicalized and
    all later filepaths in DSDT.iii are not canonicalized, all comparisons
    fail and the result is in an empty DSDT.iiii.
    
    Issue was first reported to Linaro here:
    https://bugs.linaro.org/show_bug.cgi?id=2909
    where the recommendation was to address the issue in Trim.exe.
    
    This patch normalizes the case and pathname of all filepaths
    encountered during Trim.exe execution on preprocessed files.  This
    fixes comparisons of filepaths that contain mismatching case on
    case-insensitive filesystems, redundant separators, and uplevel
    references.
    
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Christopher Co <christopher.co@xxxxxxxxxxxxx>
    Cc: Leif Lindholm <leif.lindholm@xxxxxxxxxx>
    Cc: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
    Cc: Liming Gao <liming.gao@xxxxxxxxx>
    Reviewed-by: Liming Gao <liming.gao@xxxxxxxxx>

commit cd69c8738391d9ce6c09cf5118c913cc7cd41d8b
Author: Star Zeng <star.zeng@xxxxxxxxx>
Date:   Mon Jul 2 17:07:54 2018 +0800

    MdeModulePkg SataControllerDxe: Use compare logic in if condition
    
    Use compare logic in if condition to fix ECC issue.
    It is caused by aa4240edff41034d709938a15b42cf4fd3214386.
    
    Cc: Hao Wu <hao.a.wu@xxxxxxxxx>
    Cc: Dandan Bi <dandan.bi@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Star Zeng <star.zeng@xxxxxxxxx>
    Reviewed-by: Dandan Bi <dandan.bi@xxxxxxxxx>
    Reviewed-by: Hao Wu <hao.a.wu@xxxxxxxxx>

commit 5f46352300f64d79c43cbfc67bb7a5f94bb982b8
Author: Star Zeng <star.zeng@xxxxxxxxx>
Date:   Mon Jul 2 16:54:23 2018 +0800

    MdeModulePkg Variable: Add/Correct GetHobVariableStore para description
    
    It is caused by 09808bd39b0c76559354253639766458ec24da79.
    
    Cc: Liming Gao <liming.gao@xxxxxxxxx>
    Cc: Dandan Bi <dandan.bi@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Star Zeng <star.zeng@xxxxxxxxx>
    Reviewed-by: Liming Gao <liming.gao@xxxxxxxxx>
    Reviewed-by: Dandan Bi <dandan.bi@xxxxxxxxx>

commit 9c227e07dde56c5f2ab9d00fd6740827f28dc875
Author: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
Date:   Mon Jul 2 11:19:26 2018 +0800

    MdeModulePkg/ResetSystemRuntimeDxe: Add more info in debug message
    
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Reviewed-by: Star Zeng <star.zeng@xxxxxxxxx>

commit c0bf47682d6bcecc3889ff849c81cc06a22debc4
Author: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
Date:   Mon Jul 2 11:16:22 2018 +0800

    MdeModulePkg/[Pei|Dxe]ResetSystemLib: Add PPI/Protocol dependency
    
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Reviewed-by: Star Zeng <star.zeng@xxxxxxxxx>

commit 93f5a54ff3386a44b08de22e9b9230e3ed92b90c
Author: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
Date:   Mon Jul 2 11:13:17 2018 +0800

    MdeModulePkg/ResetUtilityLib: Add pack(1) for reset data structure
    
    RESET_UTILITY_GUID_SPECIFIC_RESET_DATA structure should be declared
    as pack(1).
    The patch adds the missing pack(1) pragma.
    
    (GUID *)((UINT8 *)&ResetData +
    OFFSET_OF (RESET_UTILITY_GUID_SPECIFIC_RESET_DATA, ResetSubtype))
    is used to replace &ResetData.ResetSubType to resolve C4366 VS
    compiler warning.
    
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Reviewed-by: Star Zeng <star.zeng@xxxxxxxxx>

commit 51fe5b5140ba9ecb168fb03da328983355880c7a
Author: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
Date:   Mon Jun 25 15:35:06 2018 +0800

    MdeModulePkg/BmpSupportLib: Check PixelHeight/PixelWidth against 0
    
    The patch adds check logic to make sure that for a input BMP file,
    the width or height is not 0; for a input GOP blt buffer, the width
    or height is not 0. Otherwise, UNSUPPORTED status is returned.
    
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Reviewed-by: Star Zeng <star.zeng@xxxxxxxxx>
    Cc: Michael D Kinney <michael.d.kinney@xxxxxxxxx>

commit 2a9e1b97c9a86a736707388fe5e22cb75c378b35
Author: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
Date:   Mon Jun 25 14:54:33 2018 +0800

    MdeModulePkg/BmpSupportLib: Correct debug message
    
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Cc: Michael D Kinney <michael.d.kinney@xxxxxxxxx>
    Reviewed-by: Star Zeng <star.zeng@xxxxxxxxx>

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