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

[ovmf baseline-only test] 68080: all pass

This run is configured for baseline tests only.

flight 68080 ovmf real [real]

Perfect :-)
All tests in this flight passed as required
version targeted for testing:
 ovmf                 b43dd22981b71dd78dd4cc04d55dc23d81c3bafb
baseline version:
 ovmf                 01dd077315c6759c94af9af4232f8318db13cf8d

Last test of basis    68075  2016-11-21 16:49:08 Z    1 days
Testing same since    68080  2016-11-22 11:49:41 Z    0 days    1 attempts

People who touched revisions under test:
  Jeff Fan <jeff.fan@xxxxxxxxx>
  Laszlo Ersek <lersek@xxxxxxxxxx>

 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

Test harness code can be found at

Push not applicable.

commit b43dd22981b71dd78dd4cc04d55dc23d81c3bafb
Author: Laszlo Ersek <lersek@xxxxxxxxxx>
Date:   Thu Nov 17 21:13:29 2016 +0100

    UefiCpuPkg/PiSmmCpuDxeSmm: dynamic PcdCpuSmmApSyncTimeout, PcdCpuSmmSyncMode
    Move the declaration of these PCDs from the
      [PcdsFixedAtBuild, PcdsPatchableInModule]
    section of "UefiCpuPkg/UefiCpuPkg.dec" to the
      [PcdsFixedAtBuild, PcdsPatchableInModule, PcdsDynamic, PcdsDynamicEx]
    section. Their types, default values, and token values remain unchanged.
    Only UefiCpuPkg/PiSmmCpuDxeSmm consumes these PCDs, specifically on the
    call stack of its entry point function, and it turns them into static or
    dynamically allocated data in SMRAM:
      PiCpuSmmEntry()                            [PiSmmCpuDxeSmm.c]
        InitializeSmmTimer()                     [SyncTimer.c]
          -> mTimeoutTicker
        InitializeMpServiceData()                [MpService.c]
          InitializeMpSyncData()                 [MpService.c]
            -> mSmmMpSyncData->EffectiveSyncMode
    However, there's another call path to fetching "PcdCpuSmmSyncMode", namely
      SmmInitHandler()                           [PiSmmCpuDxeSmm.c]
        InitializeMpSyncData()                   [MpService.c]
          -> mSmmMpSyncData->EffectiveSyncMode
    and this path is exercised during S3 resume (as stated by the comment in
    SmmInitHandler() too, "Initialize private data during S3 resume").
    While we can call the PCD protocol (via PcdLib) for fetching dynamic PCDs
    in the entry point function, we cannot do that at S3 resume. Therefore
    pre-fetch PcdCpuSmmSyncMode into a new global variable (which lives in
    SMRAM) in InitializeMpServiceData(), just before calling
    InitializeMpSyncData(). This way InitializeMpSyncData() can retrieve the
    stashed PCD value from SMRAM, regardless of the boot mode.
    Cc: Jeff Fan <jeff.fan@xxxxxxxxx>
    Cc: Jordan Justen <jordan.l.justen@xxxxxxxxx>
    Cc: Michael Kinney <michael.d.kinney@xxxxxxxxx>
    Cc: Paolo Bonzini <pbonzini@xxxxxxxxxx>
    Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=230
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: Laszlo Ersek <lersek@xxxxxxxxxx>
    Reviewed-by: Jeff Fan <jeff.fan@xxxxxxxxx>

commit eaae7b33b1cf6b9f21db1636f219c2b6a8d88afd
Author: Jeff Fan <jeff.fan@xxxxxxxxx>
Date:   Fri Nov 18 10:46:43 2016 +0800

    MdeModulePkg/PiSmmCore: Cache CommunicationBuffer info before using it
    gSmmCorePrivate->CommunicationBuffer and gSmmCorePrivate->BufferSize locate 
    runtime memory region. That means they could be modified by non-SMM code 
    We should cache them into SMM local variables before we verify them. After
    verification, we should use the cached ones directly instead of the ones in
    Cc: Jiewen Yao <jiewen.yao@xxxxxxxxx>
    Cc: Feng Tian <feng.tian@xxxxxxxxx>
    Cc: Michael D Kinney <michael.d.kinney@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: Jeff Fan <jeff.fan@xxxxxxxxx>

osstest-output mailing list



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