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

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

This run is configured for baseline tests only.

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

Perfect :-)
All tests in this flight passed as required
version targeted for testing:
 ovmf                 4a2aaff2fca69d9f41c5b8906699ba242278cbaa
baseline version:
 ovmf                 b8922094f6f8b5293f01a09035b74463fff12320

Last test of basis    67624  2016-09-02 05:18:08 Z    0 days
Testing same since    67626  2016-09-02 10:46:31 Z    0 days    1 attempts

------------------------------------------------------------
People who touched revisions under test:
  Ard Biesheuvel <ard.biesheuvel@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 4a2aaff2fca69d9f41c5b8906699ba242278cbaa
Author: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx>
Date:   Wed Aug 17 16:36:42 2016 +0200

    MdeModulePkg/EbcDxe AARCH64: simplify interpreter entry point thunks
    
    The prototypes of EbcInterpret() and ExecuteEbcImageEntryPoint() are
    private to the AARCH64 implementation of EbcDxe, so we can shuffle
    the arguments around a bit and make the assembler thunking glue a lot
    simpler.
    
    For ExecuteEbcImageEntryPoint(), this involves passing the EntryPoint
    argument as the third parameter, rather than the first, which allows
    us to do a tail call. For EbcInterpret(), instead of copying each
    argument beyond #8 from one native stack frame to the next (before
    another copy is made into the VM stack), pass a pointer to the
    argument stack.
    
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx>
    Reviewed-by: Leif Lindholm <leif.lindholm@xxxxxxxxxx>
    Reviewed-by: Feng Tian <feng.tian@xxxxxxxxx>

commit 3226e315d20c6f572de818d6d1229a88b5b6e7b3
Author: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx>
Date:   Wed Aug 17 16:29:09 2016 +0200

    MdeModulePkg/EbcDxe AARCH64: use tail call for EBC to native thunk
    
    Instead of pessimistically copying at least 64 bytes from the VM stack
    to the native stack, and popping off the register arguments again
    before doing the native call, try to avoid touching the stack completely
    if the VM stack frame is <= 64 bytes. Also, if the stack frame does exceed
    64 bytes, there is no need to copy the first 64 bytes, since we are passing
    those in registers anyway.
    
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx>
    Reviewed-by: Leif Lindholm <leif.lindholm@xxxxxxxxxx>
    Reviewed-by: Feng Tian <feng.tian@xxxxxxxxx>

commit 4d1f5a214bb3c7904c26f2634294dee2a18be5d3
Author: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx>
Date:   Wed Aug 17 16:24:52 2016 +0200

    MdeModulePkg/EbcDxe AARCH64: use a fixed size thunk structure
    
    The thunk generation is needlessly complex, given that it attempts to
    deal with variable length instructions, which don't exist on AArch64.
    
    So replace it with a simple template coded in assembler, with a matching
    struct definition in C. That way, we can create and manipulate the thunks
    easily without looping over the instructions looking for 'magic' numbers.
    
    Also, use x16 rather than x9, since it is the architectural register to
    use for thunks/veneers.
    
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx>
    Reviewed-by: Leif Lindholm <leif.lindholm@xxxxxxxxxx>
    Reviewed-by: Feng Tian <feng.tian@xxxxxxxxx>

commit 72b0eaa02679de8a0f0984a4d41ed1386262f3f3
Author: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx>
Date:   Wed Aug 17 16:08:21 2016 +0200

    MdeModulePkg/EbcDxe AARCH64: clean up comment style in ASM file
    
    Change to consistent // style comments. Also, remove bogus global
    definitions for external functions, and move the real exports to
    the top of the file.
    
    Contributed-under: TianoCore Contribution Agreement 1.0
    Signed-off-by: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx>
    Reviewed-by: Leif Lindholm <leif.lindholm@xxxxxxxxxx>
    Reviewed-by: Feng Tian <feng.tian@xxxxxxxxx>

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