[ovmf baseline-only test] 74730: all pass

[Platform Team regression test user <citrix-osstest@xxxxxxxxxxxxxx>]

This run is configured for baseline tests only.

flight 74730 ovmf real [real]
http://osstest.xs.citrite.net/~osstest/testlogs/logs/74730/

Perfect :-)
All tests in this flight passed as required
version targeted for testing:
 ovmf                 3f34e36d04a8de4992a696f738643b5a11261469
baseline version:
 ovmf                 0edb7ec5ced0a28b93bf8c13b12f0a277c44dbbc

Last test of basis    74713  2018-05-12 06:51:48 Z    7 days
Testing same since    74730  2018-05-19 01:00:09 Z    1 days    1 attempts

------------------------------------------------------------
People who touched revisions under test:
  Ansen Huang <ansen.huang@xxxxxxx>
  Bi, Dandan <dandan.bi@xxxxxxxxx>
  Carsey, Jaben </o=Intel/ou=Americas01/cn=Workers/cn=Carsey, Jaben>
  cinnamon shia <cinnamon.shia@xxxxxxx>
  Dandan Bi <dandan.bi@xxxxxxxxx>
  Derek Lin <derek.lin2@xxxxxxx>
  Eric Dong <eric.dong@xxxxxxxxx>
  Jaben Carsey <jaben.carsey@xxxxxxxxx>
  Laszlo Ersek <lersek@xxxxxxxxxx>
  Lin, Derek <derek.lin2@xxxxxxx>
  Star Zeng <star.zeng@xxxxxxxxx>
  Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
  Yunhua Feng <yunhuax.feng@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                         pass    
 test-amd64-i386-xl-qemuu-ovmf-amd64                          pass    


------------------------------------------------------------
sg-report-flight on osstest.xs.citrite.net
logs: /home/osstest/logs
images: /home/osstest/images

Logs, config files, etc. are available at
    http://osstest.xs.citrite.net/~osstest/testlogs/logs

Test harness code can be found at
    http://xenbits.xensource.com/gitweb?p=osstest.git;a=summary


Push not applicable.

------------------------------------------------------------
commit 3f34e36d04a8de4992a696f738643b5a11261469
Author: Lin, Derek <derek.lin2@xxxxxxx>
Date:   Wed May 9 17:03:23 2018 +0800

    BaseTools: Fix --hash Package and Module hash value.
    
    The order of List enumeration is arbitrary.
    Need to be sorted while calculating Package/Module hash, otherwise it
    generate different hash value even nothing changes.
    
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Derek Lin <derek.lin2@xxxxxxx>
    Reviewed-by: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>

commit 63c76537c652eb5a84360ee043c5f7b63728a622
Author: Eric Dong <eric.dong@xxxxxxxxx>
Date:   Mon May 14 15:28:40 2018 +0800

    SecurityPkg/OpalPassword: Fix PSID revert no hint message.
    
    For no warning message when do the PSID revert action, the
    message in the popup dialog is not enough. The error use
    of NULL for CreatePopUp function caused this regression.
    This change fixed it.
    
    Passed Unit Test:
    1. Check PSID revert with/without warning message cases.
    
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Eric Dong <eric.dong@xxxxxxxxx>
    Reviewed-by: Hao Wu <hao.a.wu@xxxxxxxxx>

commit 5a1d521dee69854c4eba6381800f44115bf4bb6c
Author: Eric Dong <eric.dong@xxxxxxxxx>
Date:   Mon May 14 15:25:03 2018 +0800

    SecurityPkg/TcgStorageOpalLib: Fix GCC build failure.
    
    Function definition different with function implementation
    caused this build failure. Change code to make them
    consistent to pass the build.
    
    Done Unit Test:
    1. Pass GCC build.
    
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Eric Dong <eric.dong@xxxxxxxxx>
    Reviewed-by: Hao Wu <hao.a.wu@xxxxxxxxx>

commit f803c03cc2e0b6b0b0bed447a97ea2c61b04ed82
Author: Laszlo Ersek <lersek@xxxxxxxxxx>
Date:   Sun May 13 00:13:53 2018 +0200

    OvmfPkg/PlatformBootManagerLib: connect consoles unconditionally
    
    If both ConIn and ConOut exist, but ConIn references none of the PS/2
    keyboard, the USB wild-card keyboard, and any serial ports, then
    PlatformInitializeConsole() currently allows the boot to proceed without
    any input devices at all. This makes for a bad user experience -- the
    firmware menu could only be entered through OsIndications, set by a guest
    OS.
    
    Do what ArmVirtQemu does already, namely connect the consoles, and add
    them to ConIn / ConOut / ErrOut, unconditionally. (The underlying
    EfiBootManagerUpdateConsoleVariable() function checks for duplicates.)
    
    The issue used to be masked by the EfiBootManagerConnectAll() call that
    got conditionalized in commit 245c643cc8b7.
    
    This patch is best viewed with "git show -b -W".
    
    Cc: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx>
    Cc: Jordan Justen <jordan.l.justen@xxxxxxxxx>
    Fixes: 245c643cc8b73240c3b88cb55b2911b285a8c10d
    Ref: https://bugzilla.redhat.com/show_bug.cgi?id=1577546
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Laszlo Ersek <lersek@xxxxxxxxxx>
    Reviewed-by: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx>

commit 989f7a2cf0e27123fda5ca538b15832e115e0f4e
Author: cinnamon shia <cinnamon.shia@xxxxxxx>
Date:   Fri May 11 23:21:12 2018 +0800

    MdeModulePkg Variable: Fix the returned status in UpdateVariableStore
    
    If Fvb is a NULL, return EFI_UNSUPPORTED.
    If the remaining size is not enough, return EFI_OUT_OF_RESOURCES.
    
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: cinnamon shia <cinnamon.shia@xxxxxxx>
    Signed-off-by: Ansen Huang <ansen.huang@xxxxxxx>
    Reviewed-by: Star Zeng <star.zeng@xxxxxxxxx>

commit d741d1419350dc4ae868d4a9396f9da33a3416a0
Author: cinnamon shia <cinnamon.shia@xxxxxxx>
Date:   Fri May 11 23:21:11 2018 +0800

    MdeModulePkg Variable: Fix a corner case issue about setting a variable
    
    Fix the issue that failed to update or add a UEFI variable if the remaining 
size is equal to the data size
    of the variable.
    
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: cinnamon shia <cinnamon.shia@xxxxxxx>
    Signed-off-by: Ansen Huang <ansen.huang@xxxxxxx>
    Reviewed-by: Star Zeng <star.zeng@xxxxxxxxx>

commit c61db18e5d11e4c25e32bfb3f999a88e3207eb5f
Author: Lin, Derek <derek.lin2@xxxxxxx>
Date:   Wed May 9 17:03:24 2018 +0800

    BaseTools: Fix python error with --genfds-multi-thread.
    
    When self.Alignment is None, it ran into python error since there is no
    strip() in None.
    
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Derek Lin <derek.lin2@xxxxxxx>
    Reviewed-by: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>

commit c731b5450576a8dd0afb1d2dd013203235395a4f
Author: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
Date:   Wed May 9 16:41:28 2018 +0800

    BaseTools: Remove the redundant code
    
    the ArraySize and Array already be got in line 1093, so this code are
    redundant.
    
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
    Reviewed-by: Jaben Carsey <jaben.carsey@xxxxxxxxx>

commit 6b285ca3663d3c71cc68d01684a98f0d7537885e
Author: Yunhua Feng <yunhuax.feng@xxxxxxxxx>
Date:   Mon May 7 18:26:24 2018 +0800

    BaseTools: Fix generating array's size is incorrect in AutoGen.c
    
    case example:
    DSC:
     [PcdsFixedAtBuild]
      PcdToken.PcdName | "A"
     [Components]
     TestPkg/TestDriver.inf {
      PcdToken.PcdName | {0x41,0x42,0x43,0x44}
     }
    
    Generating the size of array is incorrect in AutoGen.c
     GLOBAL_REMOVE_IF_UNREFERENCED const UINT8
     _gPcd_FixedAtBuild_PcdName[2] = {0x41,0x42,0x43,0x44};
    
    Fixes: https://bugzilla.tianocore.org/show_bug.cgi?id=950
    Cc: Liming Gao <liming.gao@xxxxxxxxx>
    Cc: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Yunhua Feng <yunhuax.feng@xxxxxxxxx>
    Reviewed-by: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>

commit 1dc287c3a3f1f090ef6b8ff80f7b3b3725071471
Author: Carsey, Jaben </o=Intel/ou=Americas01/cn=Workers/cn=Carsey, Jaben>
Date:   Wed May 9 00:01:35 2018 +0800

    BaseTools: incorrect calculation for 16M
    
    the "0x" was missing.
    
    Cc: Liming Gao <liming.gao@xxxxxxxxx>
    Cc: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Jaben Carsey <jaben.carsey@xxxxxxxxx>
    Reviewed-by: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>

commit 13e3f8c03339ebc8cd25c454fca1abde098fe7ed
Author: Bi, Dandan <dandan.bi@xxxxxxxxx>
Date:   Wed May 9 13:02:11 2018 +0800

    BaseTools/VfrCompile: Avoid using uninitialized pointer
    
    V2:
    Add function _INIT_OPHDR_COND () for variable initialization.
    Make code logic more clean.
    
    Previously _CLEAR_SAVED_OPHDR () is used for variable
    initialization, and we updated it to clean memory.
    But _CLEAR_SAVED_OPHDR () is still called for variable
    initialization. This will cause uninitialized pointer
    will be checked to free and cause unexpected issue.
    
    This patch is to add new function for variable initialization
    and keep _CLEAR_SAVED_OPHDR () to clean memory which is
    aligned with its function name.
    
    Cc: Liming Gao <liming.gao@xxxxxxxxx>
    Cc: Gary Lin <glin@xxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Dandan Bi <dandan.bi@xxxxxxxxx>
    Reviewed-by: Eric Dong <eric.dong@xxxxxxxxx>

commit e91797885aee58ae65d7935332e580dc8517e8f6
Author: Star Zeng <star.zeng@xxxxxxxxx>
Date:   Wed Mar 28 16:52:12 2018 +0800

    IntelSiliconPkg MicrocodeUpdateDxe: Honor FIT table
    
    It is the second step for
    https://bugzilla.tianocore.org/show_bug.cgi?id=540.
    
    V2: Use error handling instead of ASSERT for FIT table checking result.
    
    Cc: Jiewen Yao <jiewen.yao@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Star Zeng <star.zeng@xxxxxxxxx>
    Reviewed-by: Jiewen Yao <jiewen.yao@xxxxxxxxx>

_______________________________________________
osstest-output mailing list
osstest-output@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/osstest-output