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

[ovmf baseline-only test] 68094: all pass



This run is configured for baseline tests only.

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

Perfect :-)
All tests in this flight passed as required
version targeted for testing:
 ovmf                 26b85012802ed8a2ff3db96d102121323aabcc0c
baseline version:
 ovmf                 0265811dbe56cf46fb3c152b2ccdefd8fb47a170

Last test of basis    68090  2016-11-24 05:49:22 Z    0 days
Testing same since    68094  2016-11-24 21:51:01 Z    0 days    1 attempts

------------------------------------------------------------
People who touched revisions under test:
  Baraneedharan Anbazhagan <anbazhagan@xxxxxx>
  Eric Dong <eric.dong@xxxxxxxxx>
  Feng Tian <feng.tian@xxxxxxxxx>
  Jiaxin Wu <jiaxin.wu@xxxxxxxxx>
  Jiewen Yao <jiewen.yao@xxxxxxxxx>
  Laszlo Ersek <lersek@xxxxxxxxxx>
  Pete Batard <pete@xxxxxxx>
  Star Zeng <star.zeng@xxxxxxxxx>
  Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
  Zhang Lubo <lubo.zhang@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 26b85012802ed8a2ff3db96d102121323aabcc0c
Author: Feng Tian <feng.tian@xxxxxxxxx>
Date:   Wed Nov 23 09:46:32 2016 +0800

    MdeModulePkg/Xhci: Add 10ms delay before sending SendAddr cmd to dev
    
    We send ADDRESS DEVICE CMD in XhcInitializeDeviceSlot(), which will
    cause XHC issue a USB SET_ADDRESS request to the USB Device.
    
    According to USB spec, there should have a 10ms delay before this
    operation after resetting a given port.
    
    But in original code, there is a possible path which may have no such
    10ms delay:
    UsbHubResetPort()->UsbHubSetPortFeature()->Stall(20)->UsbHubGetPortSt
    atus()->XhcPollPortStatusChange()->(if RESET_C bit is set)->
    XhcInitializeDeviceSlot()->(if RESET_C bit is set)->Stall(10)
    
    So this patch is used to fix above issue.
    
    Cc: Star Zeng <star.zeng@xxxxxxxxx>
    Cc: Baraneedharan Anbazhagan <anbazhagan@xxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: Feng Tian <feng.tian@xxxxxxxxx>
    Tested-by: Baraneedharan Anbazhagan <anbazhagan@xxxxxx>
    Reviewed-by: Star Zeng <star.zeng@xxxxxxxxx>

commit 7fa1376c5c97d18eac719f35e5e85356d3c1b033
Author: Jiewen Yao <jiewen.yao@xxxxxxxxx>
Date:   Tue Nov 22 15:05:11 2016 +0800

    UefiCpuPkg/PiSmmCpu: Correct exception message.
    
    This patch fixes the first part of
    https://bugzilla.tianocore.org/show_bug.cgi?id=242
    
    Previously, when SMM exception happens, "stack overflow" is misreported.
    This patch checked the PF address to see it is stack overflow, or
    it is caused by SMM page protection.
    
    It dumps exception data, PF address and the module trigger the issue.
    
    Cc: Laszlo Ersek <lersek@xxxxxxxxxx>
    Cc: Jeff Fan <jeff.fan@xxxxxxxxx>
    Cc: Michael D Kinney <michael.d.kinney@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: Jiewen Yao <jiewen.yao@xxxxxxxxx>
    Tested-by: Laszlo Ersek <lersek@xxxxxxxxxx>
    Reviewed-by: Laszlo Ersek <lersek@xxxxxxxxxx>
    Reviewed-by: Jeff Fan <jeff.fan@xxxxxxxxx>

commit 1cbd8330898964a465ebb91421b951595398d56b
Author: Laszlo Ersek <lersek@xxxxxxxxxx>
Date:   Tue Nov 22 13:58:54 2016 +0100

    UefiCpuPkg/MpInitLib: fix feature test for Extended Topology CPUID leaf
    
    According to the Intel SDM (325462-060US / September 2016),
    
    > INPUT EAX = 0BH: Returns Extended Topology Information
    >
    > [...] Software must detect the presence of CPUID leaf 0BH by verifying
    > (a) the highest leaf index supported by CPUID is >= 0BH, and
    > (b) CPUID.0BH:EBX[15:0] reports a non-zero value. [...]
    
    The "GetApicId" sections in the Ia32 and X64 "MpFuncs.nasm" files do not
    perform check (b).
    
    This causes an actual bug in the following OVMF setup:
    
    - Intel W3550 host processor <http://ark.intel.com/products/39720/>,
    
    - the QEMU/KVM guest's VCPU model is set to "host", that is, "the CPU
      visible to the guest should be exactly the same as the host CPU".
    
    Under "GetApicId", check (a) passes: the CPUID level of the W3550 is
    exactly 11 decimal. However, leaf 11 itself is not supported, therefore
    EDX is set to zero:
    
    > If a value entered for CPUID.EAX is less than or equal to the maximum
    > input value and the leaf is not supported on that processor then 0 is
    > returned in all the registers.
    
    Because we don't check (b), the "GetProcessorNumber" section of the code
    is reached with an initial APIC ID of 0 in EDX on all of the APs. Given
    that "GetProcessorNumber" searches the
    "MP_CPU_EXCHANGE_INFO.CpuInfo[*].InitialApicId" fields for a match, all
    APs enter ApWakeupFunction() with an identical "NumApsExecuting"
    parameter. This results in unpredictable guest behavior (crashes, reboots,
    hangs etc).
    
    Reorganize the "GetApicId" section and add the missing check in both
    assembly files.
    
    Cc: Jeff Fan <jeff.fan@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: Laszlo Ersek <lersek@xxxxxxxxxx>
    Reviewed-by: Jeff Fan <jeff.fan@xxxxxxxxx>

commit 4af3ae1463796ef7b2ecf2bfe8b390e72817212c
Author: Laszlo Ersek <lersek@xxxxxxxxxx>
Date:   Tue Nov 22 12:43:17 2016 +0100

    UefiCpuPkg/LocalApicLib: fix feature test for Extended Topology CPUID leaf
    
    According to the Intel SDM (325462-060US / September 2016),
    
    > INPUT EAX = 0BH: Returns Extended Topology Information
    >
    > [...] Software must detect the presence of CPUID leaf 0BH by verifying
    > (a) the highest leaf index supported by CPUID is >= 0BH, and
    > (b) CPUID.0BH:EBX[15:0] reports a non-zero value. [...]
    
    The LocalApicLib instances in UefiCpuPkg do not perform check (b).
    
    This causes an actual bug in the following OVMF setup:
    
    - Intel W3550 host processor <http://ark.intel.com/products/39720/>,
    
    - the QEMU/KVM guest's VCPU model is set to "host", that is, "the CPU
      visible to the guest should be exactly the same as the host CPU".
    
    In the GetInitialApicId() function, check (a) passes: the CPUID level of
    the W3550 is exactly 11 decimal. However, leaf 11 itself is not supported,
    therefore EDX is set to zero:
    
    > If a value entered for CPUID.EAX is less than or equal to the maximum
    > input value and the leaf is not supported on that processor then 0 is
    > returned in all the registers.
    
    Because we don't check (b), we return 0 as initial APIC ID on the BSP and
    on all of the APs as well.
    
    Add the missing check.
    
    Cc: Jeff Fan <jeff.fan@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: Laszlo Ersek <lersek@xxxxxxxxxx>
    Reviewed-by: Jeff Fan <jeff.fan@xxxxxxxxx>

commit 7b9b576c71c71ed134f50497fd58f862109dd80b
Author: Star Zeng <star.zeng@xxxxxxxxx>
Date:   Fri Nov 18 15:34:54 2016 +0800

    SecurityPkg DxeTcg2PPLib: Lock Tcg2PhysicalPresenceFlags variable on S4
    
    The code updates Tcg2PhysicalPresenceLibProcessRequest() to also lock
    Tcg2PhysicalPresenceFlags variable on S4.
    
    Cc: Jiewen Yao <jiewen.yao@xxxxxxxxx>
    Cc: Chao Zhang <chao.b.zhang@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: Star Zeng <star.zeng@xxxxxxxxx>
    Reviewed-by: Chao Zhang <chao.b.zhang@xxxxxxxxx>

commit 252b891b2b49dddad5e3850146719d14c22f1c60
Author: Eric Dong <eric.dong@xxxxxxxxx>
Date:   Wed Nov 23 16:26:36 2016 +0800

    Revert old "Enable BlockSid related PP actions" patch series.
    
    New solution for this issue will be provided.
    
    This reverts commits from d1947ce509d745f32db6b7fecc03dc9c778b9350
    to bda034c34deea6eb43edcef28018a9ace8f04637.
    
    Reviewed-by: Jiewen Yao <jiewen.yao@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: Eric Dong <eric.dong@xxxxxxxxx>

commit 67bb14259baaf2237b81bcbf2568517b3310b294
Author: Jiaxin Wu <jiaxin.wu@xxxxxxxxx>
Date:   Fri Nov 18 15:38:32 2016 +0800

    MdeModulePkg/DxeNetLib: Allow the IPv4/prefix case when AsciiStrToIp4
    
    This patch is used to allow the IPv4 with prefix case.
    
    Cc: Fu Siyuan <siyuan.fu@xxxxxxxxx>
    Cc: Ye Ting <ting.ye@xxxxxxxxx>
    Cc: Zhang Lubo <lubo.zhang@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: Jiaxin Wu <jiaxin.wu@xxxxxxxxx>
    Reviewed-by: Fu Siyuan <siyuan.fu@xxxxxxxxx>

commit 825f09b7621624b7ce68a629371ea64a26560cfb
Author: Zhang Lubo <lubo.zhang@xxxxxxxxx>
Date:   Tue Nov 8 17:24:12 2016 +0800

    ShellPkg: update ping6 to use timer service instead of timer arch protocol .
    
    This patch update the shell ping command to use timer service to calculate 
the
    RTT time, instead of using the timer arch protocol.
    
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: Zhang Lubo <lubo.zhang@xxxxxxxxx>
    Cc: Ni Ruiyu <ruiyu.ni@xxxxxxxxx>
    Cc: Ye Ting <ting.ye@xxxxxxxxx>
    Cc: Fu Siyuan <siyuan.fu@xxxxxxxxx>
    Reviewed-by: Fu Siyuan <siyuan.fu@xxxxxxxxx>

commit 24e7435ab474f8a2da43a086d63add4954a3034f
Author: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
Date:   Sat Nov 19 17:06:18 2016 +0800

    BaseTools: report error for same Guid's Private definition conflict
    
    Add error check for the same Guid/Protocol/PPIs/Includes defined as both
    Private and non-Private attribute.
    
    Fixes: https://bugzilla.tianocore.org/show_bug.cgi?id=209
    Cc: Liming Gao <liming.gao@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
    Reviewed-by: Liming Gao <liming.gao@xxxxxxxxx>

commit 3e2c7843022d849971a539387a4ad8b9862dadfb
Author: Pete Batard <pete@xxxxxxx>
Date:   Wed Nov 16 21:24:10 2016 +0800

    MdeModulePkg/EbcDxe: reorganize EBC header definitions
    
    VM related defs are now in EbcVmTest.h, and opocode related definitions in
    Ebc.h.
    Because it is used by both the EBC Debugger and driver,
    EbcDebugSignalException() sees its definition factorized in
    EbcDebuggerHook.h.
    
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: Pete Batard <pete@xxxxxxx>
    Reviewed-by: Jiewen Yao <jiewen.yao@xxxxxxxxx>

commit 748edcd5eb706b34ffd0af6900a8db19605b0ffd
Author: Pete Batard <pete@xxxxxxx>
Date:   Wed Nov 16 21:24:09 2016 +0800

    MdeModulePkg/EbcDxe: add EBC Debugger
    
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: Pete Batard <pete@xxxxxxx>
    Reviewed-by: Jiewen Yao <jiewen.yao@xxxxxxxxx>

commit 6f0a3cd23e4a2322c58c7dbf0ef8e66a4a01c42c
Author: Pete Batard <pete@xxxxxxx>
Date:   Wed Nov 16 21:24:08 2016 +0800

    MdeModulePkg/EbcDxe: prepare support for EBC Debugger
    
    * This patch introduces EbcDebuggerHook.c/h and inserts the required
      EBCDebugger references into the existing EBC source files.
    * With all the hooks defined to their empty version in EbcDebuggerHook.c
      the existing EBC VM behaviour is left unaffected.
    
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: Pete Batard <pete@xxxxxxx>
    Reviewed-by: Jiewen Yao <jiewen.yao@xxxxxxxxx>

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

 


Rackspace

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