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
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 v
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
---
tools/libxc/xc_cp
> -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: Properly handle pi w
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 4e7872d2f71f511df155a
> 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 iommu
> 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 Tian
__
> 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
___
Xen-devel ma
> 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 on [6/7]
[5/7]?
>
> 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 file changed, 66 inser
> -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: Make sure PI is in p
> -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: Properly handle pi w
> -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: Permanently assign P
> -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: Properly handle pi d
> 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
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 host-install(3)
> 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
> 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 thes
> 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, sw
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 suppress the update
v6:
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. S
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.
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
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 missing notification event
during movin
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 next
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 a/xen/drivers/passthrough/vtd/intremap.c
b/x
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/drivers/passthrough/vtd/intremap.c | 14 +++---
1 file changed, 7 in
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 suppress the update
v6:
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
test-amd64-amd64-
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 28b3a713b66998a8be3e8
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 v
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 5ea2bad0d9ab6a2465195
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" , "John Haxby"
Cc: "x
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
test-amd64-amd64-
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 d67f14955a854474cf1a83
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
__get
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
__get
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 2048ab4b3ec966183eb87
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)
+{
+paddr
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 and
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 b/tools/libxl/libxl_vnuma.c
index db22799..8ec
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 a/tools/libxl/libxl_xshelp.c b/tools/libxl/libxl_xshelp.c
index b3bac
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 deletions(-)
diff --git a/tools/libxl/libxl_pci.
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 a/tools/libxl/libxl_remus.c b/tools/libxl/li
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 a/tools/libxl/libxl_stream_write.c b/tools/libxl/libxl_s
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 --git a/tools/libxl/libxl_dom_save.c b/tools/
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 b/tools/libxl/libxl_x86.c
index e1844c8..5da
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 a/tools/libxl/libxl_vtpm.c b/tools/libxl/libxl_vtpm.c
index 8588569
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 deletions(-)
diff --git a/tools/libxl/libxl_usb.
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 deletions(-)
diff --git a/tools/libxl/libxl_qmp.
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 a/tools/libxl/libxl_psr.c b/tools/libxl/libxl_psr.c
index 78618
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 provided to the LOG*D f
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 deletions(-)
diff --git a/tools/libxl/libxl_dm.c
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 a/tools/libxl/libxl_no_colo.c b/tools/libxl/libxl_no_colo.c
index 15
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 a/tools/libxl/libxl_nic.c b/tools/libxl/libxl_nic.c
index 220a028..
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 deletions(-)
diff --git a/tools/libxl/libxl_netb
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 a/tools/libxl/libxl_save_callout.c b/tools/libxl/libxl_save
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 deletions(-)
diff --git a/tools/libxl/libxl_colo
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.
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 --git a/tools/libxl/libxl_internal.c b/tools/
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 a/tools/libxl/libxl_colo_qdisk.c b/tools/libxl/libxl_colo_qdisk.c
ind
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 b/tools/libxl/libxl_colo.h
index f0e438e..682275
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 a/tools/libxl/libxl_linux.c b/tools/libxl/libxl_l
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 deletions(-)
diff --git a/tools/libxl/libxl_devi
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 a/tools/libxl/libxl_colo_nic.c b/tools/libxl/libxl_colo_nic.c
index 5e0
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 a/tools/libxl/libxl_freebsd.c b/tools/libxl/libxl_freebsd.c
inde
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 deletions(-)
diff --git a/tools/libxl/libxl_crea
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 altered to
remove the domain
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 deletions(-)
diff --git a/tools/libxl/libxl_dom_
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 deletions(-)
diff --git a/tools/libxl/libxl_colo
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 a/tools/libxl/libxl_netbsd.c b/tools/libxl/libxl_netbsd.c
index
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 --
1 file changed, 49 insert
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 a/tools/libxl/libxl_colo_proxy.c b/tools/l
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 a/tools/libxl/libxl_checkpoint_device.c
b/tool
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: Cédric Bosdonnat
---
v
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.
This commit, only cha
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&m=147639616310487
> >
> >
> > I have already shown that it performs better than netfront/netback on
> > x86 in this blog post:
> >
> >
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 Stefan
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 AVI
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;
> > > > +u32
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 e242cdfb307a6dfe2c0f7
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 1
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)
> +#defin
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:
> > git://xenbits.xen.org
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 s
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.
__
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 di
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
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 red
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 review
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. Co
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 have
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 co
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 messages when the domain
> > >
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 pat
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 = &v->arch.hvm_svm;
> +struct vmcb_struct *vmcb = s-
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 02:45 PM, Andrew Cooper 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 code sorting the logs by
> > domain.
> >
> > This commit includes all LOG*
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 th
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 message by
> > "Domain %PRIu
1 - 100 of 138 matches
Mail list logo
