[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-devel] [xen-unstable test] 22363: regressions - FAIL
flight 22363 xen-unstable real [real] http://www.chiark.greenend.org.uk/~xensrcts/logs/22363/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-winxpsp3 5 xen-boot fail REGR. vs. 22354 Regressions which are regarded as allowable (not blocking): test-amd64-i386-xl-winxpsp3-vcpus1 7 windows-install fail like 22354 Tests which did not succeed, but are not blocking: test-amd64-amd64-xl-pcipt-intel 9 guest-start fail never pass test-armhf-armhf-xl 5 xen-boot fail never pass test-amd64-i386-xend-winxpsp3 16 leak-check/check fail never pass test-amd64-amd64-xl-qemuu-win7-amd64 13 guest-stop fail never pass test-amd64-amd64-xl-qemut-winxpsp3 13 guest-stop fail never pass test-amd64-amd64-xl-win7-amd64 13 guest-stop fail never pass test-amd64-i386-xend-qemut-winxpsp3 16 leak-check/check fail never pass test-amd64-i386-xl-qemut-winxpsp3-vcpus1 13 guest-stop fail never pass test-amd64-i386-xl-qemut-win7-amd64 13 guest-stop fail never pass test-amd64-amd64-xl-qemut-win7-amd64 13 guest-stop fail never pass test-amd64-amd64-xl-winxpsp3 13 guest-stop fail never pass test-amd64-i386-xl-win7-amd64 13 guest-stop fail never pass version targeted for testing: xen d96392361cd05a66b385f0153e398128b196e480 baseline version: xen 2e213747ac49a382eeed2d493e767496482da2c8 ------------------------------------------------------------ People who touched revisions under test: Andrew Cooper <andrew.cooper3@xxxxxxxxxx> Dario Faggioli <dario.faggioli@xxxxxxxxxx> George Dunlap <george.dunlap@xxxxxxxxxxxxx> Ian Campbell <ian.campbell@xxxxxxxxxx> Ian Jackson <ian.jackson@xxxxxxxxxxxxx> Julien Grall <julien.grall@xxxxxxxxxx> Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx> Wei Liu <wei.liu2@xxxxxxxxxx> ------------------------------------------------------------ jobs: build-amd64 pass build-armhf pass build-i386 pass build-amd64-oldkern pass build-i386-oldkern pass build-amd64-pvops pass build-armhf-pvops pass build-i386-pvops pass test-amd64-amd64-xl pass test-armhf-armhf-xl fail test-amd64-i386-xl pass test-amd64-i386-rhel6hvm-amd pass test-amd64-i386-qemut-rhel6hvm-amd pass test-amd64-i386-qemuu-rhel6hvm-amd pass test-amd64-i386-freebsd10-amd64 pass 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-amd64-xl-win7-amd64 fail test-amd64-i386-xl-win7-amd64 fail test-amd64-i386-xl-credit2 pass test-amd64-i386-freebsd10-i386 pass test-amd64-amd64-xl-pcipt-intel fail test-amd64-i386-rhel6hvm-intel pass test-amd64-i386-qemut-rhel6hvm-intel pass test-amd64-i386-qemuu-rhel6hvm-intel pass test-amd64-i386-xl-multivcpu pass test-amd64-amd64-pair pass test-amd64-i386-pair pass test-amd64-amd64-xl-sedf-pin pass test-amd64-amd64-pv pass test-amd64-i386-pv pass test-amd64-amd64-xl-sedf pass test-amd64-i386-xl-qemut-winxpsp3-vcpus1 fail test-amd64-i386-xl-winxpsp3-vcpus1 fail test-amd64-i386-xend-qemut-winxpsp3 fail test-amd64-amd64-xl-qemut-winxpsp3 fail test-amd64-amd64-xl-qemuu-winxpsp3 fail test-amd64-i386-xend-winxpsp3 fail test-amd64-amd64-xl-winxpsp3 fail ------------------------------------------------------------ sg-report-flight on woking.cam.xci-test.com logs: /home/xc_osstest/logs images: /home/xc_osstest/images Logs, config files, etc. are available at http://www.chiark.greenend.org.uk/~xensrcts/logs Test harness code can be found at http://xenbits.xensource.com/gitweb?p=osstest.git;a=summary Not pushing. ------------------------------------------------------------ commit d96392361cd05a66b385f0153e398128b196e480 Author: Ian Campbell <ian.campbell@xxxxxxxxxx> Date: Mon Dec 9 12:13:48 2013 +0000 xen: arm: correct return value of raw_copy_{to/from}_guest_*, raw_clear_guest This is a generic interface which is supposed to return the number of bytes which were not copied. Make it so. Update the incorrect callers prepare_dtb, decode_thumb{2} and xenmem_add_to_physmap_range. In the xenmem_add_to_physmap_range case, observe that we are not propagating errors from xenmem_add_to_physmap_one and do so. In the decode_thumb case and an emacs magic block to decode.c Make the flush_dcache parameter to the helper an int while at it. Signed-off-by: Ian Campbell <ian.campbell@xxxxxxxxxx> Acked-by: Julien Grall <julien.grall@xxxxxxxxxx> commit 2c7a21dd65d4f4a822f20d9d9e18fba027270aa5 Author: Ian Campbell <ian.campbell@xxxxxxxxxx> Date: Fri Dec 6 14:29:32 2013 +0000 xen: arm: correct definition of DCISW (data cache invalidate by set/way) We don't actually use this but I was using it locally for debugging and it tripped me up. Also add DCCIMVAC "data cache clean and invalidate by MVA" which is the only cache op missing from cpregs.h. Signed-off-by: Ian Campbell <ian.campbell@xxxxxxxxxx> Acked-by: Julien Grall <julien.grall@xxxxxxxxxx> commit ae5d5fb58dcacaf7e2d5fbfbe7b565f401c62808 Author: Ian Campbell <ian.campbell@xxxxxxxxxx> Date: Mon Dec 9 11:43:35 2013 +0000 xen: arm: handle initrd addresses above the 4G boundary The Xgene platform has no RAM below 4G. The /chosen/linux,initrd-* properties do not have "reg" semantics and therefore #*-size are not used when interpreting. Instead they are are simply numbers which are interpreted according to the properties length. Fix this both when parsing the entry in the host DTB and when creating the dom0 DTB. For dom0 we simply hardcode a 64-bit size, this is acceptable even for a 32-bit guest. Signed-off-by: Ian Campbell <ian.campbell@xxxxxxxxxx> Acked-by: Julien Grall <julien.grall@xxxxxxxxxx> commit 90ed3c202f2c8ce286735ca235dd1fdcbd35dd8f Author: Ian Campbell <ian.campbell@xxxxxxxxxx> Date: Mon Dec 9 11:13:36 2013 +0000 xen: arm: vtimer fixes for arm64 The code was writing back the register, even for writes and didn't implement CNTPCT at all. Signed-off-by: Ian Campbell <ian.campbell@xxxxxxxxxx> Acked-by: Julien Grall <julien.grall@xxxxxxxxxx> Acked-by: Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx> commit 9ed46ae6d29e5e87b45127d446e05db119c1b464 Author: Ian Campbell <ian.campbell@xxxxxxxxxx> Date: Mon Dec 9 11:09:10 2013 +0000 xen: arm: do not BUG on guest paddrs which are very high The BUG_ON in p2m_map_first was over aggressive since the paddr_t can have come from the guest, via add_to_physmap. Instead return failure to the caller. Also the check was simultaneously too lose. The valid offsets are 0..P2M_FIRST_ENTRIES-1 inclusive. Signed-off-by: Ian Campbell <ian.campbell@xxxxxxxxxx> Acked-by: Julien Grall <julien.grall@xxxxxxxxxx> commit cc0baa5cc313522282976cf291c809d644c0e644 Author: Wei Liu <wei.liu2@xxxxxxxxxx> Date: Sun Dec 8 20:50:20 2013 +0000 Config.mk: update OVMF changeset Signed-off-by: Wei Liu <wei.liu2@xxxxxxxxxx> Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx> Cc: Ian Jackson <ian.jackson@xxxxxxxxxxxxx> Cc: George Dunlap <george.dunlap@xxxxxxxxxxxxx> commit caee9a0362b198636410793e091ba901ddca9f2c Author: Dario Faggioli <dario.faggioli@xxxxxxxxxx> Date: Sat Dec 7 01:05:34 2013 +0100 xl: test script for the cpumap parser (for vCPU pinning) This commit introduces "check-xl-vcpupin-parse" for helping verifying and debugging the (v)CPU bitmap parsing code in xl. The script runs "xl -N vcpu-pin 0 all <some strings>" repeatedly, with various input strings, and checks that the output is as expected. This is what the script can do: # ./check-xl-vcpupin-parse -h usage: ./check-xl-vcpupin-parse [options] Tests various vcpu-pinning strings. If run without arguments acts as follows: - generates some test data and saves them in check-xl-vcpupin-parse.data; - tests all the generated configurations (reading them back from check-xl-vcpupin-parse.data). An example of a test vector file is provided in check-xl-vcpupin-parse.data-example. Options: -h prints this message -r seed uses seed for initializing the rundom number generator (default: the script PID) -s string tries using string as a vcpu pinning configuration and reports whether that succeeds or not -o ofile save the test data in ofile (default: check-xl-vcpupin-parse.data) -i ifile read test data from ifile An example test data file (generated on a 2 NUMA nodes, 16 CPUs host) is being provided in check-xl-vcpupin-parse.data-example. Signed-off-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx> Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx> commit 62c562557a8a52612fa4ad960a285f5cd618611e Author: Dario Faggioli <dario.faggioli@xxxxxxxxxx> Date: Sat Dec 7 01:05:26 2013 +0100 xl: implement and enable dryrun mode for `xl vcpu-pin' As it can be useful to see if the outcome of some complex vCPU pinning bitmap specification looks as expected. This also allow for the introduction of some automatic testing and verification for the bitmap parsing code, as it happens already in check-xl-disk-parse and check-xl-vif-parse. In particular, to make the above possible, this commit also changes the implementation of the vcpu-pin command so that, instead of always returning 0, it returns an error if the parsing fails. Signed-off-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx> Acked-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx> commit 23772b35cf1d18c830bc28b28b55eccbcf8e24be Author: Dario Faggioli <dario.faggioli@xxxxxxxxxx> Date: Sat Dec 7 01:05:18 2013 +0100 xl: allow for node-wise specification of vcpu pinning Making it possible to use something like the following: * "nodes:0-3": all pCPUs of nodes 0,1,2,3; * "nodes:0-3,^node:2": all pCPUS of nodes 0,1,3; * "1,nodes:1-2,^6": pCPU 1 plus all pCPUs of nodes 1,2 but not pCPU 6; * ... In both domain config file and `xl vcpu-pin'. Signed-off-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx> Reviewed-by: George Dunlap <george.dunlap@xxxxxxxxxxxxx> Acked-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx> commit 62f25a16a47e62b2abbabd3c24c30c2545c5e312 Author: Dario Faggioli <dario.faggioli@xxxxxxxxxx> Date: Sat Dec 7 01:05:11 2013 +0100 libxc/libxl: allow to retrieve the number of online pCPUs by introducing introduce xc_get_online_cpus() and libxl_get_online_cpus(). Signed-off-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx> Acked-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx> commit 2e82c18cd850592ae9a1f682eb93965a868b5f2f Author: Dario Faggioli <dario.faggioli@xxxxxxxxxx> Date: Sat Dec 7 01:05:03 2013 +0100 libxc/libxl: sanitize error handling in *_get_max_{cpus, nodes} In libxc, make xc_get_max_{cpus,node}() always return either a positive number or -1, and change all the callers to deal with that. In libxl, make libxl_get_max_{cpus,nodes}() always return either a positive number or a libxl error code. Thanks to that, it is also possible to fix loggig for libxl_{cpu,node}_bitmap_alloc(), which now happens inside the functions themselves, more accurately reporting what happened. Signed-off-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx> Acked-by: George Dunlap <george.dunlap@xxxxxxxxxxxxx> Acked-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx> commit 5ad07f45dae9d74be8d300aa5fda31cd11c10061 Author: Dario Faggioli <dario.faggioli@xxxxxxxxxx> Date: Sat Dec 7 01:04:55 2013 +0100 libxl: move libxl_{cpu, node}_bitmap_alloc() in libxl_utils.c (from .h), as they will be reworked in the next commit ("libxc/libxl: sanitize error handling in *_get_max_{cpus,nodes}") and we want to keep code motion separate from functional changes. Signed-off-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx> Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx> commit 2217a4318ff156ad86c377fc0f13f0fc6406123e Author: Dario Faggioli <dario.faggioli@xxxxxxxxxx> Date: Sat Dec 7 01:04:48 2013 +0100 libxl: fix memory leak in libxl_list_vcpu more specifically, of the cpumap inside libxl_vcpuinfo, in case of failure after it has been allocated. While at it, use the correct libxl memory allocation wrapper for calloc() in there and turn the function into using the new LOGE() logging style. Signed-off-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx> Acked-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx> commit c42fa088ec5deb9de3263ef6b7e3115f75e919e9 Author: Dario Faggioli <dario.faggioli@xxxxxxxxxx> Date: Sat Dec 7 01:04:40 2013 +0100 libxl: better name for parameters in libxl_list_vcpu so that the parameter that returns the number of pCPUs is called nr_cpus_out, while the one that returns the number of vCPUs is called nr_vcpus_out. The patch is all about renaming, so no functional change. Signed-off-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx> Acked-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx> commit 700810c7c845a6bb0c15351e5d1c3c60fc74f6fc Author: Dario Faggioli <dario.faggioli@xxxxxxxxxx> Date: Sat Dec 7 01:04:32 2013 +0100 xl: match output of vcpu-list with pinning syntax in fact, pinning to all the pcpus happens by specifying "all" (either on the command line or in the config file), while `xl vcpu-list' report it as "any cpu". Change this into something more consistent, by using "all" everywhere. Signed-off-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx> Acked-by: George Dunlap <george.dunlap@xxxxxxxxxxxxx> Acked-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx> commit 7dd4f9da063cb2cd43426c785535534c9d958ce5 Author: Andrew Cooper <andrew.cooper3@xxxxxxxxxx> Date: Mon Dec 9 14:13:23 2013 +0100 defer the domain mapping in scrub_one_page() This avoids a resource leak and needless playing with the pagetables in the case that the page is broken. Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx> Reviewed-by: Jan Beulich <JBeulich@xxxxxxxx> Reviewed-by: Keir Fraser <keir@xxxxxxx> (qemu changes not included) _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |