flight 6822 xen-unstable real [real] http://www.chiark.greenend.org.uk/~xensrcts/logs/6822/ Regressions :-( Tests which did not succeed and are blocking: test-i386-xcpkern-i386-pair 16 guest-start fail REGR. vs. 6820 Tests which did not succeed, but are not blocking, including regressions (tests previously passed) regarded as allowable: test-amd64-amd64-win 16 leak-check/check fail never pass test-amd64-amd64-xl-win 13 guest-stop fail never pass test-amd64-i386-rhel6hvm-amd 8 guest-saverestore fail never pass test-amd64-i386-rhel6hvm-intel 8 guest-saverestore fail never pass test-amd64-i386-win-vcpus1 16 leak-check/check fail never pass test-amd64-i386-win 16 leak-check/check fail never pass test-amd64-i386-xl-win-vcpus1 13 guest-stop fail never pass test-amd64-xcpkern-i386-rhel6hvm-amd 8 guest-saverestore fail never pass test-amd64-xcpkern-i386-rhel6hvm-intel 8 guest-saverestore fail never pass test-amd64-xcpkern-i386-win 16 leak-check/check fail never pass test-amd64-xcpkern-i386-xl-credit2 11 guest-localmigrate fail like 6819 test-amd64-xcpkern-i386-xl-win 13 guest-stop fail never pass test-i386-i386-win 16 leak-check/check fail never pass test-i386-i386-xl-win 13 guest-stop fail never pass test-i386-xcpkern-i386-win 16 leak-check/check fail never pass version targeted for testing: xen dbb5ad1c51bd baseline version: xen 697ac895c11c ------------------------------------------------------------ People who touched revisions under test: Ian Campbell <ian.campbell@citrix.com> Jan Beulich <jbeulich@novell.com> Keir Fraser <keir@xen.org> Shriram Rajagopalan <rshriram@cs.ubc.ca> Wei Wang <wei.wang2@amd.com> ------------------------------------------------------------ jobs: build-i386-xcpkern pass build-amd64 pass build-i386 pass build-amd64-oldkern pass build-i386-oldkern pass build-amd64-pvops pass build-i386-pvops pass test-amd64-amd64-xl pass test-amd64-i386-xl pass test-i386-i386-xl pass test-amd64-xcpkern-i386-xl pass test-i386-xcpkern-i386-xl pass test-amd64-i386-rhel6hvm-amd fail test-amd64-xcpkern-i386-rhel6hvm-amd fail test-amd64-i386-xl-credit2 pass test-amd64-xcpkern-i386-xl-credit2 fail test-amd64-i386-rhel6hvm-intel fail test-amd64-xcpkern-i386-rhel6hvm-intel fail test-amd64-i386-xl-multivcpu pass test-amd64-xcpkern-i386-xl-multivcpu pass test-amd64-amd64-pair pass test-amd64-i386-pair pass test-i386-i386-pair pass test-amd64-xcpkern-i386-pair pass test-i386-xcpkern-i386-pair fail test-amd64-amd64-pv pass test-amd64-i386-pv pass test-i386-i386-pv pass test-amd64-xcpkern-i386-pv pass test-i386-xcpkern-i386-pv pass test-amd64-i386-win-vcpus1 fail test-amd64-i386-xl-win-vcpus1 fail test-amd64-amd64-win fail test-amd64-i386-win fail test-i386-i386-win fail test-amd64-xcpkern-i386-win fail test-i386-xcpkern-i386-win fail test-amd64-amd64-xl-win fail test-i386-i386-xl-win fail test-amd64-xcpkern-i386-xl-win 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. ------------------------------------------------------------ changeset: 23219:dbb5ad1c51bd tag: tip user: Ian Campbell <ian.campbell@citrix.com> date: Tue Apr 12 14:00:49 2011 +0100 tools: hvmloader: select BIOS through xenstore. Allow the toolstack to select the BIOS to use via a xenstore key. Defaults to "rombios" for compatibility with toolstacks which do not write the key (e.g. xend). Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Signed-off-by: Keir Fraser <keir@xen.org> changeset: 23218:2869bb8995e2 user: Ian Campbell <ian.campbell@citrix.com> date: Tue Apr 12 13:47:16 2011 +0100 tools: hvmloader: Refactor MP table setup into struct bios_config Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Acked-by: Keir Fraser <keir@xen.org> changeset: 23217:36799b6d1db2 user: Ian Campbell <ian.campbell@citrix.com> date: Tue Apr 12 13:46:49 2011 +0100 tools: hvmloader: Refactor ACPI table setup into struct bios_config Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Acked-by: Keir Fraser <keir@xen.org> changeset: 23216:845d654b6117 user: Ian Campbell <ian.campbell@citrix.com> date: Tue Apr 12 13:46:20 2011 +0100 tools: hvmloader: Refactor VM86 and E820 setup into struct bios_config Signed-off-by: Ian Campbell <ian.campbell@citrix.com> changeset: 23215:8fc9d810fd16 user: Ian Campbell <ian.campbell@citrix.com> date: Tue Apr 12 13:45:59 2011 +0100 tools: hvmloader: refactor highbios and bios_info setup into struct bios_config Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Acked-by: Keir Fraser <keir@xen.org> changeset: 23214:aedd22b43b9d user: Ian Campbell <ian.campbell@citrix.com> date: Tue Apr 12 13:45:31 2011 +0100 tools: hvmloader: Refactor APIC, PCI and SMP setup into struct bios_config Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Acked-by: Keir Fraser <keir@xen.org> changeset: 23213:4f06dfe5d750 user: Ian Campbell <ian.campbell@citrix.com> date: Tue Apr 12 13:44:38 2011 +0100 tools: hvmloader: add bios_config data structure For now abstract away the actual ROM bits themselves and the various load addresses. Create a rombios.c to contain the ROMBIOS specific parts. ROMBIOS is still statically selected for the time being. Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Acked-by: Keir Fraser <keir@xen.org> changeset: 23212:560187ccaf95 user: Ian Campbell <ian.campbell@citrix.com> date: Tue Apr 12 13:43:45 2011 +0100 tools: hvmloader: Define $(OBJS) directly instead of via $(SRCS) $(SRCS) isn''t used for anything else. Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Acked-by: Keir Fraser <keir@xen.org> changeset: 23211:d237d2e2a47d user: Ian Campbell <ian.campbell@citrix.com> date: Tue Apr 12 13:41:43 2011 +0100 tools: hvmloader: rename roms.h to roms.inc It''s not really a header, it''s an autogenerated data file. Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Acked-by: Keir Fraser <keir@xen.org> changeset: 23210:95746ab3b1e4 user: Ian Campbell <ian.campbell@citrix.com> date: Tue Apr 12 13:40:49 2011 +0100 tools: hvmloader: remove rombios_sz, just use sizeof(rombios) Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Acked-by: Keir Fraser <keir@xen.org> changeset: 23209:739f7f80273e user: Ian Campbell <ian.campbell@citrix.com> date: Tue Apr 12 13:39:56 2011 +0100 tools: hvmloader: refactor Makefile to move ROM filenames into variables. Add an option to use debug Cirrus video BIOS, simply as a convenience. Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Acked-by: Keir Fraser <keir@xen.org> changeset: 23208:a09d5a85609e user: Ian Campbell <ian.campbell@citrix.com> date: Tue Apr 12 13:39:22 2011 +0100 tools: hvmloader: split scratch and hypercall addressing from ROMBIOS low heap. Although happen to live at the same physical address their lifespans do not overlap. The scratch and hypercall spaces are used only within hvmloader and the same area is reused as a heap within ROMBIOS. But each is free to make its own decisions about where to place things. Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Acked-by: Keir Fraser <keir@xen.org> changeset: 23207:ca43f04eaa8e user: Ian Campbell <ian.campbell@citrix.com> date: Tue Apr 12 13:38:38 2011 +0100 tools: hvmloader: pass option ROM end address around as a parameter. Reduces the cross talk between ROMBIOS and hvmloader. Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Acked-by: Keir Fraser <keir@xen.org> changeset: 23206:b5180edfccda user: Ian Campbell <ian.campbell@citrix.com> date: Tue Apr 12 13:38:01 2011 +0100 tools: hvmloader: pass SMBIOS location as a runtime parameter. Instead of hardcoding in a header. Reduces the cross talk between ROMBIOS and hvmloader. Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Acked-by: Keir Fraser <keir@xen.org> changeset: 23205:060f5397097b user: Ian Campbell <ian.campbell@citrix.com> date: Tue Apr 12 13:37:03 2011 +0100 tools: hvmloader: pass ACPI_PHYSICAL_ADDRESS as a runtime parameter. Instead of hardcoding in a header. Reduces the cross talk between ROMBIOS and hvmloader. Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Acked-by: Keir Fraser <keir@xen.org> changeset: 23204:846f435915a2 user: Ian Campbell <ian.campbell@citrix.com> date: Tue Apr 12 13:36:17 2011 +0100 tools: hvmloader: split e820 support into its own code module. Pass the table address as a paramter to the build function and cause it to return the number of entries. Pass both base and offset as parameters to the dump function. This adds a duplicated e820.h header to ROMBIOS. Since the e820 data structure is well defined by existing BIOS implementations I think this is OK and simplifies the cross talk between hvmloader and ROMBIOS. Reduces the cross talk between ROMBIOS and hvmloader. Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Acked-by: Keir Fraser <keir@xen.org> changeset: 23203:c8e3c66deb7b user: Ian Campbell <ian.campbell@citrix.com> date: Tue Apr 12 13:34:30 2011 +0100 tools: hvmloader: move ROMBIOS configuration into tools/firmware/rombios/ Currently rombios and hvmloader are rather intertwined. Separate the ROMBIOS configuration options out into a ROMBIOS provided file so that the dependency can become strictly from hvmloader to rombios. Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Acked-by: Keir Fraser <keir@xen.org> changeset: 23202:67911039ac94 user: Shriram Rajagopalan <rshriram@cs.ubc.ca> date: Tue Apr 12 13:28:51 2011 +0100 remus: fix incorrect error handling for switch_qemu_logdirty in checkpoint code c/s 22275: "tools: cleanup domain save switch_qemu_logdirty callback" introduced a whole bunch of error code fixups. In the process, it also ended up treating the success return code (0) from switch_qemu_logdirty as an error and vice versa. Signed-off-by: Shriram Rajagopalan <rshriram@cs.ubc.ca> changeset: 23201:e288952081a4 user: Jan Beulich <jbeulich@novell.com> date: Tue Apr 12 13:27:27 2011 +0100 passthrough: prevent non-HVM access to HVM-only data Spotted this oversight in c/s 23144:37c4f7d492a4. Signed-off-by: Jan Beulich <jbeulich@novell.com> changeset: 23200:995a0c01a076 user: Wei Wang <wei.wang2@amd.com> date: Tue Apr 12 13:26:19 2011 +0100 AMD IOMMU: Fix an interrupt remapping issue Some device could generate bogus interrupts if an IO-APIC RTE and an iommu interrupt remapping entry are not consistent during 2 adjacent 64bits IO-APIC RTE updates. For example, if the 2nd operation updates destination bits in RTE for SATA device and unmask it, in some case, SATA device will assert ioapic pin to generate interrupt immediately using new destination but iommu could still translate it into the old destination, then dom0 would be confused. To fix that, we sync up interrupt remapping entry with IO-APIC IRE on every 32 bits operation and forward IOAPIC RTE updates after interrupt. Signed-off-by: Wei Wang <wei.wang2@amd.com> Acked-by: Jan Beulich <jbeulich@novell.com> changeset: 23199:dbd98ab2f87f user: Wei Wang <wei.wang2@amd.com> date: Tue Apr 12 13:20:57 2011 +0100 amd iommu: Unmapped interrupt should generate IO page faults. This helps us to debug interrupt issues. Signed-off-by: Wei Wang <wei.wang2@amd.com> changeset: 23198:697ac895c11c user: Keir Fraser <keir@xen.org> date: Sat Apr 09 12:42:24 2011 +0100 nestedhvm: Remove nhvm_{initialise,destroy,reset}. They are a pointless level of abstraction beneath nestedhvm_* variants of the same operations, which all callers should be using. At the same time, nestedhvm_vcpu_initialise() does not need to call destroy if initialisation fails. That is the vendor-specific init function''s job (clearing up its own state on failure). Signed-off-by: Keir Fraser <keir@xen.org> (qemu changes not included) _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel