|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-devel] [ovmf baseline-only test] 72112: all pass
This run is configured for baseline tests only. flight 72112 ovmf real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/72112/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 11b4463e096523fe03ac840472d483652ae93904 baseline version: ovmf b30abe7df464d91b84a42763e61bb3c85bfd80c9 Last test of basis 72107 2017-09-15 03:50:43 Z 0 days Testing same since 72112 2017-09-15 16:19:31 Z 0 days 1 attempts ------------------------------------------------------------ People who touched revisions under test: Brijesh Singh <brijesh.singh@xxxxxxx> Laszlo Ersek <lersek@xxxxxxxxxx> Paulo Alcantara <pcacjr@xxxxxxxxx> Star Zeng <star.zeng@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 11b4463e096523fe03ac840472d483652ae93904 Author: Paulo Alcantara <pcacjr@xxxxxxxxx> Date: Tue Sep 12 09:30:23 2017 +0800 MdeModulePkg/UdfDxe: Fix NULL pointer dereference REF: https://bugzilla.tianocore.org/show_bug.cgi?id=704 For root directory, the FID (File Identifier Descriptor) pointer is accessible through PRIVATE_UDF_FILE_DATA.Root, whereas non-root directory and regular files, their FIDs are accessible through PRIVATE_UDF_FILE_DATA.File. In UdfSetPosition(), the FID was retrieved through PRIVATE_UDF_FILE_DATA.File, hence when calling it with a root directory, PRIVATE_UDF_FILE_DATA.File.FileIdentifierDescriptor would be NULL and then dereferenced. This patch fixes the NULL pointer dereference by calling _FILE() to transparently return the correct UDF_FILE_INFO * which points to a valid FID descriptor of a specific file. Cc: Star Zeng <star.zeng@xxxxxxxxx> Cc: Eric Dong <eric.dong@xxxxxxxxx> Cc: Ruiyu Ni <ruiyu.ni@xxxxxxxxx> Cc: Laszlo Ersek <lersek@xxxxxxxxxx> Cc: Steven Shi <steven.shi@xxxxxxxxx> Contributed-under: TianoCore Contribution Agreement 1.1 Reported-by: Steven Shi <steven.shi@xxxxxxxxx> Signed-off-by: Paulo Alcantara <pcacjr@xxxxxxxxx> Reviewed-by: Laszlo Ersek <lersek@xxxxxxxxxx> Reviewed-by: Star Zeng <star.zeng@xxxxxxxxx> commit 54537008a2f7d4e8a35507af8b523e7b678abacd Author: Star Zeng <star.zeng@xxxxxxxxx> Date: Thu Sep 14 18:52:22 2017 +0800 MdeModulePkg UdfDxe: Fix VS2010/VS2012 build failure VS2010/VS2012 build failure with below info: warning C4701: potentially uninitialized local variable 'DataOffset' used potentially uninitialized local variable 'FilePosition' used potentially uninitialized local variable 'FinishedSeeking' used potentially uninitialized local variable 'Data' used warning C4703: potentially uninitialized local pointer variable 'Data' used In fact, DataOffset, FilePosition and FinishedSeeking are initialized and then used if (ReadFileInfo->Flags == READ_FILE_SEEK_AND_READ). DoFreeAed will be set to TRUE when Data is allocated and returned from GetAedAdsData(), and Data will be freed if (DoFreeAed) when exiting. Use same method at 5afa5b815936e2b45a375b6521764195bed68680 to fix the build failure. There is related discussion at https://lists.01.org/pipermail/edk2-devel/2017-September/014641.html Cc: Laszlo Ersek <lersek@xxxxxxxxxx> Cc: Eric Dong <eric.dong@xxxxxxxxx> Cc: Paulo Alcantara <pcacjr@xxxxxxxxx> Cc: Ruiyu Ni <ruiyu.ni@xxxxxxxxx> 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: Paulo Alcantara <pcacjr@xxxxxxxxx> commit f60ed5ab7fb39f7327727facd56e58aedaebba31 Author: Brijesh Singh <brijesh.singh@xxxxxxx> Date: Thu Sep 14 16:22:47 2017 -0500 OvmfPkg/VirtioNetDxe: negotiate VIRTIO_F_IOMMU_PLATFORM VirtioNetDxe driver has been updated to use IOMMU-like member functions from VIRTIO_DEVICE_PROTOCOL to translate the system physical address to device address. We do not need to do anything special when VIRTIO_F_IOMMU_PLATFORM bit is present hence treat it in parallel with VIRTIO_F_VERSION_1. Cc: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx> Cc: Jordan Justen <jordan.l.justen@xxxxxxxxx> Cc: Tom Lendacky <thomas.lendacky@xxxxxxx> 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> Tested-by: Laszlo Ersek <lersek@xxxxxxxxxx> commit 8fa54a8a948fb5ea28be0caa27cedded64243741 Author: Brijesh Singh <brijesh.singh@xxxxxxx> Date: Thu Sep 14 16:22:46 2017 -0500 OvmfPkg/VirtioNetDxe: map caller-supplied Tx packet to device-address When device is behind the IOMMU, driver is require to pass the device address of caller-supplied transmit buffer for the bus master operations. The patch uses VirtioNetMapTxBuf() to map caller-supplied Tx packet to a device-address and enqueue the device address in VRING for transfer and perform the reverse mapping when transfer is completed so that we can return the caller-supplied buffer. Cc: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx> Cc: Jordan Justen <jordan.l.justen@xxxxxxxxx> Cc: Tom Lendacky <thomas.lendacky@xxxxxxx> 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> Tested-by: Laszlo Ersek <lersek@xxxxxxxxxx> commit bd114d9f776bfd26a1cf4c731429112821c9e944 Author: Brijesh Singh <brijesh.singh@xxxxxxx> Date: Thu Sep 14 16:22:45 2017 -0500 OvmfPkg/VirtioNetDxe: add Tx packet map/unmap helper functions When device is behind IOMMU, driver is require to pass the device address of TxBuf in the Tx VRING. The patch adds helper functions and data structure to map and unmap the TxBuf system physical address to a device address. Since the TxBuf is returned back to caller from VirtioNetGetStatus() hence we use OrderedCollection interface to save the TxBuf system physical to device address mapping. After the TxBuf is succesfully transmitted VirtioNetUnmapTxBuf() does the reverse lookup in OrderedCollection data structure to get the system physical address of TxBuf for a given device address. Cc: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx> Cc: Jordan Justen <jordan.l.justen@xxxxxxxxx> Cc: Tom Lendacky <thomas.lendacky@xxxxxxx> 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> Tested-by: Laszlo Ersek <lersek@xxxxxxxxxx> commit 76ad23ca82292a8fa3e9f3f8d5637f7aecbb9d69 Author: Brijesh Singh <brijesh.singh@xxxxxxx> Date: Thu Sep 14 16:22:44 2017 -0500 OvmfPkg/VirtioNetDxe: update TechNotes In next patches we will update Virtio transmit to use the device-mapped address of the caller-supplied packet. The patch documents the new model. Cc: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx> Cc: Jordan Justen <jordan.l.justen@xxxxxxxxx> Cc: Tom Lendacky <thomas.lendacky@xxxxxxx> 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> Tested-by: Laszlo Ersek <lersek@xxxxxxxxxx> commit 891f016c1b99f2f4c982f0d027c7ff0c001bd270 Author: Brijesh Singh <brijesh.singh@xxxxxxx> Date: Thu Sep 14 16:22:43 2017 -0500 OvmfPkg/VirtioNetDxe: dynamically alloc transmit header Each network packet is submitted for transmission by pushing the head descriptor of a two-part descriptor chain to the Available Ring of the TX queue. VirtioNetInitTx() sets up the the descriptor chains for all queueable packets in advance, and points all the head descriptors to the same shared, never modified, VIRTIO_1_0_NET_REQ header object (or its initial VIRTIO_NET_REQ sub-object, dependent on virtio version). VirtioNetInitTx() currently uses the header object's system physical address for populating the head descriptors. When device is behind the IOMMU, VirtioNet driver is required to provide the device address of VIRTIO_1_0_NET_REQ header. In this patch we dynamically allocate the header using AllocateSharedPages() and map with BusMasterCommonBuffer so that header can be accessed by both processor and the device. We map the header object for CommonBuffer operation because, in order to stick with the current code order, we populate the head descriptors with the header's device address first, and fill in the header itself second. Cc: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx> Cc: Jordan Justen <jordan.l.justen@xxxxxxxxx> Cc: Tom Lendacky <thomas.lendacky@xxxxxxx> 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> Tested-by: Laszlo Ersek <lersek@xxxxxxxxxx> commit 46b11f00ac704ed4a1e3c8c1c3439f914e2dfadc Author: Brijesh Singh <brijesh.singh@xxxxxxx> Date: Thu Sep 14 16:22:42 2017 -0500 OvmfPkg/VirtioNetDxe: alloc RxBuf using AllocateSharedPages() When device is behind the IOMMU, VirtioNetDxe is required to use the device address in bus master operations. RxBuf is allocated using AllocatePool() which returns the system physical address. The patch uses VIRTIO_DEVICE_PROTOCOL.AllocateSharedPages() to allocate the RxBuf and map with VirtioMapAllBytesInSharedBuffer() so that we can obtain the device address for RxBuf. Cc: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx> Cc: Jordan Justen <jordan.l.justen@xxxxxxxxx> Cc: Tom Lendacky <thomas.lendacky@xxxxxxx> 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> Tested-by: Laszlo Ersek <lersek@xxxxxxxxxx> commit 940baec09c8f4664b34842d13a30ce243d0b7b4b Author: Brijesh Singh <brijesh.singh@xxxxxxx> Date: Thu Sep 14 16:22:41 2017 -0500 OvmfPkg/VirtioNetDxe: map VRINGs using VirtioRingMap() When device is behind the IOMMU then driver need to pass the device address when programing the bus master. The patch uses VirtioRingMap() to map the VRING system physical address[es] to device address[es]. Cc: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx> Cc: Jordan Justen <jordan.l.justen@xxxxxxxxx> Cc: Tom Lendacky <thomas.lendacky@xxxxxxx> 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> Tested-by: Laszlo Ersek <lersek@xxxxxxxxxx> commit 55dd5a673bd7142e3a95a5a597db40bdf385a24c Author: Brijesh Singh <brijesh.singh@xxxxxxx> Date: Thu Sep 14 16:22:40 2017 -0500 OvmfPkg/VirtioNetDxe: add helper VirtioNetUninitRing() Consolidate the virtio VRING resource cleanup into VirtioNetUninitRing(). Cc: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx> Cc: Jordan Justen <jordan.l.justen@xxxxxxxxx> Cc: Tom Lendacky <thomas.lendacky@xxxxxxx> 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> Tested-by: Laszlo Ersek <lersek@xxxxxxxxxx> _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx https://lists.xen.org/xen-devel
|
 |
Lists.xenproject.org is hosted with RackSpace, monitoring our |