|
[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
|
 |
Lists.xenproject.org is hosted with RackSpace, monitoring our |