|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-devel] [ovmf test] 125032: regressions - FAIL
flight 125032 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/125032/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-i386-libvirt 6 libvirt-build fail REGR. vs. 124920 version targeted for testing: ovmf 75b7aa9528bdd05a7ecf4e64a6beb478d31b402c baseline version: ovmf aa4240edff41034d709938a15b42cf4fd3214386 Last test of basis 124920 2018-07-02 15:41:35 Z 4 days Failing since 124944 2018-07-03 19:02:15 Z 3 days 5 attempts Testing same since 125032 2018-07-07 00:11:08 Z 0 days 1 attempts ------------------------------------------------------------ People who touched revisions under test: Brijesh Singh <brijesh.singh@xxxxxxx> Chris Co <Christopher.Co@xxxxxxxxxxxxx> Christopher Co <christopher.co@xxxxxxxxxxxxx> Dandan Bi <dandan.bi@xxxxxxxxx> Eric Dong <eric.dong@xxxxxxxxx> Liming Gao <liming.gao@xxxxxxxxx> Ruiyu Ni <ruiyu.ni@xxxxxxxxx> Star Zeng <star.zeng@xxxxxxxxx> Thomas Palmer <thomas.palmer@xxxxxxx> jobs: build-amd64-xsm pass build-i386-xsm pass build-amd64 pass build-i386 pass build-amd64-libvirt pass build-i386-libvirt fail 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.test-lab.xenproject.org logs: /home/logs/logs images: /home/logs/images Logs, config files, etc. are available at http://logs.test-lab.xenproject.org/osstest/logs Explanation of these reports, and of osstest in general, is at http://xenbits.xen.org/gitweb/?p=osstest.git;a=blob;f=README.email;hb=master http://xenbits.xen.org/gitweb/?p=osstest.git;a=blob;f=README;hb=master Test harness code can be found at http://xenbits.xen.org/gitweb?p=osstest.git;a=summary Not pushing. ------------------------------------------------------------ commit 75b7aa9528bdd05a7ecf4e64a6beb478d31b402c Author: Brijesh Singh <brijesh.singh@xxxxxxx> Date: Fri Jul 6 10:00:42 2018 -0500 OvmfPkg/QemuFlashFvbServicesRuntimeDxe: Restore C-bit when SEV is active AmdSevDxe maps the flash memory range with C=0, but SetMemorySpaceAttributes() unconditionally resets the C-bit to '1'. Lets restore the mapping back to C=0. Cc: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx> Cc: Anthony Perard <anthony.perard@xxxxxxxxxx> Cc: Julien Grall <julien.grall@xxxxxxxxxx> Cc: Justen Jordan L <jordan.l.justen@xxxxxxxxx> Cc: Laszlo Ersek <lersek@xxxxxxxxxx> Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Brijesh Singh <brijesh.singh@xxxxxxx> Reviewed-by: Laszlo Ersek <lersek@xxxxxxxxxx> Regression-tested-by: Laszlo Ersek <lersek@xxxxxxxxxx> commit 3b3d016b7b867d7e4782af9a6b54e110d155a1b3 Author: Brijesh Singh <brijesh.singh@xxxxxxx> Date: Fri Jul 6 10:00:41 2018 -0500 OvmfPkg/QemuFlashFvbServicesRuntimeDxe: Do not expose MMIO in SMM build In the SMM build, only an SMM driver is using the address range hence we do not need to expose the flash MMIO range in EFI runtime mapping. Cc: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx> Cc: Anthony Perard <anthony.perard@xxxxxxxxxx> Cc: Julien Grall <julien.grall@xxxxxxxxxx> Cc: Justen Jordan L <jordan.l.justen@xxxxxxxxx> Cc: Laszlo Ersek <lersek@xxxxxxxxxx> Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Brijesh Singh <brijesh.singh@xxxxxxx> Reviewed-by: Laszlo Ersek <lersek@xxxxxxxxxx> Regression-tested-by: Laszlo Ersek <lersek@xxxxxxxxxx> commit 966363d5a34839399e3d9f68d4f4efb4b1a9ec66 Author: Brijesh Singh <brijesh.singh@xxxxxxx> Date: Fri Jul 6 10:00:40 2018 -0500 OvmfPkg/QemuFlashFvbServicesRuntimeDxe: mark Flash memory range as MMIO The flash memory range is an IO address and should be presented as Memory Mapped IO in EFI Runtime mapping. This information can be used by OS when mapping the flash memory range. It is especially helpful in SEV guest case, in which IO addresses should be mapped as unencrypted. If memory region is not marked as MMIO then OS maps the range as encrypted. Cc: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx> Cc: Anthony Perard <anthony.perard@xxxxxxxxxx> Cc: Julien Grall <julien.grall@xxxxxxxxxx> Cc: Justen Jordan L <jordan.l.justen@xxxxxxxxx> Cc: Laszlo Ersek <lersek@xxxxxxxxxx> Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Brijesh Singh <brijesh.singh@xxxxxxx> Reviewed-by: Laszlo Ersek <lersek@xxxxxxxxxx> Regression-tested-by: Laszlo Ersek <lersek@xxxxxxxxxx> commit 9090c8b533013078c6df773e128070c76d917830 Author: Thomas Palmer <thomas.palmer@xxxxxxx> Date: Wed Jul 4 00:46:01 2018 +0800 MdeModulePkg/DisplayEngineDxe: Fix small InitializeDisplayEngine leak After calling RegisterHotKey, the allocated memory in NewString should be freed. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Thomas Palmer <thomas.palmer@xxxxxxx> Reviewed-by: Dandan Bi <dandan.bi@xxxxxxxxx> Reviewed-by: Star Zeng <star.zeng@xxxxxxxxx> commit 4d76bbcce62e3b972cd226e5f0e43d21db48a3f7 Author: Thomas Palmer <thomas.palmer@xxxxxxx> Date: Wed Jul 4 00:40:52 2018 +0800 MdeModulePkg/UefiBootManagerLib: Fix small LoadOptionToVariable leak After calling SetVariable, the allocated memory in Variable should be freed. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Thomas Palmer <thomas.palmer@xxxxxxx> Reviewed-by: Star Zeng <star.zeng@xxxxxxxxx> commit 4adf7074eb01b5fd13b20f4d2ec05648e2231377 Author: Liming Gao <liming.gao@xxxxxxxxx> Date: Wed Jun 13 14:47:13 2018 +0800 BaseTools Conf: Update tools_def and build_rule to remove IPF setting Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming.gao@xxxxxxxxx> Cc: Yonghong Zhu <yonghong.zhu@xxxxxxxxx> Reviewed-by: Yonghong Zhu <yonghong.zhu@xxxxxxxxx> commit 9917def3294bac4d14cd86ba382652aa41c6c1d6 Author: Eric Dong <eric.dong@xxxxxxxxx> Date: Wed Jul 4 15:01:31 2018 +0800 MdeModulePkg/S3SmmInitDone.h: Fix copyright coding style error. It is caused by change d1102dba7210b95e41d06c2338a22ba6af248645. Cc: Star Zeng <star.zeng@xxxxxxxxx> Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Eric Dong <eric.dong@xxxxxxxxx> Reviewed-by: Star Zeng <star.zeng@xxxxxxxxx> commit f88290964fe528ffeb67ff108e8174fc2ce52741 Author: Brijesh Singh <brijesh.singh@xxxxxxx> Date: Wed Jul 4 10:02:16 2018 +0800 MdeModulePkg/Variable: Check EFI_MEMORY_RUNTIME attribute before setting it Set the EFI_MEMORY_RUNTIME attribute in FtwNotificationEvent() only if the attribute is not already present. This will ensure that the attributes set by the platform drivers (e.g Ovmf pflash) is not lost. Cc: Dong Eric <eric.dong@xxxxxxxxx> Cc: Justen Jordan L <jordan.l.justen@xxxxxxxxx> Cc: Zeng Star <star.zeng@xxxxxxxxx> Cc: Laszlo Ersek <lersek@xxxxxxxxxx> Contributed-under: TianoCore Contribution Agreement 1.1 Reviewed-by: Star Zeng <star.zeng@xxxxxxxxx> Signed-off-by: Brijesh Singh <brijesh.singh@xxxxxxx> commit dd3907f221fd9b774a5734cb7a000377acdfd5ca Author: Dandan Bi <dandan.bi@xxxxxxxxx> Date: Mon Jul 2 13:29:52 2018 +0800 MdeModulePkg/SmmCorePerformanceLib: Update mPerformanceLength In mSmmBootPerformanceTable there are two parts, one is the FPDT table header and the other is FPDT records. Currently: mPerformanceLength: The length of existing FPDT records. mMaxPerformanceLength: The maximum length allocated for mSmmBootPerformanceTable(including FPDT table header length and existing FPDT records length) But when compare mPerformanceLength with mMaxPerformanceLength to check whether need to allocate new buffer for new FPDT record, we miss to add mPerformanceLength with header length, which will cause pool allocation behavior is not correct. Now update the mPerformanceLength to fix this issue: updated mPerformanceLength = FPDT table header length + existing FPDT records length. Cc: Liming Gao <liming.gao@xxxxxxxxx> Cc: Star Zeng <star.zeng@xxxxxxxxx> Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Dandan Bi <dandan.bi@xxxxxxxxx> Reviewed-by: Liming Gao <liming.gao@xxxxxxxxx> commit 5895956dd955714c0e578a413d0e289153cc9eea Author: Chris Co <Christopher.Co@xxxxxxxxxxxxx> Date: Fri Jun 29 03:31:21 2018 +0800 BaseTools/Trim: Normalize filepaths to fix comparisons on Windows When using Linaro GCC5+ arm-eabi toolchain on Windows, the generated DSDT.iii contains a canonicalized ("\.\" removed and lower case) filepath for the preprocessed DSDT.i file in the first line. Trim.exe is called on DSDT.iii to generate DSDT.iiii, which does a line for line comparison of filepaths encountered to the preprocessed DSDT.i filepath found in the first line to determine what lines to place in DSDT.iiii. Since the DSDT.i filepath is canonicalized and all later filepaths in DSDT.iii are not canonicalized, all comparisons fail and the result is in an empty DSDT.iiii. Issue was first reported to Linaro here: https://bugs.linaro.org/show_bug.cgi?id=2909 where the recommendation was to address the issue in Trim.exe. This patch normalizes the case and pathname of all filepaths encountered during Trim.exe execution on preprocessed files. This fixes comparisons of filepaths that contain mismatching case on case-insensitive filesystems, redundant separators, and uplevel references. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Christopher Co <christopher.co@xxxxxxxxxxxxx> Cc: Leif Lindholm <leif.lindholm@xxxxxxxxxx> Cc: Yonghong Zhu <yonghong.zhu@xxxxxxxxx> Cc: Liming Gao <liming.gao@xxxxxxxxx> Reviewed-by: Liming Gao <liming.gao@xxxxxxxxx> commit cd69c8738391d9ce6c09cf5118c913cc7cd41d8b Author: Star Zeng <star.zeng@xxxxxxxxx> Date: Mon Jul 2 17:07:54 2018 +0800 MdeModulePkg SataControllerDxe: Use compare logic in if condition Use compare logic in if condition to fix ECC issue. It is caused by aa4240edff41034d709938a15b42cf4fd3214386. Cc: Hao Wu <hao.a.wu@xxxxxxxxx> Cc: Dandan Bi <dandan.bi@xxxxxxxxx> Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Star Zeng <star.zeng@xxxxxxxxx> Reviewed-by: Dandan Bi <dandan.bi@xxxxxxxxx> Reviewed-by: Hao Wu <hao.a.wu@xxxxxxxxx> commit 5f46352300f64d79c43cbfc67bb7a5f94bb982b8 Author: Star Zeng <star.zeng@xxxxxxxxx> Date: Mon Jul 2 16:54:23 2018 +0800 MdeModulePkg Variable: Add/Correct GetHobVariableStore para description It is caused by 09808bd39b0c76559354253639766458ec24da79. Cc: Liming Gao <liming.gao@xxxxxxxxx> Cc: Dandan Bi <dandan.bi@xxxxxxxxx> Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Star Zeng <star.zeng@xxxxxxxxx> Reviewed-by: Liming Gao <liming.gao@xxxxxxxxx> Reviewed-by: Dandan Bi <dandan.bi@xxxxxxxxx> commit 9c227e07dde56c5f2ab9d00fd6740827f28dc875 Author: Ruiyu Ni <ruiyu.ni@xxxxxxxxx> Date: Mon Jul 2 11:19:26 2018 +0800 MdeModulePkg/ResetSystemRuntimeDxe: Add more info in debug message Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Ruiyu Ni <ruiyu.ni@xxxxxxxxx> Reviewed-by: Star Zeng <star.zeng@xxxxxxxxx> commit c0bf47682d6bcecc3889ff849c81cc06a22debc4 Author: Ruiyu Ni <ruiyu.ni@xxxxxxxxx> Date: Mon Jul 2 11:16:22 2018 +0800 MdeModulePkg/[Pei|Dxe]ResetSystemLib: Add PPI/Protocol dependency Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Ruiyu Ni <ruiyu.ni@xxxxxxxxx> Reviewed-by: Star Zeng <star.zeng@xxxxxxxxx> commit 93f5a54ff3386a44b08de22e9b9230e3ed92b90c Author: Ruiyu Ni <ruiyu.ni@xxxxxxxxx> Date: Mon Jul 2 11:13:17 2018 +0800 MdeModulePkg/ResetUtilityLib: Add pack(1) for reset data structure RESET_UTILITY_GUID_SPECIFIC_RESET_DATA structure should be declared as pack(1). The patch adds the missing pack(1) pragma. (GUID *)((UINT8 *)&ResetData + OFFSET_OF (RESET_UTILITY_GUID_SPECIFIC_RESET_DATA, ResetSubtype)) is used to replace &ResetData.ResetSubType to resolve C4366 VS compiler warning. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Ruiyu Ni <ruiyu.ni@xxxxxxxxx> Reviewed-by: Star Zeng <star.zeng@xxxxxxxxx> commit 51fe5b5140ba9ecb168fb03da328983355880c7a Author: Ruiyu Ni <ruiyu.ni@xxxxxxxxx> Date: Mon Jun 25 15:35:06 2018 +0800 MdeModulePkg/BmpSupportLib: Check PixelHeight/PixelWidth against 0 The patch adds check logic to make sure that for a input BMP file, the width or height is not 0; for a input GOP blt buffer, the width or height is not 0. Otherwise, UNSUPPORTED status is returned. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Ruiyu Ni <ruiyu.ni@xxxxxxxxx> Reviewed-by: Star Zeng <star.zeng@xxxxxxxxx> Cc: Michael D Kinney <michael.d.kinney@xxxxxxxxx> commit 2a9e1b97c9a86a736707388fe5e22cb75c378b35 Author: Ruiyu Ni <ruiyu.ni@xxxxxxxxx> Date: Mon Jun 25 14:54:33 2018 +0800 MdeModulePkg/BmpSupportLib: Correct debug message Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Ruiyu Ni <ruiyu.ni@xxxxxxxxx> Cc: Michael D Kinney <michael.d.kinney@xxxxxxxxx> Reviewed-by: Star Zeng <star.zeng@xxxxxxxxx> _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxxxxxxxxx https://lists.xenproject.org/mailman/listinfo/xen-devel
|
 |
Lists.xenproject.org is hosted with RackSpace, monitoring our |