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

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

This run is configured for baseline tests only.

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

Perfect :-)
All tests in this flight passed as required
version targeted for testing:
 ovmf                 4dd8787a20e2b74cfcc297253f237e0ac86c9289
baseline version:
 ovmf                 5b54c92a6537a9b8650c054348702909c5f57f4f

Last test of basis    67888  2016-10-15 19:48:59 Z    1 days
Testing same since    67891  2016-10-16 22:21:10 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 4dd8787a20e2b74cfcc297253f237e0ac86c9289
Author: Laszlo Ersek <lersek@xxxxxxxxxx>
Date:   Wed Oct 12 10:50:18 2016 +0200

    OvmfPkg/QemuVideoDxe: upgrade VERBOSE debug messages to INFO
    
    In commit 5b2291f9567a ("OvmfPkg: QemuVideoDxe uses
    MdeModulePkg/FrameBufferLib"), QemuVideoDxe was rebased to
    FrameBufferBltLib.
    
    The FrameBufferBltLib instance added in commit b1ca386074bd
    ("MdeModulePkg: Add FrameBufferBltLib library instance") logs many
    messages on the VERBOSE level; for example, a normal boot with OVMF can
    produce 500+ "VideoFill" messages, dependent on the progress bar, when the
    VERBOSE bit is set in PcdDebugPrintErrorLevel. While FrameBufferBltLib is
    certainly allowed to log such messages on the VERBOSE level, we should
    separate those frequent messages from the (infrequent) ones produced by
    QemuVideoDxe itself.
    
    QemuVideoDxe logs VERBOSE messages in three locations (in two functions)
    at the moment. All of them are infrequent: both QemuVideoBochsModeSetup()
    and InstallVbeShim() are called from QemuVideoControllerDriverStart(),
    that is, when a device is bound. Upgrade these messages to INFO level, so
    that VERBOSE can be disabled in PcdDebugPrintErrorLevel -- perhaps
    selectively for OvmfPkg/QemuVideoDxe -- without hiding these infrequent
    messages.
    
    Cc: Jordan Justen <jordan.l.justen@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: Laszlo Ersek <lersek@xxxxxxxxxx>
    Reviewed-by: Jordan Justen <jordan.l.justen@xxxxxxxxx>

commit 88ba362da776b8971db792e32cd21d5c6233d931
Author: Laszlo Ersek <lersek@xxxxxxxxxx>
Date:   Wed Oct 12 10:38:49 2016 +0200

    OvmfPkg/QemuVideoDxe: remove useless QEMU_VIDEO_PRIVATE_DATA.LineBuffer
    
    This field is (re)allocated in QemuVideoGraphicsOutputSetMode(), released
    in QemuVideoGraphicsOutputDestructor(), and used for nothing else. Remove
    it.
    
    Cc: Jordan Justen <jordan.l.justen@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: Laszlo Ersek <lersek@xxxxxxxxxx>
    Reviewed-by: Jordan Justen <jordan.l.justen@xxxxxxxxx>

commit 37b9c0a656ea446bad1824b70a48905e5be1a37b
Author: Laszlo Ersek <lersek@xxxxxxxxxx>
Date:   Wed Oct 12 10:28:09 2016 +0200

    OvmfPkg/QemuVideoDxe: remove useless QEMU_VIDEO_PRIVATE_DATA.CurrentMode
    
    This field is never used.
    
    Cc: Jordan Justen <jordan.l.justen@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: Laszlo Ersek <lersek@xxxxxxxxxx>
    Reviewed-by: Jordan Justen <jordan.l.justen@xxxxxxxxx>

commit 9ebb7ce8dda515fa387dda44e998ceb123137c26
Author: Laszlo Ersek <lersek@xxxxxxxxxx>
Date:   Wed Oct 12 10:09:52 2016 +0200

    OvmfPkg/QemuVideoDxe: drop QEMU_VIDEO_CIRRUS_MODES.RefreshRate
    
    Thanks to the previous patch, this field is also unnecessary now. Remove
    it.
    
    The patch is best reviewed with "git show --word-diff".
    
    Cc: Jordan Justen <jordan.l.justen@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: Laszlo Ersek <lersek@xxxxxxxxxx>
    Reviewed-by: Jordan Justen <jordan.l.justen@xxxxxxxxx>

commit 0ccc97e9c858cac2b64e0ecd18352af255669e16
Author: Laszlo Ersek <lersek@xxxxxxxxxx>
Date:   Wed Oct 12 10:00:10 2016 +0200

    OvmfPkg/QemuVideoDxe: drop useless QEMU_VIDEO_MODE_DATA.RefreshRate
    
    This field is never used beyond assignment and debug-logging. Remove it.
    
    Cc: Jordan Justen <jordan.l.justen@xxxxxxxxx>
    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