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

[Xen-devel] [ovmf baseline-only test] 71992: all pass

This run is configured for baseline tests only.

flight 71992 ovmf real [real]

Perfect :-)
All tests in this flight passed as required
version targeted for testing:
 ovmf                 3a424c5f49239b810e08aa23368945a9f0360d4c
baseline version:
 ovmf                 d75b8ac278bc9f0159aa7eb9a92fd2cc87a18d8c

Last test of basis    71988  2017-08-17 14:49:14 Z    1 days
Testing same since    71992  2017-08-18 16:50:24 Z    0 days    1 attempts

People who touched revisions under test:
  Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx>
  Bi, Dandan <dandan.bi@xxxxxxxxx>
  Brijesh Singh <brijesh.singh@xxxxxxx>
  Dandan Bi <dandan.bi@xxxxxxxxx>
  Eric Dong <eric.dong@xxxxxxxxx>
  Hao Wu <hao.a.wu@xxxxxxxxx>
  Jiaxin Wu <jiaxin.wu@xxxxxxxxx>
  Laszlo Ersek <lersek@xxxxxxxxxx>
  Marvin H?user <Marvin.Haeuser@xxxxxxxxxxx>
  Marvin Haeuser <Marvin.Haeuser@xxxxxxxxxxx>
  Wu Jiaxin <jiaxin.wu@xxxxxxxxx>

 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

Test harness code can be found at

Push not applicable.

commit 3a424c5f49239b810e08aa23368945a9f0360d4c
Author: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx>
Date:   Thu Aug 17 13:16:58 2017 +0100

    ArmPkg/ArmDmaLib: use double buffering only for bus master write
    The ArmPkg implementation of DmaLib uses double buffering to ensure
    that any attempt to perform non-coherent DMA on unaligned buffers cannot
    corrupt adjacent unrelated data which happens to share cachelines with
    the data we are exchanging with the device.
    Such corruption can only occur on bus master write, in which case we have
    to invalidate the caches to ensure the CPU will see the data written to
    memory by the device. In the bus master read case, we can simply clean
    and invalidate at the same time, which may purge unrelated adjacent data
    from the caches, but will not corrupt its contents.
    Also, this double buffer does not necessarily have to be allocated from
    uncached memory: by the same reasoning, we can perform cache invalidation
    on an ordinary pool allocation as long as we take the same alignment
    constraints into account.
    So update our code accordingly: remove double buffering from the bus
    master read path, and switch to a pool allocation for the double buffer.
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx>
    Reviewed-by: Leif Lindholm <leif.lindholm@xxxxxxxxxx>

commit 5f0f5e90ae8c57f4a0702740d5aee95ce50fc684
Author: Laszlo Ersek <lersek@xxxxxxxxxx>
Date:   Wed Aug 16 12:45:59 2017 +0200

    ArmVirtPkg/FdtPL011SerialPortLib: call PL011UartLib in all SerialPortLib 
    With the SerialDxe change in commit 4cf3f37c87ba ("MdeModulePkg SerialDxe:
    Process timeout consistently in SerialRead", 2017-07-18), setting
    EFI_SERIAL_INPUT_BUFFER_EMPTY in the "Control" output parameter, in the
    GetControl() SerialPortLib function, is no longer a "small optimization".
    Namely, due to the SerialDxe change, the GetOneKeyFromSerial() call in
    TerminalDxe's TerminalConInTimerHandler() can take very long if the input
    queue is empty, even if GetOneKeyFromSerial()'s return value causes the
    loop to be exited right after, in the first iteration.
    This issue causes a boot hang in ArmVirtQemu: with the input queue empty,
    TerminalConInTimerHandler() takes so long to return that, by the time it
    returns, there's another execution queued already (due to the associated
    timer event being signaled meanwhile). The boot process is stuck in the
    timer event handler.
    Therefore even the first GetOneKeyFromSerial() iteration must be prevented
    in TerminalConInTimerHandler() if the input queue is empty, and that
    requires implementing GetControl() for real.
    Implement the SetAttributes(), SetControl() and GetControl() APIs (of
    SerialPortExtLib origin) in FdtPL011SerialPortLib with calls to matching
    PL011UartLib functions. This follows the example of
    "ArmPlatformPkg/Library/PL011SerialPortLib" and also matches Star's
    original idea under [1].
    The patch can be considered a continuation of commit ad7f6bc2e116
    ("ArmVirtPkg: Use SerialDxe in MdeModulePkg instead of EmbeddedPkg",
    2015-11-26), based on the mailing list threads [1] [2] [3].
    Cc: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx>
    Cc: Brijesh Singh <brijesh.singh@xxxxxxx>
    Cc: Heyi Guo <guoheyi@xxxxxxxxxx>
    Cc: Star Zeng <star.zeng@xxxxxxxxx>
    Originally-suggested-by: Star Zeng <star.zeng@xxxxxxxxx>
    Reported-by: Brijesh Singh <brijesh.singh@xxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Laszlo Ersek <lersek@xxxxxxxxxx>
    Reviewed-by: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx>
    Tested-by: Brijesh Singh <brijesh.singh@xxxxxxx>
    Reviewed-by: Star Zeng <star.zeng@xxxxxxxxx>

commit ca56256d5e0b7e63325b049e90a6bd03f90e3598
Author: Laszlo Ersek <lersek@xxxxxxxxxx>
Date:   Tue Aug 15 18:09:57 2017 +0200

    OvmfPkg/build.sh: select the GCC49 toolchain settings for gcc-7.*
    When UefiCpuPkg/MpInitLib is built for X64 with gcc-7, using the DEBUG
    build target and the GCC5 toolchain settings, a C-language assignment is
    miscompiled such that the initial AP startup hangs in CpuMpPei (X64) or
    CpuDxe (Ia32X64). See <https://bugzilla.tianocore.org/show_bug.cgi?id=671>
    for a detailed analysis of the symptoms, and for mailing list links.
    This issue has been reported several times (one example is
    <https://bugzilla.tianocore.org/show_bug.cgi?id=657>). Until we (or the
    upstream gcc developers) figure out how to dissuade gcc-7 from the
    miscompilation, pick the GCC49 toolchain in "build.sh" for gcc-7.*.
    Cc: Alex Williamson <alex.williamson@xxxxxxxxxx>
    Cc: Jordan Justen <jordan.l.justen@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Laszlo Ersek <lersek@xxxxxxxxxx>
    Reviewed-by: Jordan Justen <jordan.l.justen@xxxxxxxxx>

commit 5843dff25d7e74e0ddbd9528cc94b6c9fd8181cc
Author: Hao Wu <hao.a.wu@xxxxxxxxx>
Date:   Wed Aug 16 14:13:44 2017 +0800

    Maintainers.txt: Update the maintainer for SourceLevelDebugPkg
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: Hao Wu <hao.a.wu@xxxxxxxxx>
    Reviewed-by: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>

commit d654bf852f86902816f0ecddff90aa772f71dd09
Author: Bi, Dandan <dandan.bi@xxxxxxxxx>
Date:   Thu Aug 17 09:07:44 2017 +0800

    IntelSiliconPkg/IntelVTdDxe: Update function comments
    In commit 4ad5f597153c7cb20a968236c2c7d6ff01994350, the parameters
    of some functions have been updated, but miss to update the comments
    accordingly. This patch is to update the function comments.
    Cc: Jiewen Yao <jiewen.yao@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: Dandan Bi <dandan.bi@xxxxxxxxx>
    Reviewed-by: Jiewen Yao <jiewen.yao@xxxxxxxxx>

commit 4e33ff75d9dccd3a59f70eeeaff9e3208426f63e
Author: Marvin H?user <Marvin.Haeuser@xxxxxxxxxxx>
Date:   Thu Jul 20 23:11:37 2017 -0700

    QuarkSocPkg/QNCSmmDispatcher: Fix use after free issue #2
    As part of commit 5f82e02, ActiveRecordInDb was introduced as a copy
    of RecordInDb as latter may be freed by the callback function. This
    commit replaces an access of RecordInDb after the callback function
    has been executed with an access to ActiveRecordInDb.
    Cc: Michael D Kinney <michael.d.kinney@xxxxxxxxx>
    Cc: Kelly Steele <kelly.steele@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Marvin Haeuser <Marvin.Haeuser@xxxxxxxxxxx>
    Reviewed-by: Michael D Kinney <michael.d.kinney@xxxxxxxxx>

commit 6619cf3b6a84397ceff5d856111a33f9cdfb0812
Author: Eric Dong <eric.dong@xxxxxxxxx>
Date:   Wed Aug 16 13:20:36 2017 +0800

    UefiCpuPkg RegisterCpuFeaturesLib: Fix buffer pointer error usage.
    Current code allocate buffer for the pointer which later get value
    from PCD database. but current code error use "=" for this case.
    Use AllocateCopyPool instead to fix it.
    V2 enhanced to directly use AllocateCopyPool to get the PCD value.
    V3 enhanced to avoid using local temp variable.
    V4 enhanced to keep the functions to get the pcd values.
    Cc: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Cc: Shao Ming <ming.shao@xxxxxxxxx>
    Cc: Kinney Michael D <michael.d.kinney@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Eric Dong <eric.dong@xxxxxxxxx>
    Reviewed-by: Kinney Michael D <michael.d.kinney@xxxxxxxxx>

commit aa0f2bf7cc879dc732164d1ea003b53e244c2ff4
Author: Jiaxin Wu <jiaxin.wu@xxxxxxxxx>
Date:   Wed Aug 16 14:16:44 2017 +0800

    NetworkPkg/Ip6Dxe: Fix the bug when checking the DataSize
    During setting the DnsServer, the DataSize check is incorrect.
    This patch is to fix the issue.
    Cc: Ye Ting <ting.ye@xxxxxxxxx>
    Cc: Fu Siyuan <siyuan.fu@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: Wu Jiaxin <jiaxin.wu@xxxxxxxxx>
    Reviewed-by: Ye Ting <ting.ye@xxxxxxxxx>
    Reviewed-by: Fu Siyuan <siyuan.fu@xxxxxxxxx>

Xen-devel mailing list



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