flight 22090 xen-unstable real [real] http://www.chiark.greenend.org.uk/~xensrcts/logs/22090/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-qemut-rhel6hvm-amd 9 guest-start.2 fail REGR. vs. 22084 test-amd64-i386-xl-qemut-winxpsp3-vcpus1 5 xen-boot fail REGR. vs. 22084 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-win7-amd64 13 guest-stop fail never pass test-amd64-amd64-xl-qemut-win7-amd64 13 guest-stop fail never pass test-amd64-i386-xl-win7-amd64 13 guest-stop fail never pass test-amd64-amd64-xl-qemuu-winxpsp3 13 guest-stop fail never pass test-amd64-i386-xl-qemut-win7-amd64 13 guest-stop fail never pass test-amd64-amd64-xl-qemut-winxpsp3 13 guest-stop fail never pass test-amd64-amd64-xl-winxpsp3 13 guest-stop fail never pass test-amd64-i386-xend-qemut-winxpsp3 16 leak-check/check fail never pass test-amd64-i386-xl-winxpsp3-vcpus1 13 guest-stop fail never pass version targeted for testing: xen 9bc72c626512d7d79a7f0295fbfa7cd597ce620a baseline version: xen b18e2d96e785e7088f3b137e800f987bbe8c7657 ------------------------------------------------------------ People who touched revisions under test: Andrew Cooper <andrew.cooper3@citrix.com> Bob Liu <bob.liu@oracle.com> David Vrabel <david.vrabel@citrix.com> Ian Campbell <ian.campbell@citrix.com> Julien Grall <julien.grall@linaro.org> Keir Fraser <keir@xen.org> Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> Stefano Stabellini <stefano.stabellini@eu.citrix.com> Tomasz Wroblewski <tomasz.wroblewski@citrix.com> ------------------------------------------------------------ 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 fail test-amd64-i386-qemuu-rhel6hvm-amd 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-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 9bc72c626512d7d79a7f0295fbfa7cd597ce620a Author: Ian Campbell <ian.campbell@citrix.com> Date: Fri Nov 22 16:24:27 2013 +0000 xen: arm: early logging of command line Helpful for diagnosis of bad console= parameters. Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Acked-by: Julien Grall <julien.grall@linaro.org> Acked-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com> commit 9280f514c75e0ce5f470d2df8f1cd53ce5eabb16 Author: Ian Campbell <ian.campbell@citrix.com> Date: Fri Nov 22 15:25:18 2013 +0000 xen: arm: correct name of the dt node passed around when building dom0 DT In the case of the GIC, timer and write_properties the argument is the node, not the parent. Rename the argument to "node" in this case. I think this stems from the use of the name "np" in other places (confusing "node parent" for "node pointer"). Therefore replace all uses of "np" with "node". In addition in write_properties now that np=>node the name pp makes no sense. Rename it to "prop". No semantic change. Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Acked-by: Julien Grall <julien.grall@linaro.org> commit d0c5781700a62d6024d9e765dec1d3657858c865 Author: Ian Campbell <ian.campbell@citrix.com> Date: Fri Nov 22 15:07:01 2013 +0000 xen: arm: remove /xen-core-devices node from dom0 dtb The intention of adding this node to contain the GIC, timer and memory nodes (in 1c08d6004ea7) was to allow us to control the #address-cells and However in the case of the memory node the #*-cells are always taken from the root node (see ePAPR 3.1, "the following nodes shall be present at the root... memory node"). This caused breakage on the arndale platform. In addition it is not valid to just create sub-nodes like this. Unless they declare themselves as a bus then they will not necessarily be enumerated (although Linux currently does so in practice). Therefore: - Move the memory node back to the top level. - Insert the timer and primary gic nodes in the same location as the host DTB, replacing the originals. Note that the nodes here may be marked as in use by Xen and therefore the check must be before we discard nodes used by Xen. - Drop any secondary gics. Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Acked-by: Julien Grall <julien.grall@linaro.org> Tested-by: Julien Grall <julien.grall@linaro.org> commit 93218d8a25eb6f492d4a3a43b6bebe0b151b9214 Merge: 2a16fcd... d6e8fd3... Author: Jan Beulich <jbeulich@suse.com> Date: Fri Nov 22 15:27:19 2013 +0100 Merge branch ''for-4.4.tmem'' of git://xenbits.xen.org/people/konradwilk/xen into staging commit 2a16fcd5ba0244fef764886211452acc69c0ed00 Author: David Vrabel <david.vrabel@citrix.com> Date: Fri Nov 22 14:48:12 2013 +0100 x86/crash: disable the watchdog NMIs on the crashing cpu nmi_shootdown_cpus() is called during a crash to park all the other CPUs. This changes the NMI trap handlers which means there''s no point in having the watchdog still running. This also disables the watchdog before executing any crash kexec image and prevents the image from receiving unexpected NMIs. Signed-off-by: David Vrabel <david.vrabel@citrix.com> PVOps Linux as a kexec image shoots itself in the foot otherwise. On a Core2 system, Linux declares a firmware bug and tries to invert some bits in the performance counter register. It ends up setting the number of retired instructions to generate another NMI to fewer instructions than the NMI interrupt path itself, and ceases to make any useful progress. The call to disable_lapic_nmi_watchdog() must be this late into the kexec path to be sure that this cpu is the one which will execute the kexec image. Otherwise there are race conditions where the NMIs might be disabled on the wrong cpu, resulting in the kexec image still receiving NMIs. Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com> commit e95dc6ba69daef6468b3ae5912710727244d6e2f Author: Tomasz Wroblewski <tomasz.wroblewski@citrix.com> Date: Fri Nov 22 14:47:24 2013 +0100 x86/hvm: reset TSC to 0 after domain resume from S3 Host S3 implicitly resets the host TSC to 0, but the tsc offset for hvm domains is not recalculated when they resume, causing it to go into negative values. In Linux guest using tsc clocksource, this results in a hang after wrap back to positive values since the tsc clocksource implementation expects it reset. Signed-off-by: Tomasz Wroblewski <tomasz.wroblewski@citrix.com> commit 3ae607c381f83fd9256d4b9d6dd3bfdd5f3612f7 Author: Andrew Cooper <andrew.cooper3@citrix.com> Date: Fri Nov 22 13:38:44 2013 +0100 x86/stack: change show_stack_overflow() to use frame pointers if available Pass a full set of cpu_user_regs, and defer the hand-coded stack printing to _show_trace(), which will correctly use frame pointers if available. Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com> Acked-by: Keir Fraser <keir@xen.org> Reviewed-by: Jan Beulich <jbeulich@suse.com> commit cc2f01e602d9310f88625b4bdb4dc4ae179f2751 Author: Andrew Cooper <andrew.cooper3@citrix.com> Date: Fri Nov 22 13:37:25 2013 +0100 x86/stack: adjust boundary conditions for printed stacks Move the boundary into current.h along with the other stack manipulation code. The boundary is now the word adjacent to a struct cpu_info on the stack. This also fixes the somewhat spurious bounds for the case with frame pointers. Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com> Acked-by: Keir Fraser <keir@xen.org> Reviewed-by: Jan Beulich <jbeulich@suse.com> commit 9f000b981e54027b1579225a54ee991d337f8d5b Author: Andrew Cooper <andrew.cooper3@citrix.com> Date: Fri Nov 22 13:36:47 2013 +0100 x86/stack: refactor show_trace() Before, show_trace() had two implementations depending on CONFIG_FRAME_POINTER. Some parts were common, while the loops to wander up the stack were different. The version aided by frame pointers had a special case for function calls on wild function pointers, but this doesn''t need to be a special case. After the refactoring, there are now two implementations of _show_trace() which differ depending on CONFIG_FRAME_POINTER, and a single show_trace() with the common bits, including the logic for wild function pointers. Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com> Acked-by: Keir Fraser <keir@xen.org> Reviewed-by: Jan Beulich <jbeulich@suse.com> commit d6e8fd32890b663bc449ab4b60e21dafb09bb821 Author: Bob Liu <lliubbo@gmail.com> Date: Fri Nov 8 09:03:57 2013 +0800 tmem: cleanup: drop useless functions Function tmem_release_avail_pages_to_host() and tmem_scrub_page() only used once, no need to separate them out. Signed-off-by: Bob Liu <bob.liu@oracle.com> Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> commit 9a022746f9667b700e986af458697a9fde093605 Author: Bob Liu <lliubbo@gmail.com> Date: Fri Nov 8 09:03:56 2013 +0800 tmem: cleanup: drop useless ''_subpage'' wrap functions There are too many alloc place in tmem which cause the source code is hard too read. Most of the alloc path are useless and only referenced only once, this patch drops ''tmem_alloc_subpage/_thispool'' and ''_tmem_alloc''. Signed-off-by: Bob Liu <bob.liu@oracle.com> Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> commit 501d6114e672ad56c4308a58adddc719021939db Author: Bob Liu <lliubbo@gmail.com> Date: Fri Nov 8 09:03:55 2013 +0800 tmem: cleanup: drop unused function ''domain_fully_allocated'' Nobody uses this function. Signed-off-by: Bob Liu <bob.liu@oracle.com> Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> commit 0459e197baf02b671dc72e6e72d4267290e72d7a Author: Bob Liu <lliubbo@gmail.com> Date: Fri Nov 8 09:03:54 2013 +0800 tmem: cleanup: drop useless wrap functions _tmem_alloc/free_subpage_thispool() and _tmem_alloc/free_page_thispool() are useless, replace them with tmem_alloc/free_subpage_thispool() and tmem_alloc/free_page_thispool() directly. Signed-off-by: Bob Liu <bob.liu@oracle.com> Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> commit b9f9f62f5451f9610f2957438ed72fd0a88ebae4 Author: Bob Liu <lliubbo@gmail.com> Date: Fri Nov 8 09:03:53 2013 +0800 tmem: cleanup: drop typedef tmem_client_t Informations in typedef ''tmem_client_t'' can be integreated into ''struct client'' directly, no need to use a separate struct. Signed-off-by: Bob Liu <bob.liu@oracle.com> Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> commit d05eed005e784190b4560f51f14a31dad2e31c54 Author: Bob Liu <lliubbo@gmail.com> Date: Fri Nov 8 09:03:52 2013 +0800 tmem: cleanup: drop function tmem_alloc/free_infra Useless function can be replaced by xmalloc/xfree directly. Signed-off-by: Bob Liu <bob.liu@oracle.com> Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> commit b2fed42779c85992350f6c84d73516bfeb7063df Author: Bob Liu <lliubbo@gmail.com> Date: Fri Nov 8 09:03:51 2013 +0800 tmem: cleanup: drop most of the typedefs There are many typedefs in tmem most of which are useless and make code unstraightforward. This patch try to cleanup those typedefs. Signed-off-by: Bob Liu <bob.liu@oracle.com> Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> commit 1f4271763027700168f2aac9b561633c2fccca59 Author: Bob Liu <lliubbo@gmail.com> Date: Fri Nov 8 09:03:50 2013 +0800 tmem: cleanup: rename ''tmh_'' with ''tmem_'' tmem was designed can be ported to other platform besides xen easily, but I don''t think anybody will port tmem to other platform. And this flexible character made tmem not easy for understand, there are too many ''tmh_'' and ''tmem_'' functions and variables. This patch replace all ''tmh_'' functions/variables with ''tmem_'' to make code more readable. Signed-off-by: Bob Liu <bob.liu@oracle.com> Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> commit 006a687ba4de74d7933c09b43872abc19f126c63 Author: Bob Liu <lliubbo@gmail.com> Date: Fri Nov 8 09:03:49 2013 +0800 tmem: cleanup: drop typedef tmem_cli_mfn_t Using ''xen_pfn_t'' directly instead of ''tmem_cli_mfn_t'' to make code more readable. Signed-off-by: Bob Liu <bob.liu@oracle.com> Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> commit 6de3699137ed5cc802c264ab2d744b0a5efc82e6 Author: Bob Liu <lliubbo@gmail.com> Date: Fri Nov 8 09:03:48 2013 +0800 tmem: cleanup: drop typedef pfp_t Using ''struct page_info'' directly instead of ''pfp_t'' to make code more straightforward and readable. Signed-off-by: Bob Liu <bob.liu@oracle.com> Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> commit b29eba90610df0fe69dd9f51c1cacfd1a3f033a8 Author: Bob Liu <lliubbo@gmail.com> Date: Fri Nov 8 09:03:47 2013 +0800 tmem: cleanup: drop COMPARE_COPY_PAGE_SSE2 COMPARE_COPY_PAGE_SSE2 never be used in other place, memcpy() is enough. Signed-off-by: Bob Liu <bob.liu@oracle.com> Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> (qemu changes not included)