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

[Xen-devel] [xen-unstable-smoke test] 136687: regressions - FAIL



flight 136687 xen-unstable-smoke real [real]
http://logs.test-lab.xenproject.org/osstest/logs/136687/

Regressions :-(

Tests which did not succeed and are blocking,
including tests which could not be run:
 build-arm64-xsm               6 xen-build                fail REGR. vs. 136665

Tests which did not succeed, but are not blocking:
 test-arm64-arm64-xl-xsm       1 build-check(1)               blocked  n/a
 test-amd64-amd64-libvirt     13 migrate-support-check        fail   never pass
 test-armhf-armhf-xl          13 migrate-support-check        fail   never pass
 test-armhf-armhf-xl          14 saverestore-support-check    fail   never pass

version targeted for testing:
 xen                  b915f57a49bc12e9f5fb60ce604772b09777ff0d
baseline version:
 xen                  91f86f8634f99abd8f242943e62452211a09fa0a

Last test of basis   136665  2019-05-21 10:00:39 Z    0 days
Testing same since   136687  2019-05-21 13:03:27 Z    0 days    1 attempts

------------------------------------------------------------
People who touched revisions under test:
  Alistair Francis <alistair.francis@xxxxxxx>
  Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
  Jan Beulich <jbeulich@xxxxxxxx>
  Julien Grall <julien.grall@xxxxxxx>
  Stefano Stabellini <sstabellini@xxxxxxxxxx>
  Wei Liu <wei.liu2@xxxxxxxxxx>

jobs:
 build-arm64-xsm                                              fail    
 build-amd64                                                  pass    
 build-armhf                                                  pass    
 build-amd64-libvirt                                          pass    
 test-armhf-armhf-xl                                          pass    
 test-arm64-arm64-xl-xsm                                      blocked 
 test-amd64-amd64-xl-qemuu-debianhvm-amd64                    pass    
 test-amd64-amd64-libvirt                                     pass    


------------------------------------------------------------
sg-report-flight on osstest.test-lab.xenproject.org
logs: /home/logs/logs
images: /home/logs/images

Logs, config files, etc. are available at
    http://logs.test-lab.xenproject.org/osstest/logs

Explanation of these reports, and of osstest in general, is at
    http://xenbits.xen.org/gitweb/?p=osstest.git;a=blob;f=README.email;hb=master
    http://xenbits.xen.org/gitweb/?p=osstest.git;a=blob;f=README;hb=master

Test harness code can be found at
    http://xenbits.xen.org/gitweb?p=osstest.git;a=summary


Not pushing.

------------------------------------------------------------
commit b915f57a49bc12e9f5fb60ce604772b09777ff0d
Author: Julien Grall <julien.grall@xxxxxxx>
Date:   Tue May 7 16:14:46 2019 +0100

    xen/x86: Constify the parameter "d" in mfn_to_gfn
    
    The parameter "d" holds the domain and is not modified by the function.
    So constify it.
    
    Signed-off-by: Julien Grall <julien.grall@xxxxxxx>
    Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>

commit f582878732ebaa43e50872e1b286131301b9b63b
Author: Julien Grall <julien.grall@xxxxxxx>
Date:   Tue May 7 16:14:45 2019 +0100

    xen/arm: Use mfn_to_pdx instead of pfn_to_pdx when possible
    
    mfn_to_pdx adds more safety than pfn_to_pdx. Replace all but on place in
    the Arm code to use the former.
    
    No functional changes.
    
    Signed-off-by: Julien Grall <julien.grall@xxxxxxx>
    Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>

commit 8603d4a284f9f760217b74b5d91eb9439f9d9810
Author: Julien Grall <julien.grall@xxxxxxx>
Date:   Tue May 14 13:24:40 2019 +0100

    xen/arm: processor: Use BIT(.., UL) instead of _AC(1, U) in SCTLR_ defines
    
    Use the pattern BIT(..., UL) to make the code more readable. Note that
    unsigned long is used instead of unsigned because SCTLR is technically
    32-bit on Arm32 and 64-bit on Arm64.
    
    Signed-off-by: Julien Grall <julien.grall@xxxxxxx>
    Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>

commit 40d82802137385e841ab16b627f889f247c9cba2
Author: Julien Grall <julien.grall@xxxxxxx>
Date:   Tue May 14 13:24:39 2019 +0100

    xen/arm: Rename SCTLR_* defines and remove unused one
    
    The SCTLR_* are currently used for SCTLR/HSCTLR (arm32) and
    SCTLR_EL1/SCTLR_EL2 (arm64).
    
    The naming scheme is actually quite confusing because they may only be
    defined for an archicture (or even an exception level). So it is not easy
    for the developer to know which one to use.
    
    The naming scheme is reworked by adding Axx_ELx in each define:
        * xx is replaced by 32 or 64 if specific to an architecture
        * x is replaced by 2 (hypervisor) or 1 (kernel) if specific to an
        exception level
    
    While doing the renaming, remove the unused defines (or at least the ones
    that are unlikely going to be used).
    
    Signed-off-by: Julien Grall <julien.grall@xxxxxxx>
    Reviewed-by: Andrii Anisov <andrii_anisov@xxxxxxxx>
    Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>

commit a4d85a9287475738f400e4cd87750a37298c3948
Author: Julien Grall <julien.grall@xxxxxxx>
Date:   Tue May 14 13:11:28 2019 +0100

    xen/arm: tlbflush: Clarify the TLB helpers name
    
    TLB helpers in the headers tlbflush.h are currently quite confusing to
    use the name may lead to think they are dealing with hypervisors TLBs
    while they actually deal with guest TLBs.
    
    Rename them to make it clearer that we are dealing with guest TLBs.
    
    Signed-off-by: Julien Grall <julien.grall@xxxxxxx>
    Reviewed-by: Andrii Anisov <andrii_anisov@xxxxxxxx>
    Acked-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>

commit 7ad0e780857728724e59859dcc422404d4ed0c46
Author: Julien Grall <julien.grall@xxxxxxx>
Date:   Tue May 14 13:11:27 2019 +0100

    xen/arm: Remove flush_xen_text_tlb_local()
    
    The function flush_xen_text_tlb_local() has been misused and will result
    to invalidate the instruction cache more than necessary.
    
    For instance, there is no need to invalidate the instruction cache if
    we are setting SCTLR_EL2.WXN.
    
    There is effectively only one caller (i.e free_init_memory() who would
    need to invalidate the instruction cache.
    
    So rather than keeping around the function flush_xen_text_tlb_local()
    replace it with call to flush_xen_tlb_local() and explicitly flush
    the cache when necessary.
    
    Signed-off-by: Julien Grall <julien.grall@xxxxxxx>
    Reviewed-by: Andrii Anisov <andrii_anisov@xxxxxxxx>
    Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>

commit 505f1f0b19a1be0e2623aa6028db87b57e283f5a
Author: Julien Grall <julien.grall@xxxxxxx>
Date:   Tue May 14 13:11:26 2019 +0100

    xen/arm: mm: Consolidate setting SCTLR_EL2.WXN in a single place
    
    The logic to set SCTLR_EL2.WXN is the same for the boot CPU and
    non-boot CPU. So introduce a function to set the bit and clear TLBs.
    
    This new function will help us to document and update the logic in a
    single place.
    
    Signed-off-by: Julien Grall <julien.grall@xxxxxxx>
    Reviewed-by: Andrii Anisov <andrii_anisov@xxxxxxxx>
    Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>

commit 03957f58db8942d61f4889b6924e39d3b27a9e43
Author: Julien Grall <julien.grall@xxxxxxx>
Date:   Tue May 14 13:24:38 2019 +0100

    xen/const: Extend the existing macro BIT to take a suffix in parameter
    
    Arm currently provides two macro BIT and BIT_ULL that are only usable
    in C and return respectively unsigned long and unsigned long long.
    
    Extending the macros to deal with assembly would be a nice benefits as
    it could replace the common pattern to define fields (AC(1, sfx) << X)
    easier to read.
    
    Rather than extending the two macros, it was decided to drop BIT_ULL()
    and extend the macro BIT() to take a suffix (e.g U, UL, ULL) in
    parameter. This would allow to use different suffix without having to
    define new macros.
    
    The new extend macro is now moved in include/xen/const.h so it can be
    used by anyone in Xen and also avoid to include bitops.h in assembly
    code.
    
    Signed-off-by: Julien Grall <julien.grall@xxxxxxx>
    Acked-by: Jan Beulich <jbeulich@xxxxxxxx>
    Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>

commit 57d87ee3a5d10cdba972eec3a54cd971fec1b8d2
Author: Alistair Francis <alistair.francis@xxxxxxx>
Date:   Fri May 17 15:31:51 2019 -0700

    autotools: Updage config.guess and config.sub
    
    The autoconf manual [1] specifies that as we define AC_CANONICAL_HOST we
    must supply config.guess and config.sub. In which case let's update them
    from [2] commit: b98424c24 "config.guess: Remove space after "#endif", as
    Gnulib and some"
    
    This allows us to support more achitectures (RISC-V) and other general
    improvements.
    
    1: https://www.gnu.org/software/autoconf/manual/autoconf.html#Canonicalizing
    2: https://git.savannah.gnu.org/cgit/config.git/
    
    Signed-off-by: Alistair Francis <alistair.francis@xxxxxxx>
    Acked-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
    Acked-by: Wei Liu <wei.liu2@xxxxxxxxxx>
(qemu changes not included)

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