[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

 


Rackspace

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