|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-devel] [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> _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx https://lists.xen.org/xen-devel
|
 |
Lists.xenproject.org is hosted with RackSpace, monitoring our |