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-17 Thread Oleksandr Andrushchenko
Thank you all for the review and comments! I guess only these are not addressed (or still need comments): > 1. You can only say how many channels a stream has, not what the > channels correspond to (e.g., "left", "right" for a 2 channel stereo > stream; or "left-front", "rear-right", etc. for a 6

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

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

[Xen-devel] [PATCH] x86/cpuid: Add AVX512_4VNNIW and AVX512_4FMAPS support

2016-11-17 Thread He Chen
Add two new AVX512 subfeatures support for guest. AVX512_4VNNIW: Vector instructions for deep learning enhanced word variable precision. AVX512_4FMAPS: Vector instructions for deep learning floating-point single precision. Signed-off-by: Luwei Kang Signed-off-by: He Chen

Re: [Xen-devel] [PATCH v8 2/7] VMX: Properly handle pi when all the assigned devices are removed

2016-11-17 Thread Wu, Feng
> -Original Message- > From: Tian, Kevin > Sent: Friday, November 18, 2016 12:59 PM > To: Wu, Feng ; xen-devel@lists.xen.org > Cc: jbeul...@suse.com; andrew.coop...@citrix.com; > george.dun...@eu.citrix.com; dario.faggi...@citrix.com > Subject: RE: [PATCH v8 2/7] VMX:

[Xen-devel] [ovmf test] 102371: all pass - PUSHED

2016-11-17 Thread osstest service owner
flight 102371 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/102371/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf ec16deeac90e4b8014394be58a229f6aa8c493af baseline version: ovmf

Re: [Xen-devel] [PATCH v8 2/7] VMX: Properly handle pi when all the assigned devices are removed

2016-11-17 Thread Tian, Kevin
> From: Wu, Feng > Sent: Friday, November 18, 2016 12:27 PM > > > diff --git a/xen/drivers/passthrough/pci.c b/xen/drivers/passthrough/pci.c > > > index 8bce213..e71732f 100644 > > > --- a/xen/drivers/passthrough/pci.c > > > +++ b/xen/drivers/passthrough/pci.c > > > @@ -1602,6 +1602,13 @@ int

Re: [Xen-devel] [PATCH v8 7/7] VMX: Fixup PI descriptor when cpu is offline

2016-11-17 Thread Tian, Kevin
> From: Wu, Feng > Sent: Friday, November 18, 2016 9:57 AM > > When cpu is offline, we need to move all the vcpus in its blocking > list to another online cpu, this patch handles it. > > Signed-off-by: Feng Wu > Reviewed-by: Jan Beulich Acked-by: Kevin

Re: [Xen-devel] [PATCH v8 6/7] VT-d: Some cleanups

2016-11-17 Thread Tian, Kevin
> From: Wu, Feng > Sent: Friday, November 18, 2016 9:57 AM > > Use type-safe structure assignment instead of memcpy() > Use sizeof(*iremap_entry). > > Signed-off-by: Feng Wu > Reviewed-by: Konrad Rzeszutek Wilk Acked-by: Kevin Tian

Re: [Xen-devel] [PATCH v8 5/7] VT-d: No need to set irq affinity for posted format IRTE

2016-11-17 Thread Tian, Kevin
> From: Wu, Feng > Sent: Friday, November 18, 2016 9:59 AM > > We don't set the affinity for posted format IRTE, since the > destination of these interrupts is vCPU and the vCPU affinity > is set during vCPU scheduling. > > Signed-off-by: Feng Wu > --- > v8: > - Changes based

Re: [Xen-devel] [PATCH v8 4/7] VT-d: Use one function to update both remapped and posted IRTE

2016-11-17 Thread Tian, Kevin
> From: Wu, Feng > Sent: Friday, November 18, 2016 9:57 AM > > Use one function to update both remapped IRTE and posted IRET. > > Signed-off-by: Feng Wu > --- > v8: > - Newly added > > xen/drivers/passthrough/vtd/intremap.c | 162 > ++--- > 1

Re: [Xen-devel] [PATCH v8 3/7] VMX: Make sure PI is in proper state before install the hooks

2016-11-17 Thread Wu, Feng
> -Original Message- > From: Tian, Kevin > Sent: Friday, November 18, 2016 12:11 PM > To: Wu, Feng ; xen-devel@lists.xen.org > Cc: jbeul...@suse.com; andrew.coop...@citrix.com; > george.dun...@eu.citrix.com; dario.faggi...@citrix.com > Subject: RE: [PATCH v8 3/7] VMX:

Re: [Xen-devel] [PATCH v8 2/7] VMX: Properly handle pi when all the assigned devices are removed

2016-11-17 Thread Wu, Feng
> -Original Message- > From: Tian, Kevin > Sent: Friday, November 18, 2016 11:19 AM > To: Wu, Feng ; xen-devel@lists.xen.org > Cc: jbeul...@suse.com; andrew.coop...@citrix.com; > george.dun...@eu.citrix.com; dario.faggi...@citrix.com > Subject: RE: [PATCH v8 2/7] VMX:

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

2016-11-17 Thread Wu, Feng
> -Original Message- > From: Tian, Kevin > Sent: Friday, November 18, 2016 11:14 AM > To: Wu, Feng ; xen-devel@lists.xen.org > Cc: jbeul...@suse.com; andrew.coop...@citrix.com; > george.dun...@eu.citrix.com; dario.faggi...@citrix.com > Subject: RE: [PATCH v8 1/7] VMX:

Re: [Xen-devel] [PATCH v8 0/7] VMX: Properly handle pi descriptor and per-cpu blocking list

2016-11-17 Thread Wu, Feng
> -Original Message- > From: Tian, Kevin > Sent: Friday, November 18, 2016 11:03 AM > To: Wu, Feng ; xen-devel@lists.xen.org > Cc: jbeul...@suse.com; andrew.coop...@citrix.com; > george.dun...@eu.citrix.com; dario.faggi...@citrix.com > Subject: RE: [PATCH v8 0/7] VMX:

Re: [Xen-devel] [PATCH v8 3/7] VMX: Make sure PI is in proper state before install the hooks

2016-11-17 Thread Tian, Kevin
> From: Wu, Feng > Sent: Friday, November 18, 2016 9:57 AM > > We may hit the last ASSERT() in vmx_vcpu_block in the current code, > since vmx_vcpu_block() may get called before vmx_pi_switch_to() > has been installed or executed. Here We use cmpxchg to update > the NDST field, this can make sure

[Xen-devel] [xen-unstable baseline-only test] 68055: regressions - trouble: blocked/broken/fail/pass

2016-11-17 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 68055 xen-unstable real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/68055/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-xtf-amd64-amd64-13

Re: [Xen-devel] [PATCH v8 2/7] VMX: Properly handle pi when all the assigned devices are removed

2016-11-17 Thread Tian, Kevin
> From: Wu, Feng > Sent: Friday, November 18, 2016 9:57 AM > > This patch handles some corner cases when the last assigned device > is removed from the domain. In this case we should carefully handle > pi descriptor and the per-cpu blocking list, to make sure: > - all the PI descriptor are in the

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

2016-11-17 Thread Tian, Kevin
> From: Wu, Feng > Sent: Friday, November 18, 2016 9:57 AM > > PI hook vmx_pi_switch_to() is needed even when any previously even when -> even after > assigned device is detached from the domain. Since 'SN' bit is > also used to control the CPU side PI and we change the state of > SN bit in

Re: [Xen-devel] [PATCH v8 0/7] VMX: Properly handle pi descriptor and per-cpu blocking list

2016-11-17 Thread Tian, Kevin
> From: Wu, Feng > Sent: Friday, November 18, 2016 9:57 AM > > The current VT-d PI related code may operate incorrectly in the > following scenarios: > 1. When the last assigned device is dettached from the domain, all > the PI related hooks are removed then, however, the vCPU can be > blocked,

[Xen-devel] [PATCH v8 5/7] VT-d: No need to set irq affinity for posted format IRTE

2016-11-17 Thread Feng Wu
We don't set the affinity for posted format IRTE, since the destination of these interrupts is vCPU and the vCPU affinity is set during vCPU scheduling. Signed-off-by: Feng Wu --- v8: - Changes based on [6/7] v7: - Compare all the field in IRTE to justify whether we can

[Xen-devel] [PATCH v8 3/7] VMX: Make sure PI is in proper state before install the hooks

2016-11-17 Thread Feng Wu
We may hit the last ASSERT() in vmx_vcpu_block in the current code, since vmx_vcpu_block() may get called before vmx_pi_switch_to() has been installed or executed. Here We use cmpxchg to update the NDST field, this can make sure we only update the NDST when vmx_pi_switch_to() has not been called.

[Xen-devel] [PATCH v8 2/7] VMX: Properly handle pi when all the assigned devices are removed

2016-11-17 Thread Feng Wu
This patch handles some corner cases when the last assigned device is removed from the domain. In this case we should carefully handle pi descriptor and the per-cpu blocking list, to make sure: - all the PI descriptor are in the right state when next time a devices is assigned to the domain again.

[Xen-devel] [PATCH v8 1/7] VMX: Permanently assign PI hook vmx_pi_switch_to()

2016-11-17 Thread Feng Wu
PI hook vmx_pi_switch_to() is needed even when any previously assigned device is detached from the domain. Since 'SN' bit is also used to control the CPU side PI and we change the state of SN bit in these two functions, then evaluate this bit in vmx_deliver_posted_intr() when trying to deliver the

[Xen-devel] [PATCH v8 7/7] VMX: Fixup PI descriptor when cpu is offline

2016-11-17 Thread Feng Wu
When cpu is offline, we need to move all the vcpus in its blocking list to another online cpu, this patch handles it. Signed-off-by: Feng Wu Reviewed-by: Jan Beulich --- v7: - Pass unsigned int to vmx_pi_desc_fixup() v6: - Carefully suppress 'SN' to avoid

[Xen-devel] [PATCH v8 0/7] VMX: Properly handle pi descriptor and per-cpu blocking list

2016-11-17 Thread Feng Wu
The current VT-d PI related code may operate incorrectly in the following scenarios: 1. When the last assigned device is dettached from the domain, all the PI related hooks are removed then, however, the vCPU can be blocked, switched to another pCPU, etc, all without the aware of PI. After the

[Xen-devel] [PATCH v8 4/7] VT-d: Use one function to update both remapped and posted IRTE

2016-11-17 Thread Feng Wu
Use one function to update both remapped IRTE and posted IRET. Signed-off-by: Feng Wu --- v8: - Newly added xen/drivers/passthrough/vtd/intremap.c | 162 ++--- 1 file changed, 66 insertions(+), 96 deletions(-) diff --git

[Xen-devel] [PATCH v8 6/7] VT-d: Some cleanups

2016-11-17 Thread Feng Wu
Use type-safe structure assignment instead of memcpy() Use sizeof(*iremap_entry). Signed-off-by: Feng Wu Reviewed-by: Konrad Rzeszutek Wilk --- v7: - Remove a useless cleanup v6: - More descripion about the patch

[Xen-devel] [PATCH v8 5/7] VT-d: No need to set irq affinity for posted format IRTE

2016-11-17 Thread Feng Wu
We don't set the affinity for posted format IRTE, since the destination of these interrupts is vCPU and the vCPU affinity is set during vCPU scheduling. Signed-off-by: Feng Wu --- v8: - Changes based on [6/7] v7: - Compare all the field in IRTE to justify whether we can

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

2016-11-17 Thread osstest service owner
flight 102365 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/102365/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-libvirt-vhd 9 debian-di-installfail REGR. vs. 101909

[Xen-devel] [ovmf test] 102367: all pass - PUSHED

2016-11-17 Thread osstest service owner
flight 102367 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/102367/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 4e7872d2f71f511df155a9048f06761afce751da baseline version: ovmf

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

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

[Xen-devel] [ovmf test] 102363: all pass - PUSHED

2016-11-17 Thread osstest service owner
flight 102363 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/102363/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 28b3a713b66998a8be3e8558eb85f18699e15b2e baseline version: ovmf

Re: [Xen-devel] Xen like VirtualBox

2016-11-17 Thread Jason Long
Please thinking about a nice GUI. After it users like me and book author working and introduce Xen more than it. Thank you. On Tue, 11/15/16, Jason Long wrote: Subject: Re: [Xen-devel] Xen like VirtualBox To: "George Dunlap"

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

2016-11-17 Thread osstest service owner
flight 102351 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/102351/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-libvirt-vhd 9 debian-di-installfail REGR. vs. 101909

[Xen-devel] [xtf test] 102361: all pass - PUSHED

2016-11-17 Thread osstest service owner
flight 102361 xtf real [real] http://logs.test-lab.xenproject.org/osstest/logs/102361/ Perfect :-) All tests in this flight passed as required version targeted for testing: xtf a8bca39aa38912c72eb7c2a1abc1a546c46b8ca8 baseline version: xtf

[Xen-devel] [PATCH v3 (re-send)] xen/gntdev: Use mempolicy instead of VM_IO flag to avoid NUMA balancing

2016-11-17 Thread Boris Ostrovsky
Commit 9c17d96500f7 ("xen/gntdev: Grant maps should not be subject to NUMA balancing") set VM_IO flag to prevent grant maps from being subjected to NUMA balancing. It was discovered recently that this flag causes get_user_pages() to always fail with -EFAULT. check_vma_flags __get_user_pages

[Xen-devel] [PATCH v3] xen/gntdev: Use mempolicy instead of VM_IO flag to avoid NUMA balancing

2016-11-17 Thread Boris Ostrovsky
Commit 9c17d96500f7 ("xen/gntdev: Grant maps should not be subject to NUMA balancing") set VM_IO flag to prevent grant maps from being subjected to NUMA balancing. It was discovered recently that this flag causes get_user_pages() to always fail with -EFAULT. check_vma_flags __get_user_pages

[Xen-devel] [ovmf test] 102360: all pass - PUSHED

2016-11-17 Thread osstest service owner
flight 102360 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/102360/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 5ea2bad0d9ab6a2465195badd323313553baa61b baseline version: ovmf

Re: [Xen-devel] [RFC PATCH 5/9] x86/HVM/SVM: Add AVIC initialization code

2016-11-17 Thread Suravee Suthikulpanit
On 11/17/16 11:18, Konrad Rzeszutek Wilk wrote: On Thu, Nov 17, 2016 at 10:05:58AM -0600, Suravee Suthikulpanit wrote: Konrad, Thanks for the review comments. Got one quick question below. On 10/12/16 15:02, Konrad Rzeszutek Wilk wrote: +int svm_avic_init_vmcb(struct vcpu *v) +{ +

Re: [Xen-devel] Xen ARM small task (WAS: Re: [Xen Question])

2016-11-17 Thread Julien Grall
Hi Stefano, On 17/11/2016 11:26, Stefano Stabellini wrote: On Mon, 14 Nov 2016, Julien Grall wrote: On 11/11/2016 13:55, Stefano Stabellini wrote: On Fri, 11 Nov 2016, Julien Grall wrote: On 11/11/2016 02:24, Stefano Stabellini wrote: On Thu, 10 Nov 2016, Julien Grall wrote: (CC Stefano

[Xen-devel] [PATCH v2 32/35] libxl/libxl_vnuma.c: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D logging functions where possible instead of the LOG* ones. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_vnuma.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/libxl/libxl_vnuma.c

[Xen-devel] [PATCH v2 35/35] libxl/libxl_xshelp.c: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D logging functions where possible instead of the LOG* ones. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_xshelp.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[Xen-devel] [PATCH v2 25/35] libxl/libxl_pci.c: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D logging functions where possible instead of the LOG* ones. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_pci.c | 153 +--- 1 file changed, 79 insertions(+), 74

[Xen-devel] [PATCH v2 29/35] libxl/libxl_remus.c: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D logging functions where possible instead of the LOG* ones. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_remus.c | 27 ++- 1 file changed, 14 insertions(+), 13 deletions(-) diff --git

[Xen-devel] [PATCH v2 31/35] libxl/libxl_stream_write.c: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D logging functions where possible instead of the LOG* ones. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_stream_write.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git

[Xen-devel] [PATCH v2 16/35] libxl/libxl_dom_save.c: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D logging functions where possible instead of the LOG* ones. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_dom_save.c | 29 +++-- 1 file changed, 15 insertions(+), 14 deletions(-) diff

[Xen-devel] [PATCH v2 34/35] libxl/libxl_x86.c: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D logging functions where possible instead of the LOG* ones. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_x86.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tools/libxl/libxl_x86.c

[Xen-devel] [PATCH v2 33/35] libxl/libxl_vtpm.c: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D logging functions where possible instead of the LOG* ones. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_vtpm.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

[Xen-devel] [PATCH v2 27/35] libxl/libxl_pvusb.c: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D logging functions where possible instead of the LOG* ones. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_usb.c | 57 ++--- 1 file changed, 30 insertions(+), 27

[Xen-devel] [PATCH v2 28/35] libxl/libxl_qmp.c: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D logging functions where possible instead of the LOG* ones. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_qmp.c | 56 - 1 file changed, 28 insertions(+), 28

[Xen-devel] [PATCH v2 26/35] libxl/libxl_psr.c: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D logging functions where possible instead of the LOG* ones. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_psr.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git

[Xen-devel] [PATCH v2 15/35] libxl/libxl_dm.c: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D logging functions where possible instead of the LOG* ones. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_dm.c | 111 ++--- 1 file changed, 59 insertions(+), 52

[Xen-devel] [PATCH v2 04/35] libxl.c: switch to LOG*D use

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D functions to output the domain ID in logs as much as possible. This will help consumer code sorting the logs by domain. This commit includes all LOG* to LOG*D changes where the domain ID is not just a domid variable. We want the domain ID

[Xen-devel] [PATCH v2 24/35] libxl/libxl_no_colo.c: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D logging functions where possible instead of the LOG* ones. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_no_colo.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[Xen-devel] [PATCH v2 23/35] libxl/libxl_nic.c: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D logging functions where possible instead of the LOG* ones. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_nic.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git

[Xen-devel] [PATCH v2 22/35] libxl/libxl_netbuffer.c: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D logging functions where possible instead of the LOG* ones. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_netbuffer.c | 43 --- 1 file changed, 24 insertions(+), 19

[Xen-devel] [PATCH v2 30/35] libxl/libxl_save_callout.c: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D logging functions where possible instead of the LOG* ones. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_save_callout.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

[Xen-devel] [PATCH v2 11/35] libxl/libxl_colo_restore.c: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D logging functions where possible instead of the LOG* ones. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_colo_restore.c | 57 1 file changed, 28 insertions(+), 29

[Xen-devel] [PATCH v2 00/35] libxl LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
Hey all, Here is v2 addressing Wei's comments on patch #1. The 3 libxl.c patches haven't been merged, but the commit message of the first one has been slightly rewritten to help understanding the reason of the split. Cedric Bosdonnat (35): libxl: add LIBXL_LOGD_* and LOG*D function families.

[Xen-devel] [PATCH v2 19/35] libxl/libxl_internal.c: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D logging functions where possible instead of the LOG* ones. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_internal.c | 29 ++--- 1 file changed, 14 insertions(+), 15 deletions(-) diff

[Xen-devel] [PATCH v2 10/35] libxl/libxl_colo_qdisk.c: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D logging functions where possible instead of the LOG* ones. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_colo_qdisk.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[Xen-devel] [PATCH v2 07/35] libxl/libxl_colo.h: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D logging functions where possible instead of the LOG* ones. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_colo.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/libxl/libxl_colo.h

[Xen-devel] [PATCH v2 20/35] libxl/libxl_linux.c: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D logging functions where possible instead of the LOG* ones. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_linux.c | 23 ++- 1 file changed, 14 insertions(+), 9 deletions(-) diff --git

[Xen-devel] [PATCH v2 14/35] libxl/libxl_device.c: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D logging functions where possible instead of the LOG* ones. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_device.c | 70 -- 1 file changed, 37 insertions(+), 33

[Xen-devel] [PATCH v2 08/35] libxl/libxl_colo_nic.c: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D logging functions where possible instead of the LOG* ones. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_colo_nic.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[Xen-devel] [PATCH v2 18/35] libxl/libxl_freebsd.c: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D logging functions where possible instead of the LOG* ones. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_freebsd.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git

[Xen-devel] [PATCH v2 13/35] libxl/libxl_create.c: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D logging functions where possible instead of the LOG* ones. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_create.c | 119 +++-- 1 file changed, 60 insertions(+), 59

[Xen-devel] [PATCH v2 03/35] libxl.c: switch to LOG*D use (refactored messages)

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D functions to output the domain ID in logs as much as possible. This will help consumer code sorting the logs by domain. This commit, only changes LOG*() into LOG*D() and adds a domid parameter. The message of these LOG* calls has been

[Xen-devel] [PATCH v2 17/35] libxl/libxl_dom_suspend.c: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D logging functions where possible instead of the LOG* ones. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_dom_suspend.c | 45 - 1 file changed, 22 insertions(+), 23

[Xen-devel] [PATCH v2 12/35] libxl/libxl_colo_save.c: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D logging functions where possible instead of the LOG* ones. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_colo_save.c | 49 ++- 1 file changed, 25 insertions(+), 24

[Xen-devel] [PATCH v2 21/35] libxl/libxl_netbsd.c: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D logging functions where possible instead of the LOG* ones. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_netbsd.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git

[Xen-devel] [PATCH v2 05/35] libxl/libxl_bootloader.c: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D functions to output the domain ID in logs as much as possible. This will help consumer code sorting the logs by domain. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_bootloader.c | 84

[Xen-devel] [PATCH v2 09/35] libxl/libxl_colo_proxy.c: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D logging functions where possible instead of the LOG* ones. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_colo_proxy.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git

[Xen-devel] [PATCH v2 06/35] libxl/libxl_checkpoint_device.c: used LOG*D functions

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D logging functions where possible instead of the LOG* ones. Signed-off-by: Cédric Bosdonnat --- tools/libxl/libxl_checkpoint_device.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git

[Xen-devel] [PATCH v2 01/35] libxl: add LIBXL_LOGD_* and LOG*D function families.

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat These functions should be used to log messages when the domain id is known. libxl__log will now prepend the log message by "Domain %PRIu32:" if the domain id is a valid one. This aims at helping consumers filter logs on domain IDs. Signed-off-by:

[Xen-devel] [PATCH v2 02/35] libxl.c: switch to LOG*D use

2016-11-17 Thread Cédric Bosdonnat
From: Cedric Bosdonnat Use LOG*D functions to output the domain ID in logs as much as possible. This will help consumer code sorting the logs by domain. libxl.c changes have been split into 3 commits to help review them and isolate more instances that could be problematic.

Re: [Xen-devel] [PATCH V4] xen/arm: domain_build: allocate lowmem for dom0 as much as possible

2016-11-17 Thread Stefano Stabellini
On Wed, 16 Nov 2016, Andrii Anisov wrote: > > For example, take a look at PVCalls which is entirely based on data > > copies: > > > > http://marc.info/?l=xen-devel=147639616310487 > > > > > > I have already shown that it performs better than netfront/netback on > > x86 in this blog post: > > > >

Re: [Xen-devel] Xen ARM small task (WAS: Re: [Xen Question])

2016-11-17 Thread Stefano Stabellini
On Mon, 14 Nov 2016, Julien Grall wrote: > Hi Stefano, > > On 11/11/2016 13:55, Stefano Stabellini wrote: > > On Fri, 11 Nov 2016, Julien Grall wrote: > > > Hi Stefano, > > > > > > On 11/11/2016 02:24, Stefano Stabellini wrote: > > > > On Thu, 10 Nov 2016, Julien Grall wrote: > > > > > (CC

Re: [Xen-devel] [RFC PATCH 5/9] x86/HVM/SVM: Add AVIC initialization code

2016-11-17 Thread Konrad Rzeszutek Wilk
On Thu, Nov 17, 2016 at 10:55:52AM -0600, Suravee Suthikulpanit wrote: > Konrad, > > On 10/12/16 15:02, Konrad Rzeszutek Wilk wrote: > > > + > > > > +/* NOTE: Current max index allowed for physical APIC ID table is 255 */ > > > > +#define AVIC_PHY_APIC_ID_MAX0xFF > > > > + > > > > +#define

Re: [Xen-devel] [RFC PATCH 5/9] x86/HVM/SVM: Add AVIC initialization code

2016-11-17 Thread Konrad Rzeszutek Wilk
On Thu, Nov 17, 2016 at 10:05:58AM -0600, Suravee Suthikulpanit wrote: > Konrad, > > Thanks for the review comments. Got one quick question below. > > On 10/12/16 15:02, Konrad Rzeszutek Wilk wrote: > > > +int svm_avic_init_vmcb(struct vcpu *v) > > > > +{ > > > > +paddr_t ma; > > > > +

[Xen-devel] [ovmf test] 102355: all pass - PUSHED

2016-11-17 Thread osstest service owner
flight 102355 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/102355/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 2048ab4b3ec966183eb87d93d9cdab05fe9cc840 baseline version: ovmf

[Xen-devel] [xen-unstable test] 102338: tolerable FAIL - PUSHED

2016-11-17 Thread osstest service owner
flight 102338 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/102338/ Failures :-/ but no regressions. Tests which are failing intermittently (not blocking): test-armhf-armhf-libvirt-xsm 6 xen-boot fail in 102318 pass in 102338 test-armhf-armhf-xl-cubietruck

Re: [Xen-devel] [RFC PATCH 5/9] x86/HVM/SVM: Add AVIC initialization code

2016-11-17 Thread Suravee Suthikulpanit
Konrad, On 10/12/16 15:02, Konrad Rzeszutek Wilk wrote: + > +/* NOTE: Current max index allowed for physical APIC ID table is 255 */ > +#define AVIC_PHY_APIC_ID_MAX0xFF > + > +#define AVIC_DOORBELL 0xc001011b > +#define AVIC_HPA_MASK ~((0xFFFULL << 52) || 0xFFF) >

Re: [Xen-devel] Request for a repo on xenbits for the XenProject Openstack CI Loop

2016-11-17 Thread Anthony PERARD
On Thu, Nov 17, 2016 at 04:34:16PM +, Ian Jackson wrote: > Anthony PERARD writes ("Request for a repo on xenbits for the XenProject > Openstack CI Loop"): > > I would like a new repo on xenbits to hold the configuration of our > > OpenStack CI loop. Its name could be: > >

Re: [Xen-devel] [PATCH 01/35] libxl: add LIBXL_LOGD_* and LOG*D function families.

2016-11-17 Thread Wei Liu
On Thu, Nov 17, 2016 at 04:36:54PM +, Ian Jackson wrote: > Wei Liu writes ("Re: [PATCH 01/35] libxl: add LIBXL_LOGD_* and LOG*D function > families."): > > Oh, yes. I misunderstood. In that case libxl_is_stubdom could be handy. > > How about this: check whether the ao is a nested ao, and if

Re: [Xen-devel] [PATCH 01/35] libxl: add LIBXL_LOGD_* and LOG*D function families.

2016-11-17 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH 01/35] libxl: add LIBXL_LOGD_* and LOG*D function families."): > Oh, yes. I misunderstood. In that case libxl_is_stubdom could be handy. How about this: check whether the ao is a nested ao, and if so, fish the domid out of its parent ? Ian.

Re: [Xen-devel] [PATCH 01/35] libxl: add LIBXL_LOGD_* and LOG*D function families.

2016-11-17 Thread Wei Liu
On Thu, Nov 17, 2016 at 04:29:35PM +, Ian Jackson wrote: > Cedric Bosdonnat writes ("Re: [PATCH 01/35] libxl: add LIBXL_LOGD_* and LOG*D > function families."): > > On Thu, 2016-11-17 at 14:41 +, Wei Liu wrote: > > > You should be able to use ao->domid here, right? > > > > Oh indeed. I

Re: [Xen-devel] Request for a repo on xenbits for the XenProject Openstack CI Loop

2016-11-17 Thread Ian Jackson
Anthony PERARD writes ("Request for a repo on xenbits for the XenProject Openstack CI Loop"): > I would like a new repo on xenbits to hold the configuration of our > OpenStack CI loop. Its name could be: > git://xenbits.xen.org/openstack-ci-loop-config.git I am happy with this. But there may be

[Xen-devel] Request for a repo on xenbits for the XenProject Openstack CI Loop

2016-11-17 Thread Anthony PERARD
Hi, I would like a new repo on xenbits to hold the configuration of our OpenStack CI loop. Its name could be: git://xenbits.xen.org/openstack-ci-loop-config.git Right now, the repo is at https://github.com/citrix-openstack/os-ext-testing The repo contain everything needed if someone wanted to

Re: [Xen-devel] [PATCH 04/35] libxl.c: switch to LOG*D use

2016-11-17 Thread Ian Jackson
Cedric Bosdonnat writes ("Re: [PATCH 04/35] libxl.c: switch to LOG*D use"): > On Thu, 2016-11-17 at 14:50 +, Wei Liu wrote: > > The only thing I don't understand is why you had patch 2-4 for libxl.c. > > These three patches can probably be merged into one patch. > > That was to ease the

Re: [Xen-devel] [PATCH 01/35] libxl: add LIBXL_LOGD_* and LOG*D function families.

2016-11-17 Thread Ian Jackson
Cedric Bosdonnat writes ("Re: [PATCH 01/35] libxl: add LIBXL_LOGD_* and LOG*D function families."): > On Thu, 2016-11-17 at 14:41 +, Wei Liu wrote: > > You should be able to use ao->domid here, right? > > Oh indeed. I discovered this one after writing this patch and forgot > about it here.

Re: [Xen-devel] [XTF PATCH] Add a Live Patch privilege check test

2016-11-17 Thread Andrew Cooper
On 17/11/16 15:40, Ross Lagerwall wrote: > +typedef struct { > +union { > +void *p; > +uint64_t __attribute__((aligned(8))) q; > +}; > +} guest_handle_64; > + > +#define XEN_SYSCTL_INTERFACE_VERSION 0x000E /sigh This is going to require some rethinking. We cannot

Re: [Xen-devel] [PATCH 04/35] libxl.c: switch to LOG*D use

2016-11-17 Thread Wei Liu
On Thu, Nov 17, 2016 at 05:03:03PM +0100, Cedric Bosdonnat wrote: > On Thu, 2016-11-17 at 14:50 +, Wei Liu wrote: > > On Tue, Nov 15, 2016 at 11:18:42AM +0100, Cédric Bosdonnat wrote: > > > Use LOG*D functions to output the domain ID in logs as much as > > > possible. This will help consumer

Re: [Xen-devel] [PATCH 01/35] libxl: add LIBXL_LOGD_* and LOG*D function families.

2016-11-17 Thread Wei Liu
On Thu, Nov 17, 2016 at 05:01:08PM +0100, Cedric Bosdonnat wrote: > On Thu, 2016-11-17 at 14:41 +, Wei Liu wrote: > > On Tue, Nov 15, 2016 at 11:18:39AM +0100, Cédric Bosdonnat wrote: > > > From: Cédric Bosdonnat > > > > > > These functions should be used to log

Re: [Xen-devel] [PATCH v2] xen/gntdev: Use mempolicy instead of VM_IO flag to avoid NUMA balancing

2016-11-17 Thread Boris Ostrovsky
On 11/17/2016 11:01 AM, Olaf Hering wrote: > On Thu, Nov 17, Boris Ostrovsky wrote: > >> On 11/17/2016 06:28 AM, Olaf Hering wrote: >>> ERROR: "__mpol_dup" [drivers/xen/xen-gntdev.ko] undefined! >>> ERROR: "get_task_policy" [drivers/xen/xen-gntdev.ko] undefined! >> I just built 4.4.11 with this

Re: [Xen-devel] [RFC PATCH 5/9] x86/HVM/SVM: Add AVIC initialization code

2016-11-17 Thread Suravee Suthikulpanit
Konrad, Thanks for the review comments. Got one quick question below. On 10/12/16 15:02, Konrad Rzeszutek Wilk wrote: +int svm_avic_init_vmcb(struct vcpu *v) > +{ > +paddr_t ma; > +u32 apic_id_reg; > +struct arch_svm_struct *s = >arch.hvm_svm; > +struct vmcb_struct *vmcb =

Re: [Xen-devel] ACPI fixmap overflow

2016-11-17 Thread Alex Thorlton
On Thu, Nov 17, 2016 at 10:01:15AM -0500, Boris Ostrovsky wrote: > On 11/16/2016 05:08 AM, Jan Beulich wrote: > On 15.11.16 at 21:51, wrote: > >> On 11/15/2016 03:44 PM, Andrew Cooper wrote: > >>> On 15/11/2016 20:39, Boris Ostrovsky wrote: > On 11/15/2016

Re: [Xen-devel] [PATCH 04/35] libxl.c: switch to LOG*D use

2016-11-17 Thread Cedric Bosdonnat
On Thu, 2016-11-17 at 14:50 +, Wei Liu wrote: > On Tue, Nov 15, 2016 at 11:18:42AM +0100, Cédric Bosdonnat wrote: > > Use LOG*D functions to output the domain ID in logs as much as > > possible. This will help consumer code sorting the logs by > > domain. > > > > This commit includes all LOG*

Re: [Xen-devel] [PATCH v2] xen/gntdev: Use mempolicy instead of VM_IO flag to avoid NUMA balancing

2016-11-17 Thread Olaf Hering
On Thu, Nov 17, Boris Ostrovsky wrote: > On 11/17/2016 06:28 AM, Olaf Hering wrote: > > ERROR: "__mpol_dup" [drivers/xen/xen-gntdev.ko] undefined! > > ERROR: "get_task_policy" [drivers/xen/xen-gntdev.ko] undefined! > I just built 4.4.11 with this patch applied and haven't had any problems. Are

Re: [Xen-devel] [PATCH 01/35] libxl: add LIBXL_LOGD_* and LOG*D function families.

2016-11-17 Thread Cedric Bosdonnat
On Thu, 2016-11-17 at 14:41 +, Wei Liu wrote: > On Tue, Nov 15, 2016 at 11:18:39AM +0100, Cédric Bosdonnat wrote: > > From: Cédric Bosdonnat > > > > These functions should be used to log messages when the domain > > id is known. libxl__log will now prepend the log

  1   2   >