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

[Xen-devel] [ovmf baseline-only test] 75012: tolerable FAIL



This run is configured for baseline tests only.

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

Failures :-/ but no regressions.

Tests which did not succeed, but are not blocking:
 test-amd64-i386-xl-qemuu-ovmf-amd64 10 debian-hvm-install      fail like 75011
 test-amd64-amd64-xl-qemuu-ovmf-amd64 10 debian-hvm-install     fail like 75011

version targeted for testing:
 ovmf                 cf4e79e466f951e72c2cbd40002c71fdaddad389
baseline version:
 ovmf                 98d20e44dc72d9858523687fda11ab8fc570fcec

Last test of basis    75011  2018-07-26 09:49:53 Z    0 days
Testing same since    75012  2018-07-26 12:50:05 Z    0 days    1 attempts

------------------------------------------------------------
People who touched revisions under test:
  Eric Dong <eric.dong@xxxxxxxxx>
  Laszlo Ersek <lersek@xxxxxxxxxx>

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                          fail    


------------------------------------------------------------
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


Push not applicable.

------------------------------------------------------------
commit cf4e79e466f951e72c2cbd40002c71fdaddad389
Author: Eric Dong <eric.dong@xxxxxxxxx>
Date:   Thu Jul 26 16:44:22 2018 +0800

    UefiCpuPkg/MpInitLib: Not use disabled AP when call StartAllAPs.
    
    Base on UEFI spec requirement, StartAllAPs function should not use the APs 
which has been disabled before. This patch just change current code to follow 
this rule.
    
    V3 changes:
    Only called by StartUpAllAps, WakeUpAp will not wake up the disabled APs, 
in other cases also need to include the disabled APs, such as CpuDxe driver 
start up and ChangeApLoopCallback function.
    
    WakeUpAP() is called with (Broadcast && WakeUpDisabledAps) from 
MpInitLibInitialize(), CollectProcessorCount() and MpInitChangeApLoopCallback() 
only. The first two run before the PPI or Protocol user has a chance to disable 
any APs. The last one runs in response to the ExitBootServices and LegacyBoot 
events, after which the MP protocol is unusable. For this reason, it doesn't 
matter that an originally disabled AP's state is not restored to Disabled, when
    WakeUpAP() is called with (Broadcast && WakeUpDisabledAps).
    
    Cc: Laszlo Ersek <lersek@xxxxxxxxxx>
    Cc: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Eric Dong <eric.dong@xxxxxxxxx>
    Reviewed-by: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Reviewed-by: Laszlo Ersek <lersek@xxxxxxxxxx>
    Regression-tested-by: Laszlo Ersek <lersek@xxxxxxxxxx>

commit 2da3e96cb7aa4510f5b5086efca0c2188c055751
Author: Eric Dong <eric.dong@xxxxxxxxx>
Date:   Tue Jul 24 22:29:53 2018 +0800

    UefiCpuPkg/MpInitLib: Remove StartCount and volatile definition.
    
    The patch includes below changes:
    (1) It removes "volatile" from RunningCount, because only the BSP modifies 
it.
    (2) When we detect a timeout in CheckAllAPs(), and collect the list of 
failed CPUs, the size of the list is derived from the following difference, 
before the patch:
      StartCount - FinishedCount
    where "StartCount" is set by the BSP at startup, and FinishedCount is 
incremented by the APs themselves.
    Here the patch replaces this difference with
      StartCount - RunningCount
    that is, the difference is no more calculated from the BSP's startup 
counter and the AP's shared finish counter, but from the RunningCount 
measurement that the BSP does itself, in CheckAllAPs().
    (3) Finally, the patch changes the meaning of RunningCount. Before the 
patch, we have:
    - StartCount: the number of APs the BSP stars up,
    - RunningCount: the number of finished APs that the BSP collected
    After the patch, StartCount is removed, and RunningCount is *redefined* as 
the following difference:
      OLD_StartCount - OLD_RunningCount
    Giving the number of APs that the BSP started up but hasn't collected yet.
    
    Cc: Laszlo Ersek <lersek@xxxxxxxxxx>
    Cc: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Eric Dong <eric.dong@xxxxxxxxx>
    Reviewed-by: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Reviewed-by: Laszlo Ersek <lersek@xxxxxxxxxx>
    Tested-by: Laszlo Ersek <lersek@xxxxxxxxxx>

commit 2a5997f899bc0bb2851d3ba7168b78b36444e6f0
Author: Eric Dong <eric.dong@xxxxxxxxx>
Date:   Tue Jul 24 22:25:41 2018 +0800

    UefiCpuPkg/MpInitLib: Remove redundant CpuStateFinished State.
    
    Current CPU state definition include CpuStateIdle and CpuStateFinished.
    After investigation, current code can use CpuStateIdle to replace the
    CpuStateFinished. It will reduce the state number and easy for maintenance.
    
    > Before this patch, the state transitions for an AP are:
    >
    >   Idle ----> Ready ----> Busy ----> Finished ----> Idle
    >        [BSP]       [AP]       [AP]           [BSP]
    >
    > After the patch, the state transitions for an AP are:
    >
    >   Idle ----> Ready ----> Busy ----> Idle
    >        [BSP]       [AP]       [AP]
    
    Cc: Laszlo Ersek <lersek@xxxxxxxxxx>
    Cc: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Eric Dong <eric.dong@xxxxxxxxx>
    Reviewed-by: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Reviewed-by: Laszlo Ersek <lersek@xxxxxxxxxx>
    Tested-by: Laszlo Ersek <lersek@xxxxxxxxxx>

_______________________________________________
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®.