[Xen-devel] [ovmf baseline-only test] 68026: all pass

2016-11-12 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 68026 ovmf real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/68026/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf faabc5d49700a5042535ff30a07e2c9577ed3cd8 baseline v

[Xen-devel] [qemu-mainline test] 102163: regressions - trouble: blocked/broken/fail/pass

2016-11-12 Thread osstest service owner
flight 102163 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/102163/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-i3863 host-install(3)broken REGR. vs. 101909 test-amd64-amd64-

[Xen-devel] [xen-unstable test] 102160: regressions - trouble: broken/fail/pass

2016-11-12 Thread osstest service owner
flight 102160 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/102160/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemut-stubdom-debianhvm-amd64-xsm 3 host-install(3) broken REGR. vs. 102146 t

[Xen-devel] [linux-3.4 baseline-only test] 68023: regressions - FAIL

2016-11-12 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 68023 linux-3.4 real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/68023/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-raw9 debian-di-install

[Xen-devel] [qemu-mainline test] 102151: regressions - FAIL

2016-11-12 Thread osstest service owner
flight 102151 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/102151/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-libvirt 11 guest-start fail REGR. vs. 101909 test-amd64-i386-l

Re: [Xen-devel] [PATCH v2 03/11] pvh: Set online VCPU map to avail_vcpus

2016-11-12 Thread Wei Liu
On Fri, Nov 11, 2016 at 02:57:58PM -0500, Konrad Rzeszutek Wilk wrote: > On Wed, Nov 09, 2016 at 09:39:51AM -0500, Boris Ostrovsky wrote: > > ACPI builder marks VCPUS set in vcpu_online map as enabled in MADT. > > With ACPI-based CPU hotplug we only want VCPUs that are started by > > the guest to b

Re: [Xen-devel] [PATCH v3 10/12] xenstore: add helper functions for wire argument parsing

2016-11-12 Thread Wei Liu
On Fri, Nov 11, 2016 at 09:00:08AM +0100, Juergen Gross wrote: > The xenstore wire command argument parsing of the different commands > is repeating some patterns multiple times. Add some helper functions > to avoid the duplicated code. > > Signed-off-by: Juergen Gross Acked-by: Wei Liu __

Re: [Xen-devel] [PATCH v3 11/12] xenstore: add small default data buffer to internal struct

2016-11-12 Thread Wei Liu
On Fri, Nov 11, 2016 at 09:00:09AM +0100, Juergen Gross wrote: > Instead of always allocating a data buffer for incoming or outgoing > xenstore wire data add a small buffer to the buffered_data structure > of xenstored. This has the advantage that especially sending simple > response messages like

Re: [Xen-devel] [PATCH v3 12/12] xenstore: handle memory allocation failures in xenstored

2016-11-12 Thread Wei Liu
On Fri, Nov 11, 2016 at 12:07:16PM +0100, Juergen Gross wrote: > On 11/11/16 09:00, Juergen Gross wrote: > > Check for failures when allocating new memory in xenstored. > > > > Unfortunately there are several conditions which might render those > > checks void: It might be impossible to send an "E

Re: [Xen-devel] [PATCH v3 04/12] xenstore: add per-node generation counter

2016-11-12 Thread Wei Liu
On Fri, Nov 11, 2016 at 09:00:02AM +0100, Juergen Gross wrote: > In order to be able to support reading the list of a node's children in > multiple chunks (needed for list sizes > 4096 bytes) without having to > allocate a temporary buffer we need some kind of generation counter for > each node. Th

Re: [Xen-devel] [PATCH v3 08/12] xenstore: let command functions return error or success

2016-11-12 Thread Wei Liu
On Fri, Nov 11, 2016 at 09:00:06AM +0100, Juergen Gross wrote: > Add a return value to all wire command functions of xenstored. If such > a function returns an error send the error message in > process_message(). > > Signed-off-by: Juergen Gross As far as I can tell, this is merely refactoring e

Re: [Xen-devel] [PATCH v3 06/12] xenstore: support XS_DIRECTORY_PART in libxenstore

2016-11-12 Thread Wei Liu
On Fri, Nov 11, 2016 at 09:00:04AM +0100, Juergen Gross wrote: > This will enable all users of libxenstore to handle xenstore nodes > with a huge amount of children. > > In order to not depend completely on the XS_DIRECTORY_PART > functionality use it only in case of E2BIG returned by XS_DIRECTORY

Re: [Xen-devel] [PATCH v3 05/12] xenstore: add support for reading directory with many children

2016-11-12 Thread Wei Liu
On Fri, Nov 11, 2016 at 09:00:03AM +0100, Juergen Gross wrote: > As the payload size for one xenstore wire command is limited to 4096 > bytes it is impossible to read the children names of a node with a > large number of children (e.g. /local/domain in case of a host with > more than about 2000 dom

Re: [Xen-devel] [PATCH v3 07/12] xenstore: use array for xenstore wire command handling

2016-11-12 Thread Wei Liu
On Fri, Nov 11, 2016 at 09:00:05AM +0100, Juergen Gross wrote: > Instead of switch() statements for selecting wire command actions use > an array for this purpose. > > While doing this add the XS_RESTRICT type for diagnostic prints and > correct the printed string for XS_IS_DOMAIN_INTRODUCED. > >

Re: [Xen-devel] [PATCH v3 03/12] xenstore: use common tdb record header in xenstore

2016-11-12 Thread Wei Liu
On Fri, Nov 11, 2016 at 09:00:01AM +0100, Juergen Gross wrote: > The layout of the tdb record of xenstored is defined at multiple > places: read_node(), write_node() and in xs_tdb_dump.c > > Use a common structure instead. > > Signed-off-by: Juergen Gross Acked-by: Wei Liu ___

Re: [Xen-devel] [PATCH v3 02/12] xenstore: call add_change_node() directly when writing node

2016-11-12 Thread Wei Liu
On Fri, Nov 11, 2016 at 09:00:00AM +0100, Juergen Gross wrote: > Instead of calling add_change_node() at places where write_node() is > called, do that inside write_node(). > > Signed-off-by: Juergen Gross There seems to be a subtle change in behaviour -- previously in create_node, there is no

Re: [Xen-devel] [PATCH v3 01/12] xenstore: modify add_change_node() parameter types

2016-11-12 Thread Wei Liu
On Fri, Nov 11, 2016 at 08:59:59AM +0100, Juergen Gross wrote: > In order to prepare adding a generation count to each node modify > add_change_node() to take the connection pointer and a node pointer > instead of the transaction pointer and node name as parameters. This > requires moving the call

Re: [Xen-devel] [PATCH v7 1/6] VMX: Permanently assign PI hook vmx_pi_switch_to()

2016-11-12 Thread Konrad Rzeszutek Wilk
On November 12, 2016 4:14:50 AM EST, "Wu, Feng" wrote: >> > diff --git a/xen/arch/x86/hvm/vmx/vmx.c >b/xen/arch/x86/hvm/vmx/vmx.c >> > index 3d330b6..10546af 100644 >> > --- a/xen/arch/x86/hvm/vmx/vmx.c >> > +++ b/xen/arch/x86/hvm/vmx/vmx.c >> > @@ -222,8 +222,13 @@ void vmx_pi_hooks_deassign(stru

[Xen-devel] [xen-unstable test] 102146: tolerable FAIL

2016-11-12 Thread osstest service owner
flight 102146 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/102146/ Failures :-/ but no regressions. Tests which are failing intermittently (not blocking): test-armhf-armhf-xl-xsm 6 xen-boot fail pass in 102133 Regressions which are regarded a

Re: [Xen-devel] [PATCH v8] This is ABI for the two halves of a Para-virtual sound driver to communicate with each to other.

2016-11-12 Thread Oleksandr Andrushchenko
Hi, Konrad! First of all thank you for reviewing and providing valuable comments! I will put action items/TODOs at the very beginning, so we can see the summary. Also, please see answers inline. 1. Change frontend-id to frontend_id 2. Think about having a single sound card configured with a bunch

[Xen-devel] [xen-unstable-smoke test] 102149: tolerable all pass - PUSHED

2016-11-12 Thread osstest service owner
flight 102149 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/102149/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-libvirt 12 migrate-support-checkfail never pass test-armhf-armhf-xl 1

Re: [Xen-devel] [PATCH v7 1/6] VMX: Permanently assign PI hook vmx_pi_switch_to()

2016-11-12 Thread Wu, Feng
> > diff --git a/xen/arch/x86/hvm/vmx/vmx.c b/xen/arch/x86/hvm/vmx/vmx.c > > index 3d330b6..10546af 100644 > > --- a/xen/arch/x86/hvm/vmx/vmx.c > > +++ b/xen/arch/x86/hvm/vmx/vmx.c > > @@ -222,8 +222,13 @@ void vmx_pi_hooks_deassign(struct domain *d) > > > > d->arch.hvm_domain.vmx.vcpu_block =

[Xen-devel] [libvirt test] 102144: tolerable all pass - PUSHED

2016-11-12 Thread osstest service owner
flight 102144 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/102144/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-armhf-armhf-libvirt 13 saverestore-support-checkfail like 102082 test-armhf-armhf-libvirt-qcow2 1