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

[Xen-devel] [xen-unstable baseline-only test] 66616: regressions - FAIL



This run is configured for baseline tests only.

flight 66616 xen-unstable real [real]
http://osstest.xs.citrite.net/~osstest/testlogs/logs/66616/

Regressions :-(

Tests which did not succeed and are blocking,
including tests which could not be run:
 test-amd64-amd64-xl          11 guest-start               fail REGR. vs. 66611
 test-armhf-armhf-xl-vhd      14 guest-start/debian.repeat fail REGR. vs. 66611

Regressions which are regarded as allowable (not blocking):
 build-amd64-rumpuserxen       6 xen-build                    fail   like 66611
 build-i386-rumpuserxen        6 xen-build                    fail   like 66611
 test-amd64-i386-xl-qemuu-win7-amd64 16 guest-stop              fail like 66611
 test-amd64-amd64-i386-pvgrub 10 guest-start                  fail   like 66611
 test-amd64-amd64-qemuu-nested-intel 16 debian-hvm-install/l1/l2 fail like 66611

Tests which did not succeed, but are not blocking:
 test-amd64-amd64-rumpuserxen-amd64  1 build-check(1)               blocked n/a
 test-amd64-i386-rumpuserxen-i386  1 build-check(1)               blocked  n/a
 test-armhf-armhf-libvirt-xsm 12 migrate-support-check        fail   never pass
 test-armhf-armhf-libvirt-xsm 14 guest-saverestore            fail   never pass
 test-armhf-armhf-libvirt     14 guest-saverestore            fail   never pass
 test-armhf-armhf-libvirt     12 migrate-support-check        fail   never pass
 test-amd64-amd64-xl-pvh-amd  11 guest-start                  fail   never pass
 test-amd64-amd64-xl-pvh-intel 11 guest-start                  fail  never pass
 test-armhf-armhf-libvirt-qcow2 11 migrate-support-check        fail never pass
 test-armhf-armhf-libvirt-qcow2 13 guest-saverestore            fail never pass
 test-armhf-armhf-xl          12 migrate-support-check        fail   never pass
 test-armhf-armhf-xl          13 saverestore-support-check    fail   never pass
 test-armhf-armhf-xl-midway   13 saverestore-support-check    fail   never pass
 test-armhf-armhf-xl-multivcpu 13 saverestore-support-check    fail  never pass
 test-armhf-armhf-xl-midway   12 migrate-support-check        fail   never pass
 test-armhf-armhf-xl-multivcpu 12 migrate-support-check        fail  never pass
 test-armhf-armhf-xl-xsm      13 saverestore-support-check    fail   never pass
 test-armhf-armhf-xl-xsm      12 migrate-support-check        fail   never pass
 test-armhf-armhf-libvirt-raw 13 guest-saverestore            fail   never pass
 test-armhf-armhf-libvirt-raw 11 migrate-support-check        fail   never pass
 test-armhf-armhf-xl-credit2  13 saverestore-support-check    fail   never pass
 test-armhf-armhf-xl-credit2  12 migrate-support-check        fail   never pass
 test-armhf-armhf-xl-rtds     13 saverestore-support-check    fail   never pass
 test-armhf-armhf-xl-rtds     12 migrate-support-check        fail   never pass
 test-amd64-amd64-libvirt-xsm 12 migrate-support-check        fail   never pass
 test-amd64-amd64-libvirt     12 migrate-support-check        fail   never pass
 test-amd64-i386-libvirt-xsm  12 migrate-support-check        fail   never pass
 test-amd64-i386-libvirt      12 migrate-support-check        fail   never pass
 test-armhf-armhf-xl-vhd      11 migrate-support-check        fail   never pass
 test-armhf-armhf-xl-vhd      12 saverestore-support-check    fail   never pass
 test-amd64-amd64-qemuu-nested-amd 16 debian-hvm-install/l1/l2  fail never pass
 test-amd64-amd64-xl-qemut-win7-amd64 16 guest-stop             fail never pass
 test-amd64-amd64-xl-qemuu-win7-amd64 16 guest-stop             fail never pass
 test-amd64-i386-xl-qemut-win7-amd64 16 guest-stop              fail never pass
 test-amd64-amd64-libvirt-qemuu-debianhvm-amd64-xsm 10 migrate-support-check 
fail never pass
 test-amd64-i386-libvirt-qemuu-debianhvm-amd64-xsm 10 migrate-support-check 
fail never pass
 test-amd64-amd64-libvirt-vhd 11 migrate-support-check        fail   never pass

version targeted for testing:
 xen                  e763268781d341fef05d461f3057e6ced5e033f2
baseline version:
 xen                  b48be35ac86cd6369124cf06ca3006d086095297

Last test of basis    66611  2016-07-16 17:19:41 Z    3 days
Testing same since    66616  2016-07-19 15:16:34 Z    0 days    1 attempts

------------------------------------------------------------
People who touched revisions under test:
  Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
  Dario Faggioli <dario.faggioli@xxxxxxxxxx>
  George Dunlap <george.dunlap@xxxxxxxxxx>
  Wei Liu <wei.liu2@xxxxxxxxxx>

jobs:
 build-amd64-xsm                                              pass    
 build-armhf-xsm                                              pass    
 build-i386-xsm                                               pass    
 build-amd64                                                  pass    
 build-armhf                                                  pass    
 build-i386                                                   pass    
 build-amd64-libvirt                                          pass    
 build-armhf-libvirt                                          pass    
 build-i386-libvirt                                           pass    
 build-amd64-oldkern                                          pass    
 build-i386-oldkern                                           pass    
 build-amd64-prev                                             pass    
 build-i386-prev                                              pass    
 build-amd64-pvops                                            pass    
 build-armhf-pvops                                            pass    
 build-i386-pvops                                             pass    
 build-amd64-rumpuserxen                                      fail    
 build-i386-rumpuserxen                                       fail    
 test-amd64-amd64-xl                                          fail    
 test-armhf-armhf-xl                                          pass    
 test-amd64-i386-xl                                           pass    
 test-amd64-amd64-xl-qemut-debianhvm-amd64-xsm                pass    
 test-amd64-i386-xl-qemut-debianhvm-amd64-xsm                 pass    
 test-amd64-amd64-libvirt-qemuu-debianhvm-amd64-xsm           pass    
 test-amd64-i386-libvirt-qemuu-debianhvm-amd64-xsm            pass    
 test-amd64-amd64-xl-qemuu-debianhvm-amd64-xsm                pass    
 test-amd64-i386-xl-qemuu-debianhvm-amd64-xsm                 pass    
 test-amd64-amd64-xl-qemut-stubdom-debianhvm-amd64-xsm        pass    
 test-amd64-i386-xl-qemut-stubdom-debianhvm-amd64-xsm         pass    
 test-amd64-amd64-libvirt-xsm                                 pass    
 test-armhf-armhf-libvirt-xsm                                 fail    
 test-amd64-i386-libvirt-xsm                                  pass    
 test-amd64-amd64-xl-xsm                                      pass    
 test-armhf-armhf-xl-xsm                                      pass    
 test-amd64-i386-xl-xsm                                       pass    
 test-amd64-amd64-qemuu-nested-amd                            fail    
 test-amd64-amd64-xl-pvh-amd                                  fail    
 test-amd64-i386-qemut-rhel6hvm-amd                           pass    
 test-amd64-i386-qemuu-rhel6hvm-amd                           pass    
 test-amd64-amd64-xl-qemut-debianhvm-amd64                    pass    
 test-amd64-i386-xl-qemut-debianhvm-amd64                     pass    
 test-amd64-amd64-xl-qemuu-debianhvm-amd64                    pass    
 test-amd64-i386-xl-qemuu-debianhvm-amd64                     pass    
 test-amd64-i386-freebsd10-amd64                              pass    
 test-amd64-amd64-xl-qemuu-ovmf-amd64                         pass    
 test-amd64-i386-xl-qemuu-ovmf-amd64                          pass    
 test-amd64-amd64-rumpuserxen-amd64                           blocked 
 test-amd64-amd64-xl-qemut-win7-amd64                         fail    
 test-amd64-i386-xl-qemut-win7-amd64                          fail    
 test-amd64-amd64-xl-qemuu-win7-amd64                         fail    
 test-amd64-i386-xl-qemuu-win7-amd64                          fail    
 test-amd64-amd64-xl-credit2                                  pass    
 test-armhf-armhf-xl-credit2                                  pass    
 test-amd64-i386-freebsd10-i386                               pass    
 test-amd64-i386-rumpuserxen-i386                             blocked 
 test-amd64-amd64-qemuu-nested-intel                          fail    
 test-amd64-amd64-xl-pvh-intel                                fail    
 test-amd64-i386-qemut-rhel6hvm-intel                         pass    
 test-amd64-i386-qemuu-rhel6hvm-intel                         pass    
 test-amd64-amd64-libvirt                                     pass    
 test-armhf-armhf-libvirt                                     fail    
 test-amd64-i386-libvirt                                      pass    
 test-armhf-armhf-xl-midway                                   pass    
 test-amd64-amd64-migrupgrade                                 pass    
 test-amd64-i386-migrupgrade                                  pass    
 test-amd64-amd64-xl-multivcpu                                pass    
 test-armhf-armhf-xl-multivcpu                                pass    
 test-amd64-amd64-pair                                        pass    
 test-amd64-i386-pair                                         pass    
 test-amd64-amd64-libvirt-pair                                pass    
 test-amd64-i386-libvirt-pair                                 pass    
 test-amd64-amd64-amd64-pvgrub                                pass    
 test-amd64-amd64-i386-pvgrub                                 fail    
 test-amd64-amd64-pygrub                                      pass    
 test-armhf-armhf-libvirt-qcow2                               fail    
 test-amd64-amd64-xl-qcow2                                    pass    
 test-armhf-armhf-libvirt-raw                                 fail    
 test-amd64-i386-xl-raw                                       pass    
 test-amd64-amd64-xl-rtds                                     pass    
 test-armhf-armhf-xl-rtds                                     pass    
 test-amd64-i386-xl-qemut-winxpsp3-vcpus1                     pass    
 test-amd64-i386-xl-qemuu-winxpsp3-vcpus1                     pass    
 test-amd64-amd64-libvirt-vhd                                 pass    
 test-armhf-armhf-xl-vhd                                      fail    
 test-amd64-amd64-xl-qemut-winxpsp3                           pass    
 test-amd64-i386-xl-qemut-winxpsp3                            pass    
 test-amd64-amd64-xl-qemuu-winxpsp3                           pass    
 test-amd64-i386-xl-qemuu-winxpsp3                            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 e763268781d341fef05d461f3057e6ced5e033f2
Author: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
Date:   Fri Jul 15 15:36:07 2016 +0000

    xen/x86: Identify legitimate fallthrough cases
    
    The case in arch_set_info_guest() is a legitimate fallthrough.  Mark it as 
such.
    
    The cases in vlapic_accept_irq() are a terminal error path, but Coverity 
fails
    to spot this.  Reorder the comment to the end.
    
    No functional change, but fixes two MISSING_BREAK Coverity defects.
    
    Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
    Reviewed-by: George Dunlap <george.dunlap@xxxxxxxxxx>

commit e3566c190cfd3789aeea5e2273f5f8888ca8ec2f
Author: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
Date:   Fri Jul 15 16:50:18 2016 +0200

    xen: credit2: the private scheduler lock can be an rwlock.
    
    In fact, the data it protects only change either at init-time,
    during cpupools manipulation, or when changing domains' weights.
    In all other cases (namely, load balancing, reading weights
    and status dumping), information is only read.
    
    Therefore, let the lock be an read/write one. This means there
    is no full serialization point for the whole scheduler and
    for all the pCPUs of the host any longer.
    
    This is particularly good for scalability (especially when doing
    load balancing).
    
    Also, update the high level description of the locking discipline,
    and take the chance for rewording it a little bit (as well as
    for adding a couple of locking related ASSERT()-s).
    
    Signed-off-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
    Reviewed-by: George Dunlap <george.dunlap@xxxxxxxxxx>

commit d6baf64c890c7d55002e1213dffa715ea1cf6300
Author: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
Date:   Fri Jul 15 16:50:11 2016 +0200

    tools: tracing: deal with new Credit2 events
    
    more specifically, with: TICKLE_NEW, RUNQ_MAX_WEIGHT,
    MIGRATE, LOAD_CHECK, LOAD_BALANCE and PICKED_CPU, and
    in both both xenalyze and formats (for xentrace_format).
    
    Signed-off-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
    Acked-by: George Dunlap <george.dunlap@xxxxxxxxxx>
    Acked-by: Wei Liu <wei.liu2@xxxxxxxxxx>

commit f74d5d25916d772cdaf19138ac5b040ae607bafd
Author: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
Date:   Fri Jul 15 16:50:04 2016 +0200

    xen: credit2: only marshall trace point arguments if tracing enabled
    
    Signed-off-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
    Reviewed-by: George Dunlap <george.dunlap@xxxxxxxxxx>

commit da7fdee5aeb276fd4e581ade0285140c78143cab
Author: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
Date:   Fri Jul 15 16:49:56 2016 +0200

    xen: credit2: add yet some more tracing
    
    (and fix the style of two labels as well.)
    
    Signed-off-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
    Acked-by: George Dunlap <george.dunlap@xxxxxxxxxx>

commit ccf2ead7f525877597768cdeba4cae0ac506aad9
Author: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
Date:   Fri Jul 15 16:49:49 2016 +0200

    xen: credit2: make the code less experimental
    
    Mainly, almost all of the BUG_ON-s can be converted into
    ASSERTS, and almost all the debug printk can either be
    removed or turned into tracing.
    
    The 'TODO' list, in a comment at the beginning of the file,
    was also stale, so remove items that were still there but
    are actually done.
    
    Signed-off-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
    Reviewed-by: George Dunlap <george.dunlap@xxxxxxxxxx>

commit 222234f2ad171854516b78b476931a3fd784b6ae
Author: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
Date:   Fri Jul 15 16:49:40 2016 +0200

    xen: credit2: use non-atomic cpumask and bit operations
    
    as all the accesses to both the masks and the flags are
    serialized by the runqueues locks already.
    
    Signed-off-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
    Acked-by: George Dunlap <george.dunlap@xxxxxxxxxx>

commit f10d19329a3caba200e4d20f17cadc9cf509eba3
Author: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
Date:   Fri Jul 15 16:49:33 2016 +0200

    xen/tools: improve tracing of Credit2 load tracking events
    
    Add the shift used for the precision of the integer
    arithmetic to the trace records, and update both xenalyze
    and xentrace_format to make use of/print it.
    
    In particular, in xenalyze, we are can now show the
    load as a (easier to interpreet) percentage.
    
    Signed-off-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
    Reviewed-by: George Dunlap <george.dunlap@xxxxxxxxxx>
    Acked-by: Wei Liu <wei.liu2@xxxxxxxxxx>

commit d205f8a7f48e2ec173806599a6ab1c9002a7c4b0
Author: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
Date:   Fri Jul 15 16:49:26 2016 +0200

    xen: credit2: rework load tracking logic
    
    The existing load tracking code was hard to understad and
    maintain, and not entirely consistent. This is due to a
    number of reasons:
     - code and comments were not in perfect sync, making it
       difficult to figure out what the intent of a particular
       choice was (e.g., the choice of 18 for load_window_shift);
     - the math, although effective, was not entirely consistent.
       In fact, we were doing (if W is the lenght of the window):
    
        avgload = (delta*load*W + (W - delta)*avgload)/W
        avgload = avgload + delta*load - delta*avgload/W
    
       which does not match any known variant of 'smoothing
       moving average'. In fact, it should have been:
    
        avgload = avgload + delta*load/W - delta*avgload/W
    
       (for details on why, see the doc comments inside this
       patch.). Furthermore, with
    
        avgload ~= avgload + W*load - avgload
        avgload ~= W*load
    
    The reason why the formula above sort of worked was because
    the number of bits used for the fractional parts of the
    values used in fixed point math and the number of bits used
    for the lenght of the window were the same (load_window_shift
    was being used for both).
    
    This may look handy, but it introduced a (not especially well
    documented) dependency between the lenght of the window and
    the precision of the calculations, which really should be
    two independent things. Especially if treating them as such
    (like it is done in this patch) does not lead to more
    complex maths (same number of multiplications and shifts, and
    there is still room for some optimization).
    
    Therefore, in this patch, we:
     - split length of the window and precision (and, since there
       is already a command line parameter for length of window,
       introduce one for precision too),
     - align the math with one proper incarnation of exponential
       smoothing (at no added cost),
     - add comments, about the details of the algorithm and the
       math used.
    
    While there fix a couple of style issues as well (pointless
    initialization, long lines, comments).
    
    Signed-off-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
    Reviewed-by: George Dunlap <george.dunlap@xxxxxxxxxx>

commit 11eadabc34625337f7556e9d74f638229f889fd8
Author: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
Date:   Fri Jul 15 16:49:18 2016 +0200

    xen: credit2: prevent load balancing to go mad if time goes backwards
    
    This really should not happen, but:
     1. it does happen! Some more info here:
        http://lists.xen.org/archives/html/xen-devel/2016-06/msg00922.html
     2. independently from 1, it makes sense and is easy enough
        to have a 'safety catch'.
    
    The reason why this is particularly bad for Credit2 is that
    negative values of delta mean out of scale high load (because
    of the conversion to unsigned). This, for instance in the
    case of runqueue load, results in a runqueue having its load
    updated to values of the order of 10000% or so, which in turns
    means that the load balancer will migrate everything off from
    the pCPUs in the runqueue, and leave them idle until the load
    gets back to something sane... which may indeed take a while!
    
    This is not a fix for the problem of time going backwards. In
    fact, if that happens a lot, load tracking accuracy is still
    compromized, but at least the effect is a lot less bad than
    before.
    
    Signed-off-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
    Reviewed-by: George Dunlap <george.dunlap@xxxxxxxxxx>

commit b047f888d4896d0e6cca10fbc996d4f1e1d769b2
Author: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
Date:   Fri Jul 15 16:49:11 2016 +0200

    xen: sched: leave CPUs doing tasklet work alone.
    
    In both Credit1 and Credit2, stop considering a pCPU idle,
    if the reason why the idle vCPU is being selected, is to
    do tasklet work.
    
    Not doing so means that the tickling and load balancing
    logic, seeing the pCPU as idle, considers it a candidate
    for picking up vCPUs. But the pCPU won't actually pick
    up or schedule any vCPU, which would then remain in the
    runqueue, which is bad, especially if there were other,
    truly idle pCPUs, that could execute it.
    
    The only drawback is that we can't assume that a pCPU is
    in always marked as idle when being removed from an
    instance of the Credit2 scheduler (csched2_deinit_pdata).
    In fact, if we are in stop-machine (i.e., during suspend
    or shutdown), the pCPUs are running the stopmachine_tasklet
    and hence are actually marked as busy. On the other hand,
    when removing a pCPU from a Credit2 pool, it will indeed
    be idle. The only thing we can do, therefore, is to
    remove the BUG_ON() check.
    
    Signed-off-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
    Reviewed-by: George Dunlap <george.dunlap@xxxxxxxxxx>
(qemu changes not included)

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

 


Rackspace

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