[Xen-devel] [ovmf baseline-only test] 68532: all pass

[Platform Team regression test user <citrix-osstest@xxxxxxxxxxxxxx>]

This run is configured for baseline tests only.

flight 68532 ovmf real [real]
http://osstest.xs.citrite.net/~osstest/testlogs/logs/68532/

Perfect :-)
All tests in this flight passed as required
version targeted for testing:
 ovmf                 a316d7ac91d302085b5c35d76db703f2208ec026
baseline version:
 ovmf                 7c609a144b6636577dd60fbaa5fc64efeecd7baf

Last test of basis    68531  2017-02-07 11:17:52 Z    0 days
Testing same since    68532  2017-02-07 15:48:52 Z    0 days    1 attempts

------------------------------------------------------------
People who touched revisions under test:
  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                         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 a316d7ac91d302085b5c35d76db703f2208ec026
Author: Laszlo Ersek <lersek@xxxxxxxxxx>
Date:   Wed Nov 30 20:06:34 2016 +0100

    OvmfPkg/SmmControl2Dxe: select broadcast SMI if available
    
    When writing to IO port 0xB2 (ICH9_APM_CNT), QEMU by default injects an
    SMI only on the VCPU that is writing the port. This has exposed corner
    cases and strange behavior with edk2 code, which generally expects a
    software SMI to affect all CPUs at once. We've experienced instability
    despite the fact that OVMF sets PcdCpuSmmApSyncTimeout and
    PcdCpuSmmSyncMode differently from the UefiCpuPkg defaults, such that they
    match QEMU's unicast SMIs better. (Refer to edk2 commits 9b1e378811ff and
    bb0f18b0bce6.)
    
    Using the new fw_cfg-based SMI feature negotiation in QEMU (see commits
    50de920b372b "hw/isa/lpc_ich9: add SMI feature negotiation via fw_cfg" and
    5ce45c7a2b15 "hw/isa/lpc_ich9: add broadcast SMI feature"), we can ask
    QEMU to broadcast SMIs. Extensive testing from earlier proves that
    broadcast SMIs are only reliable if we use the UefiCpuPkg defaults for the
    above PCDs. With those settings however, the broadcast is very reliable --
    the most reliable configuration encountered thus far.
    
    Therefore negotiate broadcast SMIs with QEMU, and if the negotiation is
    successful, dynamically revert the PCDs to the UefiCpuPkg defaults.
    
    Setting the PCDs in this module is safe:
    
    - only PiSmmCpuDxeSmm consumes them,
    
    - PiSmmCpuDxeSmm is a DXE_SMM_DRIVER, launched by the SMM_CORE
      (MdeModulePkg/Core/PiSmmCore/PiSmmCore.inf),
    
    - the SMM_CORE is launched by the SMM IPL runtime DXE driver
      (MdeModulePkg/Core/PiSmmCore/PiSmmIpl.inf),
    
    - the SMM IPL has a DEPEX on EFI_SMM_CONTROL2_PROTOCOL,
    
    - OvmfPkg/SmmControl2Dxe produces that protocol.
    
    The end result is that PiSmmCpuDxeSmm cannot be dispatched before
    SmmControl2Dxe installs EFI_SMM_CONTROL2_PROTOCOL and returns from its
    entry point. Hence we can set the PCD's consumed by PiSmmCpuDxeSmm in
    SmmControl2Dxe.
    
    Cc: Jordan Justen <jordan.l.justen@xxxxxxxxx>
    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: Jordan Justen <jordan.l.justen@xxxxxxxxx>

commit 22d7be69c45ecc7ce2b21252710c37be7122c1bd
Author: Laszlo Ersek <lersek@xxxxxxxxxx>
Date:   Thu Nov 17 22:21:35 2016 +0100

    OvmfPkg: dynamic defaults for PcdCpuSmmApSyncTimeout, PcdCpuSmmSyncMode
    
    Move the platform-specific default values for these PCDs from the
    [PcdsFixedAtBuild] / [PcdsFixedAtBuild.X64] sections to the
    [PcdsDynamicDefault] section.
    
    Cc: Jordan Justen <jordan.l.justen@xxxxxxxxx>
    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: Jordan Justen <jordan.l.justen@xxxxxxxxx>

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel