|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-devel] [ovmf baseline-only test] 44430: all pass
This run is configured for baseline tests only. flight 44430 ovmf real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/44430/ Perfect :-) All tests in this flight passed version targeted for testing: ovmf 720eea6aa80b48acb05c1adc0f835e849d71da97 baseline version: ovmf b41ef32518095f783d0c2343b496cc857c6f3dff Last test of basis 44427 2016-05-18 06:20:27 Z 0 days Testing same since 44430 2016-05-18 20:20:23 Z 0 days 1 attempts ------------------------------------------------------------ People who touched revisions under test: Dong, Eric <eric.dong@xxxxxxxxx> Eric Dong <eric.dong@xxxxxxxxx> Gabriel Somlo <somlo@xxxxxxx> Laszlo Ersek <lersek@xxxxxxxxxx> Pedroa Liu <pedroa.liu@xxxxxxxxxx> Yonghong Zhu <yonghong.zhu@xxxxxxxxx> Zenith432 <zenith432@xxxxxxxxxxxxxxxxxxxxx> 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 720eea6aa80b48acb05c1adc0f835e849d71da97 Author: Eric Dong <eric.dong@xxxxxxxxx> Date: Tue May 17 14:00:06 2016 +0800 BootMaintenanceManagerUiLib: Rollback changes for BootNext. Commit a85be3ae48a8aaa40b755cd0ff7270c67cfed585 imports errors for BootNext question, this patch rollback the related changes. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Eric Dong <eric.dong@xxxxxxxxx> Reviewed-by: Liming Gao <liming.gao@xxxxxxxxx> commit d3bb711834acd3eda35a07d0be7911bc3dbb9e6f Author: Zenith432 <zenith432@xxxxxxxxxxxxxxxxxxxxx> Date: Mon May 16 23:52:21 2016 +0800 BaseTools: Eliminate two shift-negative-value in FvLib.c clang 3.8 flags -Wshift-negative-value warning, which turns fatal due to use of -Werror. Fixes: https://github.com/tianocore/edk2/issues/49 Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Zenith432 <zenith432@xxxxxxxxxxxxxxxxxxxxx> Reviewed-by: Liming Gao <liming.gao@xxxxxxxxx> commit b98993580e3c07cfa139ed19b6fb4f1bb4db9edc Author: Zenith432 <zenith432@xxxxxxxxxxxxxxxxxxxxx> Date: Mon May 16 23:50:06 2016 +0800 MdePkg: Reinitialize twice-iterated VA_LIST in variadic function UefiDevicePathLibCatPrint() Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Zenith432 <zenith432@xxxxxxxxxxxxxxxxxxxxx> Reviewed-by: Liming Gao <liming.gao@xxxxxxxxx> commit 415700ec3eb5f6414e6278bbf338d142052b3138 Author: Zenith432 <zenith432@xxxxxxxxxxxxxxxxxxxxx> Date: Mon May 16 23:49:12 2016 +0800 MdeModulePkg: Terminate two unterminated VA_COPYs in CheckRemainingSpaceForConsistencyInternal() Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Zenith432 <zenith432@xxxxxxxxxxxxxxxxxxxxx> Reviewed-by: Liming Gao <liming.gao@xxxxxxxxx> commit 7266e2f69b8bfa47c1c7203ad51dabf6c4cb78c9 Author: Dong, Eric <eric.dong@xxxxxxxxx> Date: Tue May 17 10:12:12 2016 +0800 MdeModulePkg UiApp: Add "Enter Setup" status code. The original BdsDxe driver has "Enter Setup" status code while current code not. This patch restores it. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Eric Dong <eric.dong@xxxxxxxxx> Reviewed-by: Liming Gao <liming.gao@xxxxxxxxx> commit b8f3601daae5c8b50ca6f7da74bb150f2eef9453 Author: Pedroa Liu <pedroa.liu@xxxxxxxxxx> Date: Mon May 16 20:48:41 2016 +0800 ShellPkg: Fix the incorrect behavior when pressing 'shift' key. If 'ReadKeyStroke' function return EFI_NOT_READY then skip it. If the return value is EFI_DEVICE_ERROR clean the currentString buffer. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Pedroa Liu <pedroa.liu@xxxxxxxxxx> Reviewed-by: Qiu Shumin <shumin.qiu@xxxxxxxxx> commit c28d2e1047816164ffec552e4a3375122cbcc6b6 Author: Yonghong Zhu <yonghong.zhu@xxxxxxxxx> Date: Tue May 10 17:58:26 2016 +0800 BaseTools: support private package definition EDKII build spec and DEC spec updated to support private package definition. If GUID, Protocol or PPI is listed in a DEC file, where the Private modifier is used in the section tag ([Guids.common.Private] for example), only modules within the package are permitted to use the GUID, Protocol or PPI. If a module or library instance outside of the package attempts to use the item, the build must fail with an appropriate error message. 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 bba734ab4c7c9b4386d39420983bf61484f65dda Author: Laszlo Ersek <lersek@xxxxxxxxxx> Date: Mon May 9 22:54:36 2016 +0200 OvmfPkg/PlatformPei: provide 10 * 4KB of PCI IO Port space on Q35 This can accommodate 10 bridges (including root bridges, PCIe upstream and downstream ports, etc -- see <https://bugzilla.redhat.com/show_bug.cgi?id=1333238#c12> for more details). 10 is not a whole lot, but closer to the architectural limit of 15 than our current 4, so it can be considered a stop-gap solution until all guests manage to migrate to virtio-1.0, and no longer need PCI IO BARs behind PCIe downstream ports. Cc: Gabriel Somlo <somlo@xxxxxxx> Cc: Jordan Justen <jordan.l.justen@xxxxxxxxx> Ref: https://bugzilla.redhat.com/show_bug.cgi?id=1333238 Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Laszlo Ersek <lersek@xxxxxxxxxx> Reviewed-by: Jordan Justen <jordan.l.justen@xxxxxxxxx> Tested-by: Gabriel Somlo <somlo@xxxxxxx> commit c4df7fd01ff46cce6baa0afcc0f32f60bd2b5ace Author: Laszlo Ersek <lersek@xxxxxxxxxx> Date: Mon May 9 22:39:44 2016 +0200 OvmfPkg/PlatformPei: set PCI IO port aperture dynamically Make PcdPciIoBase and PcdPciIoSize dynamic PCDs, and set them in MemMapInitialization(), where we produce our EFI_RESOURCE_IO descriptor HOB. (The PCD is consumed by the core PciHostBridgeDxe driver, through our PciHostBridgeLib instance.) Take special care to keep the GCD IO space map unchanged on all platforms OVMF runs on. Cc: Gabriel Somlo <somlo@xxxxxxx> Cc: Jordan Justen <jordan.l.justen@xxxxxxxxx> Ref: https://bugzilla.redhat.com/show_bug.cgi?id=1333238 Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Laszlo Ersek <lersek@xxxxxxxxxx> Reviewed-by: Jordan Justen <jordan.l.justen@xxxxxxxxx> Tested-by: Gabriel Somlo <somlo@xxxxxxx> commit 1466b76f938501269d1c95f068b6d4938acec22d Author: Laszlo Ersek <lersek@xxxxxxxxxx> Date: Mon May 9 21:04:47 2016 +0200 OvmfPkg: determine PMBA value dependent on host bridge device ID In this patch, the AcpiTimerLib instances, ResetSystemLib, and PlatformPei are modified together in order to keep VMs functional across a bisection: they all must agree on the PMBA value used. ResetSystemLib must not use dynamic PCDs. With SOURCE_DEBUG_ENABLE, it gets linked into the debug agent, therefore the same restrictions apply to it as to BaseRomAcpiTimerLib. Luckily, AcpiPmControl() is only used for powering off the virtual machine, thus the extra cost of a PCI config space read, compared to a PcdGet16(), should be negligible. This is the patch that moves the PMBA to IO port 0x0600 on Q35 in practice. The ResetSystemLib change is easiest to verify with the "reset -s" command in the UEFI shell (which goes through gRT->ResetSystem() and, in OVMF, PcAtChipsetPkg/KbcResetDxe). Cc: Gabriel Somlo <somlo@xxxxxxx> Cc: Jordan Justen <jordan.l.justen@xxxxxxxxx> Ref: https://bugzilla.redhat.com/show_bug.cgi?id=1333238 Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Laszlo Ersek <lersek@xxxxxxxxxx> Reviewed-by: Jordan Justen <jordan.l.justen@xxxxxxxxx> Tested-by: Gabriel Somlo <somlo@xxxxxxx> commit b97af6d42bfaba836a685a0df75e0304170d7a87 Author: Laszlo Ersek <lersek@xxxxxxxxxx> Date: Mon May 9 19:46:00 2016 +0200 OvmfPkg: introduce ICH9_PMBASE_VALUE According to the ICH9 spec, PMBASE "provides 128 bytes of I/O space for ACPI, GPIO, and TCO logic. This is placed on a 128-byte boundary". On the Q35 machine type of QEMU, our current PMBASE setting of 0xB000 is the only thing that prevents us from lowering the base of the PCI IO port aperture from 0xC000. (The base must be aligned to 0x1000 due to PCI bridge requirements.) By moving our PMBASE to 0x0600 (moving the register block to 0x0600..0x067F inclusive), which is also what SeaBIOS uses on Q35, we will be able to lower the PCI IO port aperture base to 0x6000 (the next IO port under it being taken by the "vmport" device, at fixed 0x5658), while steering clear of other QEMU devices. On PIIX4, freeing up the 0x1000 IO ports at 0xB000 wouldn't help much, because the 0xA000 block right below it is occupied by unmovable devices (see <https://bugzilla.redhat.com/show_bug.cgi?id=1333238#c19> for details). Doing this for Q35 only has two more benefits: - It won't interfere with Xen guests, - The Q35 machine type with the smallest version number is "pc-q35-2.4", which is guaranteed to have an ACPI generator. This matters because the ACPI tables (FACP, DSDT) have to reflect the PM base address that we program. Cc: Gabriel Somlo <somlo@xxxxxxx> Cc: Jordan Justen <jordan.l.justen@xxxxxxxxx> Ref: https://bugzilla.redhat.com/show_bug.cgi?id=1333238 Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Laszlo Ersek <lersek@xxxxxxxxxx> Reviewed-by: Jordan Justen <jordan.l.justen@xxxxxxxxx> Tested-by: Gabriel Somlo <somlo@xxxxxxx> commit 07d3ba07709d8aa417f5f887b84ec78a908c6d2a Author: Laszlo Ersek <lersek@xxxxxxxxxx> Date: Mon May 9 20:05:18 2016 +0200 OvmfPkg: add and use industry standard macro PIIX4_PMBA_MASK We already have the identical purpose (but different value) macro for ICH9, namely ICH9_PMBASE_MASK in "OvmfPkg/Include/IndustryStandard/Q35MchIch9.h". Also, stop bit-negating signed integer constants. Cc: Gabriel Somlo <somlo@xxxxxxx> Cc: Jordan Justen <jordan.l.justen@xxxxxxxxx> Ref: https://bugzilla.redhat.com/show_bug.cgi?id=1333238 Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Laszlo Ersek <lersek@xxxxxxxxxx> Reviewed-by: Jordan Justen <jordan.l.justen@xxxxxxxxx> Tested-by: Gabriel Somlo <somlo@xxxxxxx> commit b2f4da3956debf656279a0836500b6989a85e276 Author: Laszlo Ersek <lersek@xxxxxxxxxx> Date: Mon May 9 19:26:37 2016 +0200 OvmfPkg: replace PcdAcpiPmBaseAddress with PIIX4_PMBA_VALUE In the next patches, we'll differentiate the PMBA IO port address that we program on PIIX4 vs. Q35. Normally we'd just turn PcdAcpiPmBaseAddress into a dynamic PCD. However, because we need this value in BaseRomAcpiTimerLib too (which cannot access RAM and dynamic PCDs), it must remain a build time constant. We will introduce its Q35 counterpart later. As first step, replace the PCD with a new macro in "OvmfPlatforms.h"; Jordan prefers the latter to fixed PCDs in this instance. Cc: Gabriel Somlo <somlo@xxxxxxx> Cc: Jordan Justen <jordan.l.justen@xxxxxxxxx> Ref: https://bugzilla.redhat.com/show_bug.cgi?id=1333238 Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Laszlo Ersek <lersek@xxxxxxxxxx> Reviewed-by: Jordan Justen <jordan.l.justen@xxxxxxxxx> Tested-by: Gabriel Somlo <somlo@xxxxxxx> commit ac759060e6272c3e451808c9ba40927d76890f26 Author: Laszlo Ersek <lersek@xxxxxxxxxx> Date: Tue May 17 16:08:23 2016 +0200 OvmfPkg/AcpiTimerLib: don't use possibly unset PMBA register (PEI phase) We should store the right value to the PMBA (if the PMBA needs initialization) before setting mAcpiTimerIoAddr from the PMBA. Cc: Gabriel Somlo <somlo@xxxxxxx> Cc: Jordan Justen <jordan.l.justen@xxxxxxxxx> Fixes: f122712b42af3d63902c2c141e90d2c200df6790 Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Laszlo Ersek <lersek@xxxxxxxxxx> Reviewed-by: Jordan Justen <jordan.l.justen@xxxxxxxxx> Tested-by: Gabriel Somlo <somlo@xxxxxxx> _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
 |
Lists.xenproject.org is hosted with RackSpace, monitoring our |