[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-devel] [ovmf baseline-only test] 75486: trouble: blocked/broken



This run is configured for baseline tests only.

flight 75486 ovmf real [real]
http://osstest.xensource.com/osstest/logs/75486/

Failures and problems with tests :-(

Tests which did not succeed and are blocking,
including tests which could not be run:
 build-amd64-xsm                 <job status>                 broken
 build-i386                      <job status>                 broken
 build-amd64-pvops               <job status>                 broken
 build-i386-xsm                  <job status>                 broken
 build-amd64                     <job status>                 broken
 build-i386-pvops                <job status>                 broken

Tests which did not succeed, but are not blocking:
 test-amd64-amd64-xl-qemuu-ovmf-amd64  1 build-check(1)             blocked n/a
 build-amd64-libvirt           1 build-check(1)               blocked  n/a
 test-amd64-i386-xl-qemuu-ovmf-amd64  1 build-check(1)              blocked n/a
 build-i386-libvirt            1 build-check(1)               blocked  n/a
 build-i386                    4 host-install(4)       broken baseline untested
 build-amd64                   4 host-install(4)       broken baseline untested
 build-i386-pvops              4 host-install(4)       broken baseline untested
 build-i386-xsm                4 host-install(4)       broken baseline untested
 build-amd64-pvops             4 host-install(4)       broken baseline untested
 build-amd64-xsm               4 host-install(4)       broken baseline untested

version targeted for testing:
 ovmf                 68099b52b0fcc1d45864154954d776d91afb33e0
baseline version:
 ovmf                 0dab57708de64284ac83f0632daba4d9e1cb1135

Last test of basis    75483  2018-10-23 05:20:41 Z    0 days
Testing same since    75486  2018-10-23 10:20:55 Z    0 days    1 attempts

------------------------------------------------------------
People who touched revisions under test:
  Hao Wu <hao.a.wu@xxxxxxxxx>
  Star Zeng <star.zeng@xxxxxxxxx>
  Yonghong Zhu <yonghong.zhu@xxxxxxxxx>

jobs:
 build-amd64-xsm                                              broken  
 build-i386-xsm                                               broken  
 build-amd64                                                  broken  
 build-i386                                                   broken  
 build-amd64-libvirt                                          blocked 
 build-i386-libvirt                                           blocked 
 build-amd64-pvops                                            broken  
 build-i386-pvops                                             broken  
 test-amd64-amd64-xl-qemuu-ovmf-amd64                         blocked 
 test-amd64-i386-xl-qemuu-ovmf-amd64                          blocked 


------------------------------------------------------------
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.xensource.com/osstest/logs

Test harness code can be found at
    http://xenbits.xensource.com/gitweb?p=osstest.git;a=summary

broken-job build-amd64-xsm broken
broken-job build-i386 broken
broken-job build-amd64-pvops broken
broken-job build-i386-xsm broken
broken-job build-amd64 broken
broken-job build-i386-pvops broken
broken-step build-i386 host-install(4)
broken-step build-amd64 host-install(4)
broken-step build-i386-pvops host-install(4)
broken-step build-i386-xsm host-install(4)
broken-step build-amd64-pvops host-install(4)
broken-step build-amd64-xsm host-install(4)

Push not applicable.

------------------------------------------------------------
commit 68099b52b0fcc1d45864154954d776d91afb33e0
Author: Hao Wu <hao.a.wu@xxxxxxxxx>
Date:   Tue Oct 16 13:09:43 2018 +0800

    MdeModulePkg/UdfDxe: Avoid possible use of already-freed data
    
    REF:https://bugzilla.tianocore.org/show_bug.cgi?id=1255
    
    For function ReadFile():
    
    If the line
    
      Status = GetAedAdsData (
       ...
       );
    
    is reached multiple times during the 'for' loop, freeing the data pointed
    by variable 'Data' may potentially lead to variable 'Ad' referencing the
    already-freed data.
    
    After calling function GetAllocationDescriptor(), 'Data' and 'Ad' may
    point to the same memory (with some possible offset). Hence, this commit
    will move the FreePool() call backwards to ensure the data will no longer
    be used.
    
    Cc: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Hao Wu <hao.a.wu@xxxxxxxxx>
    Reviewed-by: Paulo Alcantara <palcantara@xxxxxxx>
    Acked-by: Star Zeng <star.zeng@xxxxxxxxx>

commit 85acb5e8ffef026b80241b1657ed4fba26e382b1
Author: Hao Wu <hao.a.wu@xxxxxxxxx>
Date:   Tue Oct 16 12:16:02 2018 +0800

    MdeModulePkg/UdfDxe: Add more check when getting PD from LongAd
    
    REF:https://bugzilla.tianocore.org/show_bug.cgi?id=1254
    
    This commit will add an additional check within function GetPdFromLongAd()
    when getting a Partition Descriptor from given a Long Allocation
    Descriptor.
    
    According to UDF 2.60 Spec, Section 2.2.13:
    
    > The partition reference numbers used are determined by the order of the
    > Partition Maps in the LVD.
    
    (Also the picture comes before the above contents)
    
    And a more detailed explanation of the partition reference numbers is at
    https://sites.google.com/site/udfintro/ (seems not a formal documentation
    though), Section 5.3.6.
    
    Based on the above findings, the 'PartitionReferenceNumber' field in a
    Long Allocation Descriptor is used as an index to access the Partition
    Maps data within a Logical Volume Descriptor.
    
    Hence, the new check focuses on the validity of this
    'PartitionReferenceNumber' field in a Long Allocation Descriptor. Since
    the current implementation of UdfDxe driver supports only one partition on
    a Logical Volume, so the value of 'PartitionReferenceNumber' should be 0.
    
    Cc: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Hao Wu <hao.a.wu@xxxxxxxxx>
    Reviewed-by: Paulo Alcantara <palcantara@xxxxxxx>
    Acked-by: Star Zeng <star.zeng@xxxxxxxxx>

commit 32698a8f013140b8f77f18a03c64c40ba04c2e18
Author: Hao Wu <hao.a.wu@xxxxxxxxx>
Date:   Fri Oct 12 14:49:41 2018 +0800

    MdeModulePkg/UdfDxe: Update GetInfo() for FS VolumeLabel info request
    
    REF:https://bugzilla.tianocore.org/show_bug.cgi?id=1175
    
    This commit will update the UdfGetInfo() function with the support of
    EFI_FILE_SYSTEM_VOLUME_LABEL data information request.
    
    Cc: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Hao Wu <hao.a.wu@xxxxxxxxx>
    Reviewed-by: Paulo Alcantara <palcantara@xxxxxxx>
    Acked-by: Star Zeng <star.zeng@xxxxxxxxx>

commit 6a926aaed7d110ae60d7ada3ec823b8ce85a797b
Author: Hao Wu <hao.a.wu@xxxxxxxxx>
Date:   Mon Oct 8 14:34:19 2018 +0800

    MdeModulePkg/UdfDxe: Fix a typo within SetFileInfo()
    
    REF:https://bugzilla.tianocore.org/show_bug.cgi?id=1253
    
    Within function SetFileInfo():
    This commit will fix a typo where 'Minute' should be used instead of
    'Second'.
    
    Cc: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Hao Wu <hao.a.wu@xxxxxxxxx>
    Reviewed-by: Paulo Alcantara <palcantara@xxxxxxx>
    Acked-by: Star Zeng <star.zeng@xxxxxxxxx>

commit 6df465b21213ac7c8df78c8db261b064aa6aa56c
Author: Hao Wu <hao.a.wu@xxxxxxxxx>
Date:   Mon Oct 8 14:04:48 2018 +0800

    MdeModulePkg/UdfDxe: Correct behavior for UdfSetPosition()
    
    REF:https://bugzilla.tianocore.org/show_bug.cgi?id=1252
    
    Calling the 'SetPosition' service of the EFI_FILE_PROTOCOL with 'Position'
    equals to 0xFFFFFFFFFFFFFFFF for a file is to set the current position to
    the end of the file. But the current implementation of function
    UdfSetPosition() is to set it to the last byte (not EOF).
    
    This commit will resolve this issue.
    
    Cc: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Hao Wu <hao.a.wu@xxxxxxxxx>
    Reviewed-by: Paulo Alcantara <palcantara@xxxxxxx>
    Acked-by: Star Zeng <star.zeng@xxxxxxxxx>

commit 3b30351b75d70ea65701ac999875fbb81a89a5ca
Author: Hao Wu <hao.a.wu@xxxxxxxxx>
Date:   Wed Dec 13 16:28:33 2017 +0800

    MdeModulePkg/UdfDxe: Add boundary check for getting volume (free) size
    
    REF:https://bugzilla.tianocore.org/show_bug.cgi?id=828
    
    Within GetVolumeSize():
    
    The boundary check will validate the 'NumberOfPartitions' field of a
    Logical Volume Integrity Descriptor matches the data within the relating
    Logical Volume Descriptor.
    
    Cc: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Cc: Jiewen Yao <jiewen.yao@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Hao Wu <hao.a.wu@xxxxxxxxx>
    Reviewed-by: Paulo Alcantara <palcantara@xxxxxxx>
    Acked-by: Star Zeng <star.zeng@xxxxxxxxx>

commit 89f75aa04a97293a8ed9db2a90851a5053730cf5
Author: Hao Wu <hao.a.wu@xxxxxxxxx>
Date:   Wed Dec 13 14:39:07 2017 +0800

    MdeModulePkg/UdfDxe: Add boundary check for ComponentIdentifier decode
    
    REF:https://bugzilla.tianocore.org/show_bug.cgi?id=828
    
    Within ResolveSymlink():
    
    The boundary check will validate the 'LengthofComponentIdentifier' field
    of a Path Component matches the data within the relating (Extended) File
    Entry.
    
    Cc: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Cc: Jiewen Yao <jiewen.yao@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Hao Wu <hao.a.wu@xxxxxxxxx>
    Reviewed-by: Paulo Alcantara <palcantara@xxxxxxx>
    Acked-by: Star Zeng <star.zeng@xxxxxxxxx>

commit 5c0748f43f4e1cc15fdd0be64a764eacd7df92f6
Author: Hao Wu <hao.a.wu@xxxxxxxxx>
Date:   Wed Dec 13 10:23:41 2017 +0800

    MdeModulePkg/UdfDxe: Add boundary check the read of FE/EFE
    
    REF:https://bugzilla.tianocore.org/show_bug.cgi?id=828
    
    Within ReadFile():
    
    Add checks to ensure that when getting the raw data or the Allocation
    Descriptors' data from a FE/EFE, it will not consume data beyond the
    size of a FE/EFE.
    
    Cc: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Cc: Jiewen Yao <jiewen.yao@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Hao Wu <hao.a.wu@xxxxxxxxx>
    Reviewed-by: Paulo Alcantara <palcantara@xxxxxxx>
    Acked-by: Star Zeng <star.zeng@xxxxxxxxx>

commit b9ae1705adfdd43668027a25a2b03c2e81960219
Author: Hao Wu <hao.a.wu@xxxxxxxxx>
Date:   Tue Dec 12 16:30:18 2017 +0800

    MdeModulePkg/UdfDxe: Refine boundary checks for file/path name string
    
    REF:https://bugzilla.tianocore.org/show_bug.cgi?id=828
    
    The commit refines the boundary checks for file/path name string to
    prevent possible buffer overrun.
    
    Cc: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Cc: Jiewen Yao <jiewen.yao@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Hao Wu <hao.a.wu@xxxxxxxxx>
    Reviewed-by: Paulo Alcantara <palcantara@xxxxxxx>
    Acked-by: Star Zeng <star.zeng@xxxxxxxxx>

commit 4df8f5bfa28b8b881e506437e8f08d92c1a00370
Author: Hao Wu <hao.a.wu@xxxxxxxxx>
Date:   Fri Dec 15 11:22:16 2017 +0800

    MdeModulePkg/PartitionDxe: Add check for underlying device block size
    
    REF:https://bugzilla.tianocore.org/show_bug.cgi?id=828
    
    Within FindAnchorVolumeDescriptorPointer():
    
    Add a check for the underlying device block size to ensure it is greater
    than the size of an Anchor Volume Descriptor Pointer.
    
    Cc: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Cc: Jiewen Yao <jiewen.yao@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Hao Wu <hao.a.wu@xxxxxxxxx>
    Reviewed-by: Paulo Alcantara <palcantara@xxxxxxx>
    Acked-by: Star Zeng <star.zeng@xxxxxxxxx>

commit 4a723d3d7fd1cbbc28c92f14361761831ad27bab
Author: Star Zeng <star.zeng@xxxxxxxxx>
Date:   Sun Oct 21 11:12:01 2018 +0800

    MdeModulePkg Xhci: Handle value 5 in Port Speed field of PORTSC
    
    REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1267
    
    The value 5 Port Speed field of PORTSC is new defined in
    XHCI 1.1 spec November 2017.
    
    This patch updates XhciDxe and XhciPei to handle it, otherwise
    the USB 3.1 device may not be recognized with the XHCI controller
    following XHCI 1.1 spec November 2017.
    
    Cc: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Cc: Hao Wu <hao.a.wu@xxxxxxxxx>
    Cc: Jian J Wang <jian.j.wang@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Star Zeng <star.zeng@xxxxxxxxx>
    Reviewed-by: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>

commit fed6cf25b8eefccf302f90f1fa7e54bf4a91b124
Author: Star Zeng <star.zeng@xxxxxxxxx>
Date:   Sun Oct 21 12:12:39 2018 +0800

    MdeModulePkg XhciDxe: Assign Usb2Hc.XXXRevision based on SBRN
    
    Current hard code Usb2Hc.XXXRevision may be not accurate.
    This patch updates code to assign Usb2Hc.XXXRevision based on
    SBRN (Serial Bus Release Number, PCI configuration space offset
    0x60) although there is no code consuming them.
    
    Cc: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Cc: Hao Wu <hao.a.wu@xxxxxxxxx>
    Cc: Jian J Wang <jian.j.wang@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Star Zeng <star.zeng@xxxxxxxxx>
    Reviewed-by: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>

commit a6a326770b0e9e209c604d26fd861a9714c36bef
Author: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
Date:   Fri Oct 19 15:10:30 2018 +0800

    BaseTools: Fix the *B and *F Flag display for Structure Pcd
    
    Because of we newly add the PcdFieldValueFromComm and
    PcdFieldValueFromFdf in early parser phase, so in the report we use
    the saved value in this two variables to print it.
    
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
    Reviewed-by: Liming Gao <liming.gao@xxxxxxxxx>

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

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