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

[Xen-devel] [ovmf test] 121374: regressions - FAIL



flight 121374 ovmf real [real]
http://logs.test-lab.xenproject.org/osstest/logs/121374/

Regressions :-(

Tests which did not succeed and are blocking,
including tests which could not be run:
 test-amd64-amd64-xl-qemuu-ovmf-amd64 16 guest-localmigrate/x10 fail REGR. vs. 
121338

version targeted for testing:
 ovmf                 3d7ebd6434311f6cf7e9dbb42c9db0f6f3185085
baseline version:
 ovmf                 0760ed06a139aa6f84568147e3ee4fe919469238

Last test of basis   121338  2018-03-29 04:22:00 Z    2 days
Testing same since   121374  2018-03-30 09:27:30 Z    1 days    1 attempts

------------------------------------------------------------
People who touched revisions under test:
  Alex James <theracermaster@xxxxxxxxx>
  Bin Wang <binx.a.wang@xxxxxxxxx>
  Carsey, Jaben </o=Intel/ou=Americas01/cn=Workers/cn=Carsey, Jaben>
  Jaben Carsey <jaben.carsey@xxxxxxxxx>
  Jeff Brasen <jbrasen.qdt@xxxxxxxxxxxxxxxxxxxxxx>
  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                         fail    
 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 3d7ebd6434311f6cf7e9dbb42c9db0f6f3185085
Author: Star Zeng <star.zeng@xxxxxxxxx>
Date:   Wed Mar 28 17:27:50 2018 +0800

    MdeModulePkg Variable: Align TPL level for (Smm)EndOfDxe callback
    
    VariableRuntimeDxe will have OnEndOfDxe() callback function at
    TPL_NOTIFY level on EndOfDxe event when DXE variable solution is
    used.
      Status = gBS->CreateEventEx (
                      EVT_NOTIFY_SIGNAL,
                      TPL_NOTIFY,
                      OnEndOfDxe,
                      NULL,
                      &gEfiEndOfDxeEventGroupGuid,
                      &EndOfDxeEvent
                      );
    
    VariableSmm will have SmmEndOfDxeCallback() callback function at
    TPL_CALLBACK level on SmmEndOfDxe event when SMM variable solution
    is used.
    SmmIplGuidedEventNotify()  -  PiSmmIpl.c TPL_CALLBACK on EndOfDxe
    ->
    SmmEndOfDxeHandler()  -  PiSmmCore.c install SmmEndOfDxe protocol
    ->
    SmmEndOfDxeCallback() - VariableSmm.c
    
    The TPL level for (Smm)EndOfDxe callback between VariableRuntimeDxe
    and VariableSmm is inconsistent, it will make the unified platform
    code could not make sure its TPL_NOTIFY EndOfDxe callback function
    (to use variable lock/check) executed before (Smm)EndOfDxe callback
    function in variable driver. The variable lock/check will start to
    protect after (Smm)EndOfDxe callback function in variable driver is
    executed.
    
    This patch is to algin the TPL level to TPL_CALLBACK for (Smm)EndOfDxe
    callback between VariableRuntimeDxe and VariableSmm.
    
    Cc: Chasel Chiu <chasel.chiu@xxxxxxxxx>
    Cc: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Cc: Liming Gao <liming.gao@xxxxxxxxx>
    Cc: Jiewen Yao <jiewen.yao@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Star Zeng <star.zeng@xxxxxxxxx>
    Reviewed-by: Chasel Chiu <chasel.chiu@xxxxxxxxx>

commit 0b560b980ce00588e540a480532dd48328a9f003
Author: Carsey, Jaben </o=Intel/ou=Americas01/cn=Workers/cn=Carsey, Jaben>
Date:   Thu Mar 29 08:02:20 2018 +0800

    BaseTools: cleanup class heirarchy
    
    remove totally empty classes from class heirarchy
    
    Cc: Liming Gao <liming.gao@xxxxxxxxx>
    Cc: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Jaben Carsey <jaben.carsey@xxxxxxxxx>
    Reviewed-by: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>

commit 5fbb0f9908ded1944aa4bba599ccc4c605987cb7
Author: Carsey, Jaben </o=Intel/ou=Americas01/cn=Workers/cn=Carsey, Jaben>
Date:   Thu Mar 29 08:02:19 2018 +0800

    BaseTools: remove loop and variables.
    
    this loop does nothing. none of Key, Item, nor DevicePathList
    are ever used.
    
    Cc: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
    Cc: Liming Gao <liming.gao@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Jaben Carsey <jaben.carsey@xxxxxxxxx>
    Reviewed-by: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>

commit 0a014fba41b631258687f56e630518b54817d062
Author: Carsey, Jaben </o=Intel/ou=Americas01/cn=Workers/cn=Carsey, Jaben>
Date:   Thu Mar 29 08:02:18 2018 +0800

    BaseTools: use in to compare single chars
    
    instead if 3 Startswith for single chars, just use in with a list of chars
    
    Cc: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
    Cc: Liming Gao <liming.gao@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Jaben Carsey <jaben.carsey@xxxxxxxxx>
    Reviewed-by: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>

commit 0944818a1972b07b09b53a2a1e88295cd92361cf
Author: Carsey, Jaben </o=Intel/ou=Americas01/cn=Workers/cn=Carsey, Jaben>
Date:   Thu Mar 29 08:02:17 2018 +0800

    BaseTools: no need to do int() API work for it
    
    int() with base=0 will already auto determine base from preceeding 0x/0X
    
    Cc: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
    Cc: Liming Gao <liming.gao@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Jaben Carsey <jaben.carsey@xxxxxxxxx>
    Reviewed-by: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>

commit 4231a8193ec0d52df7e0a101d96c51b1a2b7a996
Author: Carsey, Jaben </o=Intel/ou=Americas01/cn=Workers/cn=Carsey, Jaben>
Date:   Tue Mar 27 04:25:43 2018 +0800

    BaseTools: Remove equality operator with None
    
    replace "== None" with "is None" and "!= None" with "is not None"
    
    Cc: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
    Cc: Liming Gao <liming.gao@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Jaben Carsey <jaben.carsey@xxxxxxxxx>
    Reviewed-by: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>

commit 05a32984ab799a564e2eeb7dff128fe0992910d8
Author: Alex James <theracermaster@xxxxxxxxx>
Date:   Tue Mar 27 00:15:10 2018 +0800

    CorebootPayloadPkg: Conditionally add DebugAgentLib for DXE drivers
    
    To fix building with SOURCE_DEBUG_ENABLE, add DebugAgentLib for
    LibraryClasses.common.DXE_DRIVER, as is done with Vlv2TbltDevicePkg.
    
    Cc: Maurice Ma <maurice.ma@xxxxxxxxx>
    Cc: Prince Agyeman <prince.agyeman@xxxxxxxxx>
    Cc: Benjamin You <benjamin.you@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Alex James <theracermaster@xxxxxxxxx>
    Reviewed-by: Benjamin You <benjamin.you@xxxxxxxxx>

commit b1956b5d42182309604d008036882e62f17d1ccf
Author: Bin Wang <binx.a.wang@xxxxxxxxx>
Date:   Mon Feb 26 16:19:30 2018 +0800

    BaseTools: compare GUID value should not case-sensitive
    
    build report error when the same Guid value in FDF file use lowercase,
    in tools_def.txt file use uppercase.
    The guid value's compare should not case-sensitive.
    
    Cc: Liming Gao <liming.gao@xxxxxxxxx>
    Cc: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Bin Wang <binx.a.wang@xxxxxxxxx>
    Reviewed-by: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>

commit cc0321f22ac7422a2d477c98fed209547eaf0cb5
Author: Carsey, Jaben </o=Intel/ou=Americas01/cn=Workers/cn=Carsey, Jaben>
Date:   Tue Mar 27 08:33:08 2018 +0800

    BaseTools: refactor repeated RegExp when no special searching is needed.
    
    use str.replace and try/except.
    
    Cc: Liming Gao <liming.gao@xxxxxxxxx>
    Cc: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Jaben Carsey <jaben.carsey@xxxxxxxxx>
    Reviewed-by: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>

commit e52aed0d855218f23cbd94629561eb4155936cec
Author: Carsey, Jaben </o=Intel/ou=Americas01/cn=Workers/cn=Carsey, Jaben>
Date:   Wed Mar 28 07:42:47 2018 +0800

    BaseTools: dont use enumerate when un-needed
    
    Since we only use the item from the list and not the numeric value,
    dont bother with enumerate()
    
    Cc: Liming Gao <liming.gao@xxxxxxxxx>
    Cc: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Jaben Carsey <jaben.carsey@xxxxxxxxx>
    Reviewed-by: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>

commit 38504ad3e3163fd8afbfc66760455df9dd223713
Author: Carsey, Jaben </o=Intel/ou=Americas01/cn=Workers/cn=Carsey, Jaben>
Date:   Wed Mar 28 07:42:46 2018 +0800

    BaseTools: move regular expression compile out of function call.
    
    move to the root of the file and dont recompile.
    
    Cc: Liming Gao <liming.gao@xxxxxxxxx>
    Cc: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Jaben Carsey <jaben.carsey@xxxxxxxxx>
    Reviewed-by: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>

commit 663b9e061ed1b48e562159e51333e996f1efc830
Author: Carsey, Jaben </o=Intel/ou=Americas01/cn=Workers/cn=Carsey, Jaben>
Date:   Wed Mar 28 07:42:45 2018 +0800

    BaseTools: expression can use single in instead of 3 API calls.
    
    change 3 StartsWith() calls to a single 'in' operation.
    
    Cc: Liming Gao <liming.gao@xxxxxxxxx>
    Cc: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Jaben Carsey <jaben.carsey@xxxxxxxxx>
    Reviewed-by: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>

commit 56326323e6579d4cde9802c684baba06acbdb1d2
Author: Carsey, Jaben </o=Intel/ou=Americas01/cn=Workers/cn=Carsey, Jaben>
Date:   Sat Mar 17 07:27:46 2018 +0800

    BaseTools: remove local hex number regular expression
    
    Change to using the new shared hex number regular expression
    
    Cc: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
    Cc: Liming Gao <liming.gao@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Jaben Carsey <jaben.carsey@xxxxxxxxx>
    Reviewed-by: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>

commit 018f7b827fa4def3476f76cdf1d6400d4a8e6ebc
Author: Carsey, Jaben </o=Intel/ou=Americas01/cn=Workers/cn=Carsey, Jaben>
Date:   Sat Mar 17 07:27:44 2018 +0800

    BaseTools: Use precompiled RegExp
    
    avoid recompiling the regular expression for each use in a while loop
    
    Cc: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
    Cc: Liming Gao <liming.gao@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Jaben Carsey <jaben.carsey@xxxxxxxxx>
    Reviewed-by: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>

commit b1a9e404d4e91729b99d690fa849451269dd3a47
Author: Carsey, Jaben </o=Intel/ou=Americas01/cn=Workers/cn=Carsey, Jaben>
Date:   Sat Mar 17 07:27:41 2018 +0800

    BaseTools: use new shared GUID regular expressions
    
    remove local variables that are GUID matching and replace with shared
    expression.
    
    Cc: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
    Cc: Liming Gao <liming.gao@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Jaben Carsey <jaben.carsey@xxxxxxxxx>
    Reviewed-by: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>

commit 709c9fd56b2a3303e679858a4927e382f255d8e4
Author: Jeff Brasen <jbrasen.qdt@xxxxxxxxxxxxxxxxxxxxxx>
Date:   Mon Mar 26 16:57:04 2018 +0800

    MdeModulePkg/PartitionDxe: Add partition type guid to installed handle
    
    Add the partition type GUID for every partition to the installed handle,
    this is required per the UEFI specification.
    
    "The firmware must add the PartitionTypeGuid to the handle of every
    active GPT partition using EFI_BOOT_SERVICES.InstallProtocolInterface()."
    
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Jeff Brasen <jbrasen.qdt@xxxxxxxxxxxxxxxxxxxxxx>
    Reviewed-by: Hao Wu <hao.a.wu@xxxxxxxxx>
    Reviewed-by: Ruiyu Ni <ruiyu.ni@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®.