Re: [Xen-devel] [PATCH v6 11/18] tools/libxl: Add back channel to allow migration target send data back

2016-01-25 Thread Wen Congyang
On 01/26/2016 03:17 AM, Konrad Rzeszutek Wilk wrote: > On Wed, Dec 30, 2015 at 10:29:01AM +0800, Wen Congyang wrote: >> In colo mode, slave needs to send data to master, but the io_fd > > > > In previous patches you used COLO in all caps, can that be uniform > across the patches? OK, I will che

Re: [Xen-devel] [PATCH V13 3/5] libxl: add pvusb API

2016-01-25 Thread Chun Yan Liu
>>> On 1/20/2016 at 12:56 PM, in message <569f83f502660009e...@relay2.provo.novell.com>, "Chun Yan Liu" wrote: > On 1/19/2016 at 11:48 PM, in message > <22174.23240.402164.635...@mariner.uk.xensource.com>, Ian Jackson > wrote: > > Chunyan Liu writes ("[PATCH V13 3/5] libxl: add

Re: [Xen-devel] [PATCH v2 3/3] tools: introduce parameter max_wp_ram_ranges.

2016-01-25 Thread Yu, Zhang
Thank you, Jan. On 1/22/2016 4:01 PM, Jan Beulich wrote: On 22.01.16 at 04:20, wrote: --- a/xen/arch/x86/hvm/hvm.c +++ b/xen/arch/x86/hvm/hvm.c @@ -940,6 +940,10 @@ static int hvm_ioreq_server_alloc_rangesets(struct hvm_ioreq_server *s, { unsigned int i; int rc; +unsigned int

[Xen-devel] [linux-3.14 test] 78986: tolerable FAIL - PUSHED

2016-01-25 Thread osstest service owner
flight 78986 linux-3.14 real [real] http://logs.test-lab.xenproject.org/osstest/logs/78986/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-i386-rumpuserxen-i386 10 guest-startfail like 65787 test-amd64-i386-xl-qemut-win7-a

Re: [Xen-devel] [PATCH v6 09/18] tools/libxl: introduce libxl__domain_common_switch_qemu_logdirty()

2016-01-25 Thread Wen Congyang
On 01/26/2016 02:59 AM, Konrad Rzeszutek Wilk wrote: > On Wed, Dec 30, 2015 at 10:28:59AM +0800, Wen Congyang wrote: >> Secondary vm is running in colo mode, we need to send >> secondary vm's dirty page information to master at checkpoint, > > In previous patch you called it primary, so perhaps: >

Re: [Xen-devel] [PATCH v3 4/5] tools/libxc: error handling for the postcopy() callback

2016-01-25 Thread Yang Hongyang
On 01/26/2016 02:48 PM, Wen Congyang wrote: On 01/26/2016 02:45 PM, Yang Hongyang wrote: ditto Reviewed-by: Yang Hongyang The newest version is v5, and this series is in the staging now. Sorry for the noise...I saw the series too late, please ignore my comments... Thanks for your revi

Re: [Xen-devel] [PATCH v3 4/5] tools/libxc: error handling for the postcopy() callback

2016-01-25 Thread Wen Congyang
On 01/26/2016 02:45 PM, Yang Hongyang wrote: > ditto > > Reviewed-by: Yang Hongyang The newest version is v5, and this series is in the staging now. Thanks for your review. Wen Congyang > > On 01/08/2016 02:38 PM, Wen Congyang wrote: >> Signed-off-by: Wen Congyang >> Reviewed-by: Andrew Coop

Re: [Xen-devel] [PATCH v3 4/5] tools/libxc: error handling for the postcopy() callback

2016-01-25 Thread Yang Hongyang
ditto Reviewed-by: Yang Hongyang On 01/08/2016 02:38 PM, Wen Congyang wrote: Signed-off-by: Wen Congyang Reviewed-by: Andrew Cooper --- tools/libxc/xc_sr_save.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tools/libxc/xc_sr_save.c b/tools/libxc/xc_sr_save.c index

Re: [Xen-devel] [PATCH v3 3/5] tools/libxc: don't send end record if remus fails

2016-01-25 Thread Yang Hongyang
some commit message would be better. Reviewed-by: Yang Hongyang On 01/08/2016 02:38 PM, Wen Congyang wrote: Signed-off-by: Wen Congyang Reviewed-by: Andrew Cooper --- tools/libxc/xc_sr_save.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/libxc/xc_sr_save.c b/to

Re: [Xen-devel] [PATCH v3 2/5] remus: resume immediately if libxl__xc_domain_save_done() completes

2016-01-25 Thread Yang Hongyang
On 01/08/2016 02:38 PM, Wen Congyang wrote: For example: if the secondary host is down, and we fail to send the data to the secondary host. xc_domain_save() returns 0. So in the function libxl__xc_domain_save_done(), rc is 0(the helper program exits normally), and retval is 0(it is xc_domain_sa

Re: [Xen-devel] [PATCH v3 1/5] remus: don't call stream_continue() when doing failover

2016-01-25 Thread Yang Hongyang
On 01/08/2016 02:38 PM, Wen Congyang wrote: stream_continue() is used for migration to read emulator xenstore data and emulator context. For remus, if we do failover, we have read it in the checkpoint cycle, and we only need to complete the stream. Reviewed-by: Yang Hongyang Signed-off-by

Re: [Xen-devel] [PATCH v5 COLOPre 08/18] tools/libxl: introduce libxl__domain_restore_device_model to load qemu state

2016-01-25 Thread Wen Congyang
On 01/26/2016 02:41 AM, Konrad Rzeszutek Wilk wrote: > On Thu, Dec 17, 2015 at 03:48:12PM +0800, Wen Congyang wrote: >> From: Yang Hongyang >> >> In normal migration, the qemu state was passed to qemu as a parameter. > > /was/is/ > >> With COLO, Secondary vm is running. So we will do the followi

Re: [Xen-devel] [v3,11/41] mips: reuse asm-generic/barrier.h

2016-01-25 Thread Paul E. McKenney
On Mon, Jan 25, 2016 at 06:02:34PM +, Will Deacon wrote: > Hi Paul, > > On Fri, Jan 15, 2016 at 09:39:12AM -0800, Paul E. McKenney wrote: > > On Fri, Jan 15, 2016 at 09:55:54AM +0100, Peter Zijlstra wrote: > > > On Thu, Jan 14, 2016 at 01:29:13PM -0800, Paul E. McKenney wrote: > > > > So smp_m

Re: [Xen-devel] [v3,11/41] mips: reuse asm-generic/barrier.h

2016-01-25 Thread Paul E. McKenney
On Mon, Jan 25, 2016 at 04:42:43PM +, Will Deacon wrote: > On Fri, Jan 15, 2016 at 01:58:53PM -0800, Paul E. McKenney wrote: > > On Fri, Jan 15, 2016 at 10:27:14PM +0100, Peter Zijlstra wrote: > > > On Fri, Jan 15, 2016 at 09:46:12AM -0800, Paul E. McKenney wrote: > > > > On Fri, Jan 15, 2016 a

Re: [Xen-devel] [PATCH v5 COLOPre 07/18] migration/save: pass checkpointed_stream from libxl to libxc

2016-01-25 Thread Wen Congyang
On 01/26/2016 03:14 AM, Konrad Rzeszutek Wilk wrote: > On Mon, Jan 25, 2016 at 01:38:36PM -0500, Konrad Rzeszutek Wilk wrote: >> On Thu, Dec 17, 2015 at 03:48:11PM +0800, Wen Congyang wrote: >>> From: Yang Hongyang >>> >>> Pass checkpointed_stream from libxl to libxc. >>> It won't affact legacy mi

Re: [Xen-devel] [v3,11/41] mips: reuse asm-generic/barrier.h

2016-01-25 Thread Paul E. McKenney
On Mon, Jan 25, 2016 at 02:41:34PM +, Will Deacon wrote: > On Fri, Jan 15, 2016 at 11:28:45AM -0800, Paul E. McKenney wrote: > > On Fri, Jan 15, 2016 at 09:54:01AM -0800, Paul E. McKenney wrote: > > > On Fri, Jan 15, 2016 at 10:24:32AM +, Will Deacon wrote: > > > > See my earlier reply [1]

[Xen-devel] [PATCH] xen vtd : set msi guest_masked 0 by default

2016-01-25 Thread Jianzhong,Chang
There are some problems when msi guest_masked is set to 1 by default. When guest os is windows 2008 r2 server, the device(eg X540-AT2 vf) is not initialized correctly. Host will always receive message like this :"VF Reset msg received from vf". Guest has network connectivity issues, and can not cor

[Xen-devel] [linux-3.10 test] 78980: tolerable FAIL - PUSHED

2016-01-25 Thread osstest service owner
flight 78980 linux-3.10 real [real] http://logs.test-lab.xenproject.org/osstest/logs/78980/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-amd64-rumpuserxen-amd64 15 rumpuserxen-demo-xenstorels/xenstorels.repeat fail like 65778 test-amd6

Re: [Xen-devel] [PATCH v6 00/18] Prerequisite patches for COLO

2016-01-25 Thread Wen Congyang
On 01/26/2016 04:06 AM, Konrad Rzeszutek Wilk wrote: > On Mon, Jan 25, 2016 at 12:12:48PM -0500, Konrad Rzeszutek Wilk wrote: >> On Wed, Dec 30, 2015 at 10:28:50AM +0800, Wen Congyang wrote: >>> This patchset is Prerequisite for COLO feature. Refer to: >>> http://wiki.xen.org/wiki/COLO_-_Coarse_Gra

Re: [Xen-devel] [PATCH v6 05/18] tools/libxc: support to resume uncooperative HVM guests

2016-01-25 Thread Wen Congyang
On 01/26/2016 02:21 AM, Konrad Rzeszutek Wilk wrote: > On Wed, Dec 30, 2015 at 10:28:55AM +0800, Wen Congyang wrote: >> Befor this patch: > > s/Befor/Before >> 1. suspend >> a. PVHVM and PV: we use the same way to suspend the guest (send the suspend >>request to the guest). If the guest doesn'

Re: [Xen-devel] [PATCH v6 04/18] libxl/save: Refactor libxl__domain_suspend_state

2016-01-25 Thread Wen Congyang
On 01/26/2016 01:29 AM, Konrad Rzeszutek Wilk wrote: > .snip.. >> --- a/tools/libxl/libxl_dom_suspend.c >> +++ b/tools/libxl/libxl_dom_suspend.c >> @@ -19,14 +19,71 @@ >> >> /*== Domain suspend ===*/ >> >> +int libxl__domain_suspend_init(libxl__egc *egc,

[Xen-devel] [RFC] support more qdisk types

2016-01-25 Thread Jim Fehlig
I would like to hear the community's opinion on supporting more qdisk types in xl/libxl, e.g. nbd, rbd, iSCSI, etc. I prefer supporting additional qdisk types in libxl over apps like xl or libvirt doing all the setup, producing a block device, and then passing that to libxl. Each libxl app would ha

[Xen-devel] [linux-linus test] 78977: regressions - FAIL

2016-01-25 Thread osstest service owner
flight 78977 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/78977/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-xl-cubietruck 6 xen-bootfail REGR. vs. 59254 test-armhf-armhf-xl-x

Re: [Xen-devel] [PATCH 5/5] Allow all user to create a file under the directory /var/lib/xen

2016-01-25 Thread Andrew Cooper
On 25/01/2016 20:36, Konrad Rzeszutek Wilk wrote: > On Wed, Dec 30, 2015 at 11:00:52AM +, Andrew Cooper wrote: >> On 30/12/2015 05:25, Wen Congyang wrote: >>> On 12/30/2015 12:11 PM, Doug Goldstein wrote: On 12/29/15 8:39 PM, Wen Congyang wrote: > We may use non-root user to run qemu,

Re: [Xen-devel] [PATCH V2 1/1] Improved RTDS scheduler

2016-01-25 Thread Meng Xu
On Mon, Jan 25, 2016 at 5:04 PM, Tianyang Chen wrote: > I have removed some of the Ccs so they won't get bothered as we discussed > previously. > > On 1/25/2016 4:00 AM, Dario Faggioli wrote: >> >> On Thu, 2015-12-31 at 05:20 -0500, Tianyang Chen wrote: >>> >>> >>> @@ -147,6 +148,16 @@ static unsi

Re: [Xen-devel] [PATCH v1 04/12] xen/hvmlite: Bootstrap HVMlite guest

2016-01-25 Thread Boris Ostrovsky
On 01/25/2016 05:19 PM, Luis R. Rodriguez wrote: On Sat, Jan 23, 2016 at 02:49:36PM +, Andrew Cooper wrote: it causes inappropriate linkage between the toolstack and the version of Linux wishing to be booted. There are ways to do it in such a way that it does not create dependency issues

Re: [Xen-devel] [PATCH v1 04/12] xen/hvmlite: Bootstrap HVMlite guest

2016-01-25 Thread Boris Ostrovsky
On 01/25/2016 04:21 PM, H. Peter Anvin wrote: On 01/25/16 13:12, Luis R. Rodriguez wrote: Perhaps, but someone would still have to set hardware_subarch. And it's hvmlite_bootparams() that does it. No, Xen would do it as well, essentially all of hvmlite_bootparams() could be done in Xen. Or a

Re: [Xen-devel] [PATCH v1 04/12] xen/hvmlite: Bootstrap HVMlite guest

2016-01-25 Thread Luis R. Rodriguez
On Sat, Jan 23, 2016 at 02:49:36PM +, Andrew Cooper wrote: > On 23/01/2016 00:55, Luis R. Rodriguez wrote: > > On Fri, Jan 22, 2016 at 4:30 PM, Andrew Cooper > > wrote: > >> the DMLite boot > >> protocol is OS agnostic, and will be staying that way. > > What's the DMLite boot protocol? > > It

Re: [Xen-devel] [PATCH V2 1/1] Improved RTDS scheduler

2016-01-25 Thread Tianyang Chen
I have removed some of the Ccs so they won't get bothered as we discussed previously. On 1/25/2016 4:00 AM, Dario Faggioli wrote: On Thu, 2015-12-31 at 05:20 -0500, Tianyang Chen wrote: @@ -147,6 +148,16 @@ static unsigned int nr_rt_ops; * Global lock is referenced by schedule_data.schedul

Re: [Xen-devel] pre Sandy bridge IOMMU support (gm45)

2016-01-25 Thread Thierry Laurion
The case is 1) disabling iommu for IGD, unilaterally since i915 + gm45 doesn't play well together. Iommu is still desired to isolate usb and network devices, so we don't want to disable iommu completely. The side effect of this would be to have IGD only for dom0, which would also completely make se

Re: [Xen-devel] [PATCH v3 00/17] Enhance iomem search interfaces and support EINJ to NVDIMM

2016-01-25 Thread Borislav Petkov
On Mon, Jan 25, 2016 at 02:34:21PM -0700, Toshi Kani wrote: > I verified the patches and tested the kernel in the tree. All look > good. Thanks! -- Regards/Gruss, Boris. ECO tip #101: Trim your mails when you reply. ___ Xen-devel mailing list Xen

Re: [Xen-devel] [PATCH v3 00/17] Enhance iomem search interfaces and support EINJ to NVDIMM

2016-01-25 Thread Toshi Kani
On Mon, 2016-01-25 at 20:18 +0100, Borislav Petkov wrote: > On Tue, Jan 05, 2016 at 11:54:28AM -0700, Toshi Kani wrote: > > This patch-set enhances the iomem table and its search interfacs, and > > then changes EINJ to support NVDIMM. > > : > > Ok, all applied ontop of 4.5-rc1. > > You could ta

Re: [Xen-devel] [PATCH v1 04/12] xen/hvmlite: Bootstrap HVMlite guest

2016-01-25 Thread H. Peter Anvin
On 01/25/16 13:12, Luis R. Rodriguez wrote: >> >> Perhaps, but someone would still have to set hardware_subarch. And >> it's hvmlite_bootparams() that does it. > > No, Xen would do it as well, essentially all of hvmlite_bootparams() could be > done in Xen. > Or a stub code. -hpa

Re: [Xen-devel] [PATCH v1 04/12] xen/hvmlite: Bootstrap HVMlite guest

2016-01-25 Thread Luis R. Rodriguez
On Mon, Jan 25, 2016 at 11:08:47AM -0500, Boris Ostrovsky wrote: > On 01/22/2016 06:32 PM, Luis R. Rodriguez wrote: > >On Fri, Jan 22, 2016 at 04:35:50PM -0500, Boris Ostrovsky wrote: > > > >>+/* > >>+ * This routine (and those that it might call) should not use > >>+ * anything that lives in .bss

[Xen-devel] [PATCH] toolstack documentation/asserts cleanups. (v1)

2016-01-25 Thread Konrad Rzeszutek Wilk
Hey! As I was reviewing the v6 of the remus/COLO patches I ran into some parts of code that was not exactly clear to me. These patches update the comments or move the code a bit to make more sense. Please review at your convience. tools/libxc/xc_resume.c | 7 +-- tools/libxl/libx

[Xen-devel] [PATCH 1/3] libxc/xc_domain_resume: Update comment.

2016-01-25 Thread Konrad Rzeszutek Wilk
To hopefully clarify what it meant. Signed-off-by: Konrad Rzeszutek Wilk --- tools/libxc/xc_resume.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/tools/libxc/xc_resume.c b/tools/libxc/xc_resume.c index 87d4324..19ba2a3 100644 --- a/tools/libxc/xc_resume.c +++ b/tool

[Xen-devel] [PATCH 3/3] tools/libxl: run_helper - add #define for arguments.

2016-01-25 Thread Konrad Rzeszutek Wilk
Describe what the four (or more in the future) arguments are for. Signed-off-by: Konrad Rzeszutek Wilk --- tools/libxl/libxl_save_callout.c | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/tools/libxl/libxl_save_callout.c b/tools/libxl/libxl_save_callout.c index 3a

[Xen-devel] [PATCH 2/3] libxl/remus: Move the assert before the info is used.

2016-01-25 Thread Konrad Rzeszutek Wilk
The assert(info) is after quite a lot of manipulations on 'info' - which makes the assert pointless because if info was NULL it would have crashed earlier. Move it earlier so that it guards before we try using the 'info' structure. CC: Wen Congyang CC: Yang Hongyang Signed-off-by: Konrad Rzeszu

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

2016-01-25 Thread osstest service owner
flight 79041 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/79041/ 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 12

Re: [Xen-devel] [PATCH 5/5] Allow all user to create a file under the directory /var/lib/xen

2016-01-25 Thread Konrad Rzeszutek Wilk
On Wed, Dec 30, 2015 at 11:00:52AM +, Andrew Cooper wrote: > On 30/12/2015 05:25, Wen Congyang wrote: > >On 12/30/2015 12:11 PM, Doug Goldstein wrote: > >>On 12/29/15 8:39 PM, Wen Congyang wrote: > >>>We may use non-root user to run qemu, and the qemu needs to write > >>>save file to /var/lib/x

Re: [Xen-devel] [PATCH RFC 3/6] x86/time: streamline platform time init on plt_init()

2016-01-25 Thread Konrad Rzeszutek Wilk
> +if ( clocksource_is_tsc ) > +{ > +plt_init(); > +} > +else > +{ > +plt_overflow_period = scale_delta( > +1ull << (pts->counter_bits-1), &plt_scale); > +init_timer(&plt_overflow_timer, plt_overflow, NULL, 0); > +plt_overflow(NULL); >

[Xen-devel] [linux-mingo-tip-master test] 78976: regressions - FAIL

2016-01-25 Thread osstest service owner
flight 78976 linux-mingo-tip-master real [real] http://logs.test-lab.xenproject.org/osstest/logs/78976/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-i386-pvops 5 kernel-build fail REGR. vs. 60684 build-amd6

Re: [Xen-devel] [PATCH RFC 1/6] public/xen.h: add flags field to vcpu_time_info

2016-01-25 Thread Konrad Rzeszutek Wilk
On Mon, Dec 28, 2015 at 04:59:40PM +, Joao Martins wrote: > This field has two possible flags (as of latest pvclock ABI > shared with KVM). Wish they had CC-ed xen-devel instead of just doing their change > > flags: bits in this field indicate extended capabilities > coordinated between the

Re: [Xen-devel] [PATCH v6 00/18] Prerequisite patches for COLO

2016-01-25 Thread Konrad Rzeszutek Wilk
On Mon, Jan 25, 2016 at 12:12:48PM -0500, Konrad Rzeszutek Wilk wrote: > On Wed, Dec 30, 2015 at 10:28:50AM +0800, Wen Congyang wrote: > > This patchset is Prerequisite for COLO feature. Refer to: > > http://wiki.xen.org/wiki/COLO_-_Coarse_Grain_Lock_Stepping > > > > It was based on the following

Re: [Xen-devel] [PATCH v6 18/18] tools/libxl: seperate device init/cleanup from checkpoint device layer

2016-01-25 Thread Konrad Rzeszutek Wilk
On Wed, Dec 30, 2015 at 10:29:08AM +0800, Wen Congyang wrote: > we call (init|cleanup)_subkind_nic and (init|cleanup)_subkind_drbd_disk > directly in checkpoint device. Move them to libxl_remus.c, Call them before > calling libxl__checkpoint_devices_setup() or after calling > libxl__checkpoint_devi

Re: [Xen-devel] [PATCH v6 17/18] tools/libxl: move remus state into a seperate structure

2016-01-25 Thread Konrad Rzeszutek Wilk
On Wed, Dec 30, 2015 at 10:29:07AM +0800, Wen Congyang wrote: > Add a new structure remus state, and move concrete layer's private > member to remus state. > it is pure refactoring and no functional changes. > Init interval in libxl__remus_setup(). It is safe to move this initialisation, > because

Re: [Xen-devel] [PATCH v6 16/18] tools/libxl: store remus_ops in checkpoint device state

2016-01-25 Thread Konrad Rzeszutek Wilk
On Wed, Dec 30, 2015 at 10:29:06AM +0800, Wen Congyang wrote: > Checkpoint device is an abstract layer to do checkpoint. > COLO can also use it to do checkpoint. But there are > still some codes in checkpoint device which touch remus. > > This patch and the following 2 will seperate remus from s/

Re: [Xen-devel] [PATCH v6 15/18] tools/libxl: adjust the indentation

2016-01-25 Thread Konrad Rzeszutek Wilk
On Wed, Dec 30, 2015 at 10:29:05AM +0800, Wen Congyang wrote: > This is just tidying up after the previous automatic renaming. s/previous/ tools/libxl: rename remus device to checkpoint device patch/ > > Signed-off-by: Yang Hongyang > Signed-off-by: Wen Congyang > Acked-by: Ian Campbell Revi

Re: [Xen-devel] [PATCH v5 COLOPre 14/18] tools/libxl: fix backword compatibility after the automatic renaming

2016-01-25 Thread Konrad Rzeszutek Wilk
On Thu, Dec 17, 2015 at 03:48:18PM +0800, Wen Congyang wrote: > From: Yang Hongyang > > The error code ERROR_REMUS_XXX was introduced in Xen 4.5, and > changed to ERROR_CHECKPOINT_XXX after previous renaming. > The patch fix the backword compatibility. > > Signed-off-by: Yang Hongyang > Signed-

Re: [Xen-devel] [PATCH v6 12/18] tools/libx{l, c}: add back channel to libxc

2016-01-25 Thread Konrad Rzeszutek Wilk
On Wed, Dec 30, 2015 at 10:29:02AM +0800, Wen Congyang wrote: > In COLO mode, both VMs are running, and are considered in sync if the > visible network traffic is identical. After some time, they fall out of > sync. > > At this point, the two VMs have definitely diverged. Lets call the > primary

Re: [Xen-devel] [PATCH v6 13/18] tools/libxl: rename remus device to checkpoint device

2016-01-25 Thread Konrad Rzeszutek Wilk
On Wed, Dec 30, 2015 at 10:29:03AM +0800, Wen Congyang wrote: > This patch is auto generated by the following commands: > 1. git mv tools/libxl/libxl_remus_device.c > tools/libxl/libxl_checkpoint_device.c > 2. perl -pi -e 's/libxl_remus_device/libxl_checkpoint_device/g' > tools/libxl/Makefile >

[Xen-devel] [libvirt test] 78978: tolerable FAIL - PUSHED

2016-01-25 Thread osstest service owner
flight 78978 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/78978/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-armhf-armhf-libvirt-raw 13 guest-saverestorefail never pass test-armhf-armhf-libvirt-raw 11 migrate-sup

Re: [Xen-devel] [PATCH v3 00/17] Enhance iomem search interfaces and support EINJ to NVDIMM

2016-01-25 Thread Borislav Petkov
On Tue, Jan 05, 2016 at 11:54:28AM -0700, Toshi Kani wrote: > This patch-set enhances the iomem table and its search interfacs, and > then changes EINJ to support NVDIMM. > > - Patches 1-2 add a new System RAM type, IORESOURCE_SYSTEM_RAM, and >make the iomem search interfaces work with resour

Re: [Xen-devel] [PATCH v6 11/18] tools/libxl: Add back channel to allow migration target send data back

2016-01-25 Thread Konrad Rzeszutek Wilk
On Wed, Dec 30, 2015 at 10:29:01AM +0800, Wen Congyang wrote: > In colo mode, slave needs to send data to master, but the io_fd In previous patches you used COLO in all caps, can that be uniform across the patches? Also, slave == secondary and master == primary? Perhaps you could s/slave/second

Re: [Xen-devel] [PATCH v5 COLOPre 07/18] migration/save: pass checkpointed_stream from libxl to libxc

2016-01-25 Thread Konrad Rzeszutek Wilk
On Mon, Jan 25, 2016 at 01:38:36PM -0500, Konrad Rzeszutek Wilk wrote: > On Thu, Dec 17, 2015 at 03:48:11PM +0800, Wen Congyang wrote: > > From: Yang Hongyang > > > > Pass checkpointed_stream from libxl to libxc. > > It won't affact legacy migration because legacy migration > > won't use this par

[Xen-devel] [qemu-mainline test] 78928: trouble: broken/fail/pass

2016-01-25 Thread osstest service owner
flight 78928 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/78928/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemuu-debianhvm-amd64-xsm 3 host-install(3) broken REGR. v

Re: [Xen-devel] [PATCH v6 09/18] tools/libxl: introduce libxl__domain_common_switch_qemu_logdirty()

2016-01-25 Thread Konrad Rzeszutek Wilk
On Wed, Dec 30, 2015 at 10:28:59AM +0800, Wen Congyang wrote: > Secondary vm is running in colo mode, we need to send > secondary vm's dirty page information to master at checkpoint, In previous patch you called it primary, so perhaps: s/master/primary/ ? > so we have to enable qemu logdirty on s

Re: [Xen-devel] [PATCH v6 10/18] tools/libxl: export logdirty_init

2016-01-25 Thread Konrad Rzeszutek Wilk
On Wed, Dec 30, 2015 at 10:29:00AM +0800, Wen Congyang wrote: > We need to enable logdirty on secondary, so we export logdirty_init > for internal use. Rename it to libxl__logdirty_init. > > Signed-off-by: Yang Hongyang > Signed-off-by: Wen Congyang > Acked-by: Ian Campbell Reviewed-by: Konrad

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

2016-01-25 Thread osstest service owner
flight 79031 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/79031/ 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 12

Re: [Xen-devel] [PATCH v5 COLOPre 08/18] tools/libxl: introduce libxl__domain_restore_device_model to load qemu state

2016-01-25 Thread Konrad Rzeszutek Wilk
On Thu, Dec 17, 2015 at 03:48:12PM +0800, Wen Congyang wrote: > From: Yang Hongyang > > In normal migration, the qemu state was passed to qemu as a parameter. /was/is/ > With COLO, Secondary vm is running. So we will do the following steps > at every checkpoint: > 1. suspend both primary vm and

Re: [Xen-devel] [PATCH v5 COLOPre 07/18] migration/save: pass checkpointed_stream from libxl to libxc

2016-01-25 Thread Konrad Rzeszutek Wilk
On Thu, Dec 17, 2015 at 03:48:11PM +0800, Wen Congyang wrote: > From: Yang Hongyang > > Pass checkpointed_stream from libxl to libxc. > It won't affact legacy migration because legacy migration > won't use this param. Ah, that is why you wanted to keep it as in 'int' in the struct domain_create.

Re: [Xen-devel] [PATCH v6 06/18] tools/libxl: introduce enum type libxl_checkpointed_stream

2016-01-25 Thread Konrad Rzeszutek Wilk
On Wed, Dec 30, 2015 at 10:28:56AM +0800, Wen Congyang wrote: > introduce enum type libxl_checkpointed_stream in IDL. s/introduce/Introduce/ > rename the last argument of migrate_receive from "remus" to > "checkpointed" since the semantics of this parameter has > changed. > > NOTE: > libxl_domai

Re: [Xen-devel] [PATCH v6 05/18] tools/libxc: support to resume uncooperative HVM guests

2016-01-25 Thread Konrad Rzeszutek Wilk
On Wed, Dec 30, 2015 at 10:28:55AM +0800, Wen Congyang wrote: > Befor this patch: s/Befor/Before > 1. suspend > a. PVHVM and PV: we use the same way to suspend the guest (send the suspend >request to the guest). If the guest doesn't support evtchn, the xenstore >variant will be used, suspe

Re: [Xen-devel] [PATCH V2 1/1] Improved RTDS scheduler

2016-01-25 Thread Dario Faggioli
On Thu, 2015-12-31 at 05:20 -0500, Tianyang Chen wrote: >  > @@ -147,6 +148,16 @@ static unsigned int nr_rt_ops; >   * Global lock is referenced by schedule_data.schedule_lock from all >   * physical cpus. It can be grabbed via vcpu_schedule_lock_irq() >   */ > + > +/* dedicated timer for replenish

[Xen-devel] [ovmf test] 78929: regressions - FAIL

2016-01-25 Thread osstest service owner
flight 78929 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/78929/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemuu-ovmf-amd64 17 guest-start/debianhvm.repeat fail REGR. vs. 65543 test-amd64-amd64-

Re: [Xen-devel] [v3,11/41] mips: reuse asm-generic/barrier.h

2016-01-25 Thread Will Deacon
Hi Paul, On Fri, Jan 15, 2016 at 09:39:12AM -0800, Paul E. McKenney wrote: > On Fri, Jan 15, 2016 at 09:55:54AM +0100, Peter Zijlstra wrote: > > On Thu, Jan 14, 2016 at 01:29:13PM -0800, Paul E. McKenney wrote: > > > So smp_mb() provides transitivity, as do pairs of smp_store_release() > > > and s

Re: [Xen-devel] [PATCH 1/2] tools: rename libxc's evtchn_port_or_error_t with an xc_ prefix

2016-01-25 Thread Olaf Hering
On Mon, Jan 25, Ian Campbell wrote: > This is used only for xc_evtchn_alloc_unbound and the legacy/compat > versions of the old interfaces and avoids redefining the typedef. The > evtchn_port_or_error_t name is now used only be libxenevtchn. Tested-by: Olaf Hering Olaf

Re: [Xen-devel] [PATCH 2/2] tools: avoid redefinining xenevtchn_handle typedef for xc_suspend_*

2016-01-25 Thread Olaf Hering
On Mon, Jan 25, Ian Campbell wrote: > Similar to the previous xentoollog case this is not allowed. Switch to > a forward decl of the struct and use of it in the APIs. Tested-by: Olaf Hering Olaf ___ Xen-devel mailing list Xen-devel@lists.xen.org http

Re: [Xen-devel] [PATCH 2/2] tools: avoid redefinining xenevtchn_handle typedef for xc_suspend_*

2016-01-25 Thread Boris Ostrovsky
On 01/25/2016 12:22 PM, Ian Jackson wrote: Ian Campbell writes ("[PATCH 2/2] tools: avoid redefinining xenevtchn_handle typedef for xc_suspend_*"): Similar to the previous xentoollog case this is not allowed. Switch to a forward decl of the struct and use of it in the APIs. Both of these Acke

Re: [Xen-devel] [PATCH v6 04/18] libxl/save: Refactor libxl__domain_suspend_state

2016-01-25 Thread Konrad Rzeszutek Wilk
.snip.. > --- a/tools/libxl/libxl_dom_suspend.c > +++ b/tools/libxl/libxl_dom_suspend.c > @@ -19,14 +19,71 @@ > > /*== Domain suspend ===*/ > > +int libxl__domain_suspend_init(libxl__egc *egc, > + libxl__domain_suspend_state

Re: [Xen-devel] [PATCH v6 00/18] Prerequisite patches for COLO

2016-01-25 Thread Konrad Rzeszutek Wilk
On Wed, Dec 30, 2015 at 10:28:50AM +0800, Wen Congyang wrote: > This patchset is Prerequisite for COLO feature. Refer to: > http://wiki.xen.org/wiki/COLO_-_Coarse_Grain_Lock_Stepping > > It was based on the following series: > http://lists.xenproject.org/archives/html/xen-devel/2015-12/msg02881.ht

Re: [Xen-devel] [PATCH v6 01/18] libxl/remus: init checkpoint_callback in Remus setup callback

2016-01-25 Thread Konrad Rzeszutek Wilk
On Wed, Dec 30, 2015 at 10:28:51AM +0800, Wen Congyang wrote: > init stream {read/write} state checkpoint_callback in Remus setup callback. > There's no functional change, it's just refactoring so that we can move > all remus code into one file. > > Signed-off-by: Yang Hongyang > Signed-off-by: W

Re: [Xen-devel] [PATCH 2/2] tools: avoid redefinining xenevtchn_handle typedef for xc_suspend_*

2016-01-25 Thread Ian Jackson
Ian Campbell writes ("[PATCH 2/2] tools: avoid redefinining xenevtchn_handle typedef for xc_suspend_*"): > Similar to the previous xentoollog case this is not allowed. Switch to > a forward decl of the struct and use of it in the APIs. Both of these Acked-by: Ian Jackson Committed-by: Ian Jacks

Re: [Xen-devel] [PATCH] tools: avoid redefinition of typedefs

2016-01-25 Thread Ian Campbell
On Mon, 2016-01-25 at 16:37 +, Ian Campbell wrote: > On Mon, 2016-01-25 at 17:24 +0100, Olaf Hering wrote: > > On Mon, Jan 25, Ian Campbell wrote: > > > > > When splitting out various functionality from libxc into tools/libs/* > > > I attempted to make it possible to avoid callers being unnece

[Xen-devel] Revokable Grants Design (draft B)

2016-01-25 Thread David Vrabel
% Revokable Grants % David Vrabel <> % Draft B http://xenbits.xen.org/people/dvrabel/revokable-grants-B.pdf Introduction Revision History Version Date Changes --- --- --

Re: [Xen-devel] [PATCH v5] x86/p2m: use large pages for MMIO mappings

2016-01-25 Thread Ian Campbell
On Mon, 2016-01-25 at 09:18 -0700, Jan Beulich wrote: > When mapping large BARs (e.g. the frame buffer of a graphics card) the > overhead of establishing such mappings using only 4k pages has, > particularly after the XSA-125 fix, become unacceptable. Alter the > XEN_DOMCTL_memory_mapping semantics

[Xen-devel] [PATCH 2/2] tools: avoid redefinining xenevtchn_handle typedef for xc_suspend_*

2016-01-25 Thread Ian Campbell
Similar to the previous xentoollog case this is not allowed. Switch to a forward decl of the struct and use of it in the APIs. Signed-off-by: Ian Campbell --- tools/libxc/include/xenguest.h | 15 ++- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/tools/libxc/include/x

[Xen-devel] [PATCH 1/2] tools: rename libxc's evtchn_port_or_error_t with an xc_ prefix

2016-01-25 Thread Ian Campbell
This is used only for xc_evtchn_alloc_unbound and the legacy/compat versions of the old interfaces and avoids redefining the typedef. The evtchn_port_or_error_t name is now used only be libxenevtchn. None of the callers of xc_evtchn_alloc_unbound use the type themselves. NB xc_evtchn_alloc_unboun

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

2016-01-25 Thread osstest service owner
flight 79021 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/79021/ 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 12

Re: [Xen-devel] [PATCH v3 15/17] ARM64: XEN: Add a function to initialize Xen specific UEFI runtime services

2016-01-25 Thread Stefano Stabellini
On Sat, 23 Jan 2016, Shannon Zhao wrote: > From: Shannon Zhao > > When running on Xen hypervisor, runtime services are supported through > hypercall. Add a Xen specific function to initialize runtime services. > > Signed-off-by: Shannon Zhao > --- > arch/arm64/include/asm/xen/xen-ops.h | 6 ++

Re: [Xen-devel] [PATCH v3 14/17] XEN: EFI: Move x86 specific codes to architecture directory

2016-01-25 Thread Stefano Stabellini
On Sat, 23 Jan 2016, Shannon Zhao wrote: > From: Shannon Zhao > > Move x86 specific codes to architecture directory and export those EFI > runtime service functions. This will be useful for initializing runtime > service on ARM later. > > Signed-off-by: Shannon Zhao This patch causes a build bre

Re: [Xen-devel] [v3,11/41] mips: reuse asm-generic/barrier.h

2016-01-25 Thread Will Deacon
On Fri, Jan 15, 2016 at 01:58:53PM -0800, Paul E. McKenney wrote: > On Fri, Jan 15, 2016 at 10:27:14PM +0100, Peter Zijlstra wrote: > > On Fri, Jan 15, 2016 at 09:46:12AM -0800, Paul E. McKenney wrote: > > > On Fri, Jan 15, 2016 at 10:13:48AM +0100, Peter Zijlstra wrote: > > > > > > And the stuff

Re: [Xen-devel] [PATCH] tools: avoid redefinition of typedefs

2016-01-25 Thread Ian Campbell
On Mon, 2016-01-25 at 17:24 +0100, Olaf Hering wrote: > On Mon, Jan 25, Ian Campbell wrote: > > > When splitting out various functionality from libxc into tools/libs/* > > I attempted to make it possible to avoid callers being unnecessarily > > exposed to the xentoollog interface by providing a ty

[Xen-devel] [PATCH v5] x86/p2m: use large pages for MMIO mappings

2016-01-25 Thread Jan Beulich
When mapping large BARs (e.g. the frame buffer of a graphics card) the overhead of establishing such mappings using only 4k pages has, particularly after the XSA-125 fix, become unacceptable. Alter the XEN_DOMCTL_memory_mapping semantics once again, so that there's no longer a fixed amount of guest

Re: [Xen-devel] [PATCH] tools: avoid redefinition of typedefs

2016-01-25 Thread Olaf Hering
On Mon, Jan 25, Ian Campbell wrote: > When splitting out various functionality from libxc into tools/libs/* > I attempted to make it possible to avoid callers being unnecessarily > exposed to the xentoollog interface by providing a typedef of the > xentoollog_logger handle in each of the headers.

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

2016-01-25 Thread osstest service owner
flight 78926 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/78926/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-freebsd10-amd64 3 host-install(3) broken REGR. vs. 78610 test-amd64-i386-xl-q

Re: [Xen-devel] [PATCH v3 2/3] XENVER_build_id: Provide ld-embedded build-ids (v8)

2016-01-25 Thread Jan Beulich
>>> On 25.01.16 at 16:16, wrote: >> >> > Or are you suggesting that perhaps the kernel should at boot time >> > print the build-id (like it does the changset)? >> >> Perhaps, albeit to me that's a bit orthogonal to being able to find out >> the build ID for a given binary. > > > I looked in t

Re: [Xen-devel] [PATCH v1 04/12] xen/hvmlite: Bootstrap HVMlite guest

2016-01-25 Thread Boris Ostrovsky
On 01/22/2016 06:32 PM, Luis R. Rodriguez wrote: On Fri, Jan 22, 2016 at 04:35:50PM -0500, Boris Ostrovsky wrote: +/* + * This routine (and those that it might call) should not use + * anything that lives in .bss since that segment will be cleared later + */ +void __init xen_prepare_hvmlite(voi

Re: [Xen-devel] [PATCH v3 13/17] ARM: Xen: Document UEFI support on Xen ARM virtual platforms

2016-01-25 Thread Stefano Stabellini
On Mon, 25 Jan 2016, Mark Rutland wrote: > On Mon, Jan 25, 2016 at 03:50:52PM +, Stefano Stabellini wrote: > > On Sat, 23 Jan 2016, Shannon Zhao wrote: > > > From: Shannon Zhao > > > > > > Add a "uefi" node under /hypervisor node in FDT, then Linux kernel could > > > scan this to get the UEFI

Re: [Xen-devel] [PATCH V6 5/5] x86/hvm: pkeys, add pkeys support for cpuid handling

2016-01-25 Thread Jan Beulich
>>> On 19.01.16 at 08:30, wrote: > --- a/xen/arch/x86/hvm/hvm.c > +++ b/xen/arch/x86/hvm/hvm.c > @@ -4566,7 +4566,7 @@ void hvm_cpuid(unsigned int input, unsigned int *eax, > unsigned int *ebx, > __clear_bit(X86_FEATURE_APIC & 31, edx); > > /* Fix up OSXSAVE. */ > -

Re: [Xen-devel] [PATCH v3 13/17] ARM: Xen: Document UEFI support on Xen ARM virtual platforms

2016-01-25 Thread Mark Rutland
On Mon, Jan 25, 2016 at 03:50:52PM +, Stefano Stabellini wrote: > On Sat, 23 Jan 2016, Shannon Zhao wrote: > > From: Shannon Zhao > > > > Add a "uefi" node under /hypervisor node in FDT, then Linux kernel could > > scan this to get the UEFI information. > > > > Signed-off-by: Shannon Zhao >

Re: [Xen-devel] [PATCH V6 4/5] xen/mm: Clean up pfec handling in gva_to_gfn

2016-01-25 Thread Jan Beulich
>>> On 19.01.16 at 08:30, wrote: > At the moment, the pfec argument to gva_to_gfn has two functions: > > * To inform guest_walk what kind of access is happenind > > * As a value to pass back into the guest in the event of a fault. > > Unfortunately this is not quite treated consistently: the hv

Re: [Xen-devel] [PATCH v3 13/17] ARM: Xen: Document UEFI support on Xen ARM virtual platforms

2016-01-25 Thread Stefano Stabellini
On Sat, 23 Jan 2016, Shannon Zhao wrote: > From: Shannon Zhao > > Add a "uefi" node under /hypervisor node in FDT, then Linux kernel could > scan this to get the UEFI information. > > Signed-off-by: Shannon Zhao > Acked-by: Rob Herring > --- > CC: Rob Herring > --- > Documentation/devicetree

Re: [Xen-devel] [PATCH V6 3/5] x86/hvm: pkeys, add xstate support for pkeys

2016-01-25 Thread Jan Beulich
>>> On 19.01.16 at 08:30, wrote: > --- a/xen/arch/x86/xstate.c > +++ b/xen/arch/x86/xstate.c > @@ -579,6 +579,10 @@ int handle_xsetbv(u32 index, u64 new_bv) > if ( (new_bv & ~xfeature_mask) || !valid_xcr0(new_bv) ) > return -EINVAL; > > +/* XCR0.PKRU is disabled on PV mode. */

Re: [Xen-devel] [PATCH V6 2/5] x86/hvm: pkeys, add pkeys support for guest_walk_tables

2016-01-25 Thread Jan Beulich
>>> On 19.01.16 at 08:30, wrote: > Signed-off-by: Huaitong Han > --- Please get used to put here per-patch info on what changed from the previous revision. > --- a/xen/arch/x86/mm/guest_walk.c > +++ b/xen/arch/x86/mm/guest_walk.c > @@ -90,6 +90,53 @@ static uint32_t set_ad_bits(void *guest_p, v

Re: [Xen-devel] [PATCH] tools: avoid redefinition of typedefs

2016-01-25 Thread Ian Jackson
Ian Campbell writes ("[PATCH] tools: avoid redefinition of typedefs"): > When splitting out various functionality from libxc into tools/libs/* > I attempted to make it possible to avoid callers being unnecessarily > exposed to the xentoollog interface by providing a typedef of the > xentoollog_logg

Re: [Xen-devel] [PATCH v1 02/12] xen/hvmlite: Factor out common kernel init code

2016-01-25 Thread Boris Ostrovsky
On 01/25/2016 06:04 AM, David Vrabel wrote: On 22/01/16 21:35, Boris Ostrovsky wrote: HVMlite guests (to be introduced in subsequent patches) share most of the kernel initialization code with PV(H). Where possible, HVMlite should share initialization with bare metal/HVM and not PV(H). This is

Re: [Xen-devel] [PATCH v3 12/17] ARM64: ACPI: Check if it runs on Xen to enable or disable ACPI

2016-01-25 Thread Stefano Stabellini
On Sat, 23 Jan 2016, Shannon Zhao wrote: > From: Shannon Zhao > > When it's a Xen domain0 booting with ACPI, it will supply a /chosen and > a /hypervisor node in DT. So check if it needs to enable ACPI. > > Signed-off-by: Shannon Zhao Reviewed-by: Stefano Stabellini > CC: Hanjun Guo > ---

Re: [Xen-devel] [PATCH v1 01/12] x86/smp: Make start_secondary() and initial_pg_pmd visible globally

2016-01-25 Thread Boris Ostrovsky
On 01/25/2016 05:53 AM, David Vrabel wrote: On 22/01/16 21:35, Boris Ostrovsky wrote: Xen's HVMlite guests will want to use them. Please explain in detail why they are needed. Actually, start_secondary is not needed anymore, I removed its use at some point. initial_pg_pmd (together with

[Xen-devel] [PATCH] tools: avoid redefinition of typedefs

2016-01-25 Thread Ian Campbell
When splitting out various functionality from libxc into tools/libs/* I attempted to make it possible to avoid callers being unnecessarily exposed to the xentoollog interface by providing a typedef of the xentoollog_logger handle in each of the headers. However such typedefs are not allowed in C,

  1   2   >