Re: [Xen-devel] [PATCH v10 6/7] vmx: VT-d posted-interrupt core logic handling

2016-01-19 Thread Wu, Feng
> -Original Message- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: Monday, January 18, 2016 11:14 PM > To: Wu, Feng > Cc: Andrew Cooper ; Dario Faggioli > ; George Dunlap ; > Tian, Kevin ; xen-devel@lists.xen.org; Keir Fraser > > Subject: Re: [PATCH v10 6/7] vmx: VT-d posted-in

Re: [Xen-devel] [PATCH v2 08/16] xen/hvm/params: Add a new delivery type for event-channel in HVM_PARAM_CALLBACK_IRQ

2016-01-19 Thread Shannon Zhao
On 2016/1/18 20:52, Andrew Cooper wrote: > On 18/01/16 12:46, Stefano Stabellini wrote: >> > On Mon, 18 Jan 2016, Andrew Cooper wrote: >>> >> On 18/01/16 12:38, Stefano Stabellini wrote: >>> On Fri, 15 Jan 2016, Shannon Zhao wrote: > From: Shannon Zhao > > Add a n

Re: [Xen-devel] [PATCH 4/4] hvmloader: add support to load extra ACPI tables from qemu

2016-01-19 Thread Zhang, Haozhong
On 01/20/16 13:14, Tian, Kevin wrote: > > From: Jan Beulich [mailto:jbeul...@suse.com] > > Sent: Tuesday, January 19, 2016 7:47 PM > > > > >>> On 19.01.16 at 12:37, wrote: > > > On Mon, Jan 18, 2016 at 01:46:29AM -0700, Jan Beulich wrote: > > >> >>> On 18.01.16 at 01:52, wrote: > > >> > On 01/15

[Xen-devel] [distros-debian-snapshot test] 38664: trouble: broken/pass

2016-01-19 Thread Platform Team regression test user
flight 38664 distros-debian-snapshot real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/38664/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-amd64-daily-netboot-pvgrub 3 host-install(3) br

Re: [Xen-devel] [PATCH 4/4] hvmloader: add support to load extra ACPI tables from qemu

2016-01-19 Thread Haozhong Zhang
Hi Jan, Wei and Kevin, On 01/18/16 01:46, Jan Beulich wrote: > >>> On 18.01.16 at 01:52, wrote: > > On 01/15/16 10:10, Jan Beulich wrote: > >> >>> On 29.12.15 at 12:31, wrote: > >> > NVDIMM devices are detected and configured by software through > >> > ACPI. Currently, QEMU maintains ACPI tables

Re: [Xen-devel] [PATCH 4/4] hvmloader: add support to load extra ACPI tables from qemu

2016-01-19 Thread Tian, Kevin
> From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: Tuesday, January 19, 2016 7:47 PM > > >>> On 19.01.16 at 12:37, wrote: > > On Mon, Jan 18, 2016 at 01:46:29AM -0700, Jan Beulich wrote: > >> >>> On 18.01.16 at 01:52, wrote: > >> > On 01/15/16 10:10, Jan Beulich wrote: > >> >> >>> On 29.12.1

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

2016-01-19 Thread Yu, Zhang
On 1/20/2016 11:58 AM, Tian, Kevin wrote: From: Yu, Zhang [mailto:yu.c.zh...@linux.intel.com] Sent: Wednesday, January 20, 2016 11:33 AM As a feature this write-protection has nothing to be GPU virtualization specific. In the future the same mediated pass-through idea used in XenGT may be use

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

2016-01-19 Thread Chun Yan Liu
>>> 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 pvusb API"): > > Add pvusb APIs, including: > > - attach/detach (create/destroy) virtual usb controller. > > - attach/detach u

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

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

Re: [Xen-devel] [PATCH LIBVIRT] libxl: Support cmdline= in xl config files

2016-01-19 Thread Jim Fehlig
On 01/19/2016 05:03 AM, Ian Campbell wrote: > I went to ping this but noticed that I had sent it to "jimfehlig" (i.e. no > domain), so no wonder there was no reply! > > To: line fixed here, let me know if you would prefer a resend. That would be much appreciated, thanks! > > Ian. > > On Wed, 2015

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

2016-01-19 Thread osstest service owner
flight 78537 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/78537/ 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] [PATCH 3/3] tools: introduce parameter max_ranges.

2016-01-19 Thread Tian, Kevin
> From: Yu, Zhang [mailto:yu.c.zh...@linux.intel.com] > Sent: Wednesday, January 20, 2016 11:33 AM > > As a feature this write-protection has nothing to be GPU virtualization > > specific. > > In the future the same mediated pass-through idea used in XenGT may be > > used on other I/O devices whic

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

2016-01-19 Thread Yu, Zhang
On 1/20/2016 11:14 AM, Tian, Kevin wrote: From: Ian Campbell [mailto:ian.campb...@citrix.com] Sent: Tuesday, January 19, 2016 11:19 PM On Tue, 2016-01-19 at 15:04 +, Wei Liu wrote: This patch doesn't seem to have been CCd to the tools maintainers, adding Ian too, I think everyone else was

Re: [Xen-devel] [PATCH 0/4] add support for vNVDIMM

2016-01-19 Thread Tian, Kevin
> From: Zhang, Haozhong > Sent: Tuesday, December 29, 2015 7:32 PM > > This patch series is the Xen part patch to provide virtual NVDIMM to > guest. The corresponding QEMU patch series is sent separately with the > title "[PATCH 0/2] add vNVDIMM support for Xen". > > * Background > > NVDIMM (No

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

2016-01-19 Thread Tian, Kevin
> From: Ian Campbell [mailto:ian.campb...@citrix.com] > Sent: Tuesday, January 19, 2016 11:19 PM > > On Tue, 2016-01-19 at 15:04 +, Wei Liu wrote: > > This patch doesn't seem to have been CCd to the tools maintainers, adding > Ian too, I think everyone else was picked up along the way. > > P

Re: [Xen-devel] [PATCH V13 5/5] xl: add pvusb commands

2016-01-19 Thread Jim Fehlig
On 01/19/2016 11:11 AM, Ian Jackson wrote: > Chunyan Liu writes ("[PATCH V13 5/5] xl: add pvusb commands"): >> Add pvusb commands: usbctrl-attach, usbctrl-detach, usb-list, >> usbdev-attach and usbdev-detach. > Thanks for swapping this with the other patch. It is better now. > >> +=item B I I > Ho

Re: [Xen-devel] [PATCH v5 1/6] remus: don't do failover if we don't have an consistent state

2016-01-19 Thread Wen Congyang
On 01/19/2016 06:48 PM, Ian Campbell wrote: > On Tue, 2016-01-19 at 15:17 +0800, Wen Congyang wrote: >> We will have an consistent state when a CHECKPOINT_END record > > "a consistent ..." (and in the subject too). > >> is received. After the first CHECKPOINT_END record is received, >> we will bu

Re: [Xen-devel] [PATCH v5 3/6] remus: resume immediately if libxl__xc_domain_save_done() completes

2016-01-19 Thread Wen Congyang
On 01/19/2016 06:55 PM, Ian Campbell wrote: > On Tue, 2016-01-19 at 15:17 +0800, 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

Re: [Xen-devel] Questions about the use of idle_vcpu[]

2016-01-19 Thread Tianyang Chen
On 1/18/2016 11:07 AM, Meng Xu wrote: On Mon, Jan 18, 2016 at 6:00 AM, Dario Faggioli wrote: On Mon, 2016-01-18 at 10:47 +, George Dunlap wrote: On Fri, Jan 15, 2016 at 1:04 AM, Tianyang Chen If an idle vcpu is picked, the ret.time is set accordingly in both credit and credit2 by che

Re: [Xen-devel] [PATCH v6 1/2] memory-hotplug: add automatic onlining policy for the newly added memory

2016-01-19 Thread David Rientjes
On Fri, 15 Jan 2016, Vitaly Kuznetsov wrote: > Currently, all newly added memory blocks remain in 'offline' state unless > someone onlines them, some linux distributions carry special udev rules > like: > > SUBSYSTEM=="memory", ACTION=="add", ATTR{state}=="offline", > ATTR{state}="online" > > t

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

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

[Xen-devel] [xen-unstable-smoke test] 78548: regressions - FAIL

2016-01-19 Thread osstest service owner
flight 78548 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/78548/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-debianhvm-i386 12 guest-saverestore fail REGR. vs. 78522 Tests whi

Re: [Xen-devel] Error booting Xen

2016-01-19 Thread Harmandeep Kaur
On Tue, Jan 19, 2016 at 10:38 PM, Dario Faggioli wrote: > On Tue, 2016-01-19 at 17:02 +, Andrew Cooper wrote: >> On 19/01/16 16:47, Jan Beulich wrote: >> > > > > On 19.01.16 at 17:27, wrote: >> > > Adding 'xsave=0' is working for now. Thank you >> > > all for your help :) >> > But that means

Re: [Xen-devel] [PATCH] build: specify minimum versions of make and binutils

2016-01-19 Thread Doug Goldstein
On 1/19/16 2:48 AM, Jan Beulich wrote: On 18.01.16 at 18:21, wrote: >> On 1/18/16 11:03 AM, Jan Beulich wrote: >> On 18.01.16 at 17:53, wrote: To help people avoid having to figure out what versions of make and binutils need to be supported document them explicitly. The version

Re: [Xen-devel] [PATCH 2/2] libxl: fix UUID usage on FreeBSD

2016-01-19 Thread Ian Jackson
Ian Campbell writes ("Re: [Xen-devel] [PATCH 2/2] libxl: fix UUID usage on FreeBSD"): > On Fri, 2016-01-15 at 16:11 +0100, Roger Pau Monné wrote: > > Yes, we can remove the uuid_t from libxl_uuid, but this is AFAICT a > > structure that belongs to the stable API. > > Oh b*m, so it is. I think we

Re: [Xen-devel] [xen-unstable-smoke test] 78539: regressions - FAIL [and 3 more messages]

2016-01-19 Thread Ian Jackson
Roger Pau Monne writes ("[Xen-devel] [PATCH] x86/HVM: memset CPU context save area"): > In order to prevent leaking data in the padding field. Also remove the > memset done to the fpu_regs in case of no FPU context present, since it's > already taken care by the memset of the whole CPU context str

Re: [Xen-devel] [PATCH OSSTEST] sg-report-job-history: alternate color of osstest column only when it changes

2016-01-19 Thread Ian Jackson
Ian Campbell writes ("Re: [PATCH OSSTEST] sg-report-job-history: alternate color of osstest column only when it changes"): > On Wed, 2016-01-06 at 11:08 +, Ian Campbell wrote: > > ping? Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@l

Re: [Xen-devel] [PATCH] x86/HVM: memset CPU context save area

2016-01-19 Thread Andrew Cooper
On 19/01/16 17:42, Roger Pau Monne wrote: > In order to prevent leaking data in the padding field. Also remove the > memset done to the fpu_regs in case of no FPU context present, since it's > already taken care by the memset of the whole CPU context structure. The > same applies to setting ctxt.fl

Re: [Xen-devel] [PATCH V13 5/5] xl: add pvusb commands

2016-01-19 Thread Ian Jackson
Chunyan Liu writes ("[PATCH V13 5/5] xl: add pvusb commands"): > Add pvusb commands: usbctrl-attach, usbctrl-detach, usb-list, > usbdev-attach and usbdev-detach. Thanks for swapping this with the other patch. It is better now. > +=item B I I However, I think you need to explictly state that the

Re: [Xen-devel] [PATCH V13 4/5] domcreate: support pvusb in configuration file

2016-01-19 Thread Ian Jackson
Chunyan Liu writes ("[PATCH V13 4/5] domcreate: support pvusb in configuration file"): > Add code to support pvusb in domain config file. One could specify > usbctrl and usb in domain's configuration file and create domain, > then usb controllers will be created and usb device would be attached >

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

2016-01-19 Thread osstest service owner
flight 78488 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/78488/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-amd64-libvirt-vhd 9 debian-di-installfail like 78311 Tests which did not succeed, but a

Re: [Xen-devel] [PATCH] tools/toollog: Drop XTL_NEW_LOGGER()

2016-01-19 Thread Ian Jackson
Andrew Cooper writes ("Re: [PATCH] tools/toollog: Drop XTL_NEW_LOGGER()"): > On 19/01/16 17:36, Ian Jackson wrote: > > I think this macro is useful because if you wanted to write (say) > > xtl_logger_syslog, you would want to use it to help you with some > > boilerplate. > > WTF? Even documented,

Re: [Xen-devel] [xen-unstable-smoke test] 78539: regressions - FAIL

2016-01-19 Thread Roger Pau Monné
El 19/01/16 a les 18.43, osstest service owner ha escrit: > flight 78539 xen-unstable-smoke real [real] > http://logs.test-lab.xenproject.org/osstest/logs/78539/ > > Regressions :-( > > Tests which did not succeed and are blocking, > including tests which could not be run: > test-amd64-amd64-xl-

Re: [Xen-devel] [PATCH] tools/toollog: Drop XTL_NEW_LOGGER()

2016-01-19 Thread Andrew Cooper
On 19/01/16 17:36, Ian Jackson wrote: > Ian Jackson writes ("Re: [PATCH] tools/toollog: Drop XTL_NEW_LOGGER()"): >> Ian Campbell writes ("Re: [PATCH] tools/toollog: Drop XTL_NEW_LOGGER()"): >>> The underlying issue with all of these is the _undocumented_ nature of the >>> assumptions, which is cert

Re: [Xen-devel] [PATCH] x86/HVM: memset CPU context save area

2016-01-19 Thread Roger Pau Monné
El 19/01/16 a les 18.42, Roger Pau Monne ha escrit: > In order to prevent leaking data in the padding field. Also remove the > memset done to the fpu_regs in case of no FPU context present, since it's > already taken care by the memset of the whole CPU context structure. The > same applies to setti

[Xen-devel] [xen-unstable-smoke test] 78539: regressions - FAIL

2016-01-19 Thread osstest service owner
flight 78539 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/78539/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-debianhvm-i386 12 guest-saverestore fail REGR. vs. 78522 Tests whi

[Xen-devel] [PATCH] x86/HVM: memset CPU context save area

2016-01-19 Thread Roger Pau Monne
In order to prevent leaking data in the padding field. Also remove the memset done to the fpu_regs in case of no FPU context present, since it's already taken care by the memset of the whole CPU context structure. The same applies to setting ctxt.flags to 0 in case there's no FPU context. Signed-o

Re: [Xen-devel] [PATCH] tools/toollog: Drop XTL_NEW_LOGGER()

2016-01-19 Thread Ian Jackson
Ian Jackson writes ("Re: [PATCH] tools/toollog: Drop XTL_NEW_LOGGER()"): > Ian Campbell writes ("Re: [PATCH] tools/toollog: Drop XTL_NEW_LOGGER()"): > > The underlying issue with all of these is the _undocumented_ nature of the > > assumptions, which is certainly a bug, however those assumptions ar

Re: [Xen-devel] [PATCH] tools/toollog: Drop XTL_NEW_LOGGER()

2016-01-19 Thread Ian Jackson
Ian Jackson writes ("Re: [PATCH] tools/toollog: Drop XTL_NEW_LOGGER()"): > Ian Campbell writes ("Re: [PATCH] tools/toollog: Drop XTL_NEW_LOGGER()"): > > > If vtable isn't the first element in the structure, it follows a wild > > > pointer on error. > > This could be fixed. Actually, no it couldn'

Re: [Xen-devel] [PATCH] tools/toollog: Drop XTL_NEW_LOGGER()

2016-01-19 Thread Ian Jackson
Ian Campbell writes ("Re: [PATCH] tools/toollog: Drop XTL_NEW_LOGGER()"): > The underlying issue with all of these is the _undocumented_ nature of the > assumptions, which is certainly a bug, however those assumptions are not in > themselves "unreasonable" as was claimed. Maybe I should submit a c

Re: [Xen-devel] Error booting Xen

2016-01-19 Thread Andrew Cooper
On 19/01/16 17:08, Dario Faggioli wrote: > On Tue, 2016-01-19 at 17:02 +, Andrew Cooper wrote: >> On 19/01/16 16:47, Jan Beulich wrote: >> On 19.01.16 at 17:27, wrote: Adding 'xsave=0' is working for now. Thank you all for your help :) >>> But that means we actually should get to

Re: [Xen-devel] Error booting Xen

2016-01-19 Thread Dario Faggioli
On Tue, 2016-01-19 at 17:02 +, Andrew Cooper wrote: > On 19/01/16 16:47, Jan Beulich wrote: > > > > > On 19.01.16 at 17:27, wrote: > > > Adding 'xsave=0' is working for now. Thank you > > > all for your help :) > > But that means we actually should get to the bottom of your > > problem! > > T

Re: [Xen-devel] [PATCH] tools/toollog: Drop XTL_NEW_LOGGER()

2016-01-19 Thread Ian Campbell
On Tue, 2016-01-19 at 16:40 +, Andrew Cooper wrote: > On 19/01/16 16:24, Ian Campbell wrote: > > On Thu, 2016-01-14 at 20:13 +, Andrew Cooper wrote: > > > XTL_NEW_LOGGER() makes a number of unreasonable assumptions about the > > > symbols > > > visible in its scope, > > It assumes that the

Re: [Xen-devel] Error booting Xen

2016-01-19 Thread Andrew Cooper
On 19/01/16 16:47, Jan Beulich wrote: On 19.01.16 at 17:27, wrote: >> Adding 'xsave=0' is working for now. Thank you >> all for your help :) > But that means we actually should get to the bottom of your problem! There was some discussion on IRC. `xrstror` was repeatedly taking the same faul

Re: [Xen-devel] [PATCH v2 08/13] xsplice: Implement payload loading (v2)

2016-01-19 Thread Konrad Rzeszutek Wilk
> >+static void find_hole(ssize_t pages, unsigned long *hole_start, > >+ unsigned long *hole_end) > >+{ > >+struct payload *data, *data2; > >+ > >+spin_lock(&payload_list_lock); > >+list_for_each_entry ( data, &payload_list, list ) > >+{ > >+list_for_eac

Re: [Xen-devel] [PATCH v2 09/13] xsplice: Implement support for applying/reverting/replacing patches. (v2)

2016-01-19 Thread Konrad Rzeszutek Wilk
On Tue, Jan 19, 2016 at 02:39:40PM +, Ross Lagerwall wrote: > On 01/14/2016 09:47 PM, Konrad Rzeszutek Wilk wrote: > >From: Ross Lagerwall > > > >Implement support for the apply, revert and replace actions. > > > snip > >+#include > > #include > > #include > > #include > >@@ -10,25 +11,

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

2016-01-19 Thread Meng Xu
On Tue, Jan 19, 2016 at 11:35 AM, Tianyang Chen wrote: > > > > On 1/15/2016 10:33 AM, Meng Xu wrote: >> >> On Wed, Jan 6, 2016 at 2:57 AM, Tianyang Chen wrote: >>> >>> >>> >>> >>> On 12/31/2015 8:44 AM, Meng Xu wrote: On Thu, Dec 31, 2015 at 6:20 PM, Tianyang Chen wrote: >>>

Re: [Xen-devel] [PATCH] libxc: fix comment in xc__hypercall_bounce_pre

2016-01-19 Thread Ian Campbell
On Tue, 2016-01-19 at 12:03 +, Wei Liu wrote: > Signed-off-by: Wei Liu Acked + applied. ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCHv2] xenpm: Add option to report average CPU frequency

2016-01-19 Thread Ian Campbell
On Tue, 2016-01-19 at 11:27 +, Malcolm Crossley wrote: > The average is calculated over the period of time from the last > xenpm report of the average CPU frequency. > > Reporting the average CPU frequency helps confirm the level of turbo > boost being achieved per CPU. > > Signed-off-by: Mal

Re: [Xen-devel] [PATCH] docs: correct descriptions of gnttab_max_{, maptrack}_frames

2016-01-19 Thread Ian Campbell
On Tue, 2016-01-19 at 06:06 -0700, Jan Beulich wrote: > > > > On 19.01.16 at 12:56, wrote: > > -Specify the maximum number of maptrack frames domain. > > -The default value is 8 times **gnttab\_max\_frames**. > > +> Default: `8 * gnttab_max_frames` > > No backslash escaping needed here? > > In a

Re: [Xen-devel] [PATCH v4.2] libxc: Defer initialization of start_page for HVM guests

2016-01-19 Thread Ian Campbell
On Tue, 2016-01-19 at 15:13 +, Ian Campbell wrote: > On Tue, 2016-01-19 at 10:02 -0500, Boris Ostrovsky wrote: > > On 01/08/2016 10:11 AM, Ian Campbell wrote: > > > > > > BTW, it might be possible to use xc_dom_seg_to_ptr instead of an open > > > coded > > > xc_map_foreign? Although it wasn't

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

2016-01-19 Thread Konrad Rzeszutek Wilk
On Tue, Jan 19, 2016 at 09:36:27AM -0700, Jan Beulich wrote: > >>> On 19.01.16 at 17:05, wrote: > > Are you OK if I re-institute the $(build_id) back in the Makefile, perhaps > > rename it to $(build_id_linker) to make it more clear? > > Well, perhaps that's the least troublesome route right now,

Re: [Xen-devel] Error booting Xen

2016-01-19 Thread Jan Beulich
>>> On 19.01.16 at 17:27, wrote: > Adding 'xsave=0' is working for now. Thank you > all for your help :) But that means we actually should get to the bottom of your problem! Jan ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/

Re: [Xen-devel] [PATCH] libxl: add basic spice support for pv domUs

2016-01-19 Thread Ian Campbell
On Wed, 2016-01-13 at 11:47 +0100, Fabio Fantoni wrote: > > >   =head2 Keymaps > > >   > > >   The keymaps available are defined by the device-model which you are > > > diff --git a/tools/libxl/libxl.h b/tools/libxl/libxl.h > > > index 6b73848..a5cbcfc 100644 > > > --- a/tools/libxl/libxl.h > > >

Re: [Xen-devel] [PATCH v2 10/13] xen_hello_world.xsplice: Test payload for patching 'xen_extra_version'.

2016-01-19 Thread Ross Lagerwall
On 01/14/2016 09:47 PM, Konrad Rzeszutek Wilk wrote: snip diff --git a/tools/misc/xen_hello_world.c b/tools/misc/xen_hello_world.c new file mode 100644 index 000..8c24d8f --- /dev/null +++ b/tools/misc/xen_hello_world.c @@ -0,0 +1,15 @@ +#include "xsplice.h" + +/* Our replacement function for

Re: [Xen-devel] [PATCH v2 08/13] xsplice: Implement payload loading (v2)

2016-01-19 Thread Ross Lagerwall
On 01/14/2016 09:47 PM, Konrad Rzeszutek Wilk wrote: snip +static int move_payload(struct payload *payload, struct xsplice_elf *elf) +{ +uint8_t *buf; +unsigned int i; +size_t core_size = 0; + +/* Compute text regions */ +for ( i = 0; i < elf->hdr->e_shnum; i++ ) +{ +

Re: [Xen-devel] [PATCH] tools/toollog: Drop XTL_NEW_LOGGER()

2016-01-19 Thread Andrew Cooper
On 19/01/16 16:24, Ian Campbell wrote: > On Thu, 2016-01-14 at 20:13 +, Andrew Cooper wrote: >> XTL_NEW_LOGGER() makes a number of unreasonable assumptions about the symbols >> visible in its scope, > It assumes that the function names to fill in the vtable and the type name > are related, that

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

2016-01-19 Thread Jan Beulich
>>> On 19.01.16 at 17:05, wrote: > Are you OK if I re-institute the $(build_id) back in the Makefile, perhaps > rename it to $(build_id_linker) to make it more clear? Well, perhaps that's the least troublesome route right now, even if I don't really like it. Jan ___

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

2016-01-19 Thread Tianyang Chen
On 1/15/2016 10:33 AM, Meng Xu wrote: On Wed, Jan 6, 2016 at 2:57 AM, Tianyang Chen wrote: On 12/31/2015 8:44 AM, Meng Xu wrote: On Thu, Dec 31, 2015 at 6:20 PM, Tianyang Chen wrote: Budget replenishment is now handled by a dedicated timer which is triggered at the most imminent rele

Re: [Xen-devel] Error booting Xen

2016-01-19 Thread Harmandeep Kaur
Hi, Adding 'xsave=0' is working for now. Thank you all for your help :) Regards, Harman On Tue, Jan 19, 2016 at 8:37 PM, Andrew Cooper wrote: > On 19/01/16 14:51, Dario Faggioli wrote: >> On Tue, 2016-01-19 at 06:36 -0700, Jan Beulich wrote: >> On 18.01.16 at 19:29, wrote: Yeah,

Re: [Xen-devel] [PATCH] tools/toollog: Drop XTL_NEW_LOGGER()

2016-01-19 Thread Ian Campbell
On Thu, 2016-01-14 at 20:13 +, Andrew Cooper wrote: > XTL_NEW_LOGGER() makes a number of unreasonable assumptions about the symbols > visible in its scope, It assumes that the function names to fill in the vtable and the type name are related, that hardly seems totally "unreasonable". What els

[Xen-devel] [qemu-upstream-unstable test] 78442: tolerable FAIL - PUSHED

2016-01-19 Thread osstest service owner
flight 78442 qemu-upstream-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/78442/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-xl-pvh-amd 11 guest-start fail never pass test-amd64-amd64-xl-pvh-inte

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

2016-01-19 Thread Konrad Rzeszutek Wilk
>>> Wouldn't it be better to probe the linker for recognizing the --build-id >>> command line option, along the lines of $(cc-option)? >> >> +ld-ver-build-id = $(shell $(1) --build-id 2>&1 | \ >> + grep -q unrecognized && echo n >> || echo y) >> >> -ish ? > > I

Re: [Xen-devel] [PATCH XEN v8 24/29] tools/libs/call: linux: touch newly allocated pages after madvise lockdown

2016-01-19 Thread Ian Campbell
On Tue, 2016-01-19 at 15:49 +, Wei Liu wrote: >  > I see. I wonder why we didn't use mlock(2) in Linux too in the first > place. We did, but mlock(2) on Linux doesn't guarantee that there will be no page faults, it only guarantees that there will be no page faults which require I/O to satisfy

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

2016-01-19 Thread Ian Jackson
Chunyan Liu writes ("[PATCH V13 3/5] libxl: add pvusb API"): > Add pvusb APIs, including: > - attach/detach (create/destroy) virtual usb controller. > - attach/detach usb device > - list usb controller and usb devices > - some other helper functions Thanks. This is making progress but I'm af

Re: [Xen-devel] [PATCH XEN v8 24/29] tools/libs/call: linux: touch newly allocated pages after madvise lockdown

2016-01-19 Thread Wei Liu
On Tue, Jan 19, 2016 at 03:03:31PM +, Ian Campbell wrote: > On Tue, 2016-01-19 at 14:58 +, Wei Liu wrote: > > On Tue, Jan 19, 2016 at 03:54:54PM +0100, Roger Pau Monné wrote: > > > El 19/01/16 a les 14.24, Wei Liu ha escrit: > > > > On Fri, Jan 15, 2016 at 01:23:03PM +, Ian Campbell wro

Re: [Xen-devel] [Minios-devel] [PATCH v8 0/] Begin to disentangle libxenctrl and provide some stable libraries

2016-01-19 Thread Ian Campbell
On Fri, 2016-01-15 at 13:22 +, Ian Campbell wrote: >  > Therefore needing attention from Ian and/or Wei are: > > tools/libs/foreignmemory: Mention restrictions on fork in docs. > N tools/libs/evtchn: Use uint32_t for domid arguments > D tools/libs/gnttab: Extensive updates to API

Re: [Xen-devel] [PATCH v5 1/6] remus: don't do failover if we don't have an consistent state

2016-01-19 Thread Andrew Cooper
On 19/01/16 10:48, Ian Campbell wrote: > On Tue, 2016-01-19 at 15:17 +0800, Wen Congyang wrote: >> We will have an consistent state when a CHECKPOINT_END record > "a consistent ..." (and in the subject too). > >> is received. After the first CHECKPOINT_END record is received, >> we will buffer all

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

2016-01-19 Thread osstest service owner
flight 78496 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/78496/ 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] [PATCH v3] x86/HVM: add padding to struct hvm_hw_cpu

2016-01-19 Thread Roger Pau Monné
El 19/01/16 a les 16.13, Jan Beulich ha escrit: On 18.01.16 at 17:25, wrote: >> --- a/xen/arch/x86/hvm/hvm.c >> +++ b/xen/arch/x86/hvm/hvm.c >> @@ -1990,6 +1990,9 @@ static int hvm_load_cpu_ctxt(struct domain *d, >> hvm_domain_context_t *h) >> if ( hvm_load_entry_zeroextend(CPU, h, &ctx

Re: [Xen-devel] [PATCH v3] x86/HVM: add padding to struct hvm_hw_cpu

2016-01-19 Thread Jan Beulich
>>> On 18.01.16 at 17:25, wrote: > --- a/xen/arch/x86/hvm/hvm.c > +++ b/xen/arch/x86/hvm/hvm.c > @@ -1990,6 +1990,9 @@ static int hvm_load_cpu_ctxt(struct domain *d, > hvm_domain_context_t *h) > if ( hvm_load_entry_zeroextend(CPU, h, &ctxt) != 0 ) > return -EINVAL; > > +if ( c

Re: [Xen-devel] [PATCH v4.2] libxc: Defer initialization of start_page for HVM guests

2016-01-19 Thread Ian Campbell
On Tue, 2016-01-19 at 10:02 -0500, Boris Ostrovsky wrote: > On 01/08/2016 10:11 AM, Ian Campbell wrote: > > > > BTW, it might be possible to use xc_dom_seg_to_ptr instead of an open > > coded > > xc_map_foreign? Although it wasn't used before and maybe there is a > > reason > > for that which stil

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

2016-01-19 Thread Ian Campbell
On Tue, 2016-01-19 at 15:04 +, Wei Liu wrote: This patch doesn't seem to have been CCd to the tools maintainers, adding Ian too, I think everyone else was picked up along the way. Please use ./scripts/get_maintainers.pl in the future. > On Tue, Jan 19, 2016 at 02:47:40PM +, Paul Durrant

Re: [Xen-devel] [PATCH v3 05/11] igd: TYPE_IGD_PASSTHROUGH_I440FX_PCI_DEVICE: call parent realize

2016-01-19 Thread Eduardo Habkost
On Wed, Jan 06, 2016 at 04:45:01PM +0100, Gerd Hoffmann wrote: > > > > > > +static void (*i440fx_realize)(PCIDevice *pci_dev, Error **errp); > > > static void igd_pt_i440fx_realize(PCIDevice *pci_dev, Error **errp) > > > { > > > +Error *err = NULL; > > > uint32_t val = 0; > > > in

Re: [Xen-devel] [PATCH v2 03/16] Xen: xlate: Use page_to_xen_pfn instead of page_to_pfn

2016-01-19 Thread Shannon Zhao
On 2016/1/19 22:59, Stefano Stabellini wrote: On Mon, 18 Jan 2016, Shannon Zhao wrote: On 2016/1/16 1:08, Stefano Stabellini wrote: On Fri, 15 Jan 2016, Shannon Zhao wrote: From: Shannon Zhao Use page_to_xen_pfn in case of 64KB page. Signed-off-by: Shannon Zhao --- drivers/xen/xlate_mm

Re: [Xen-devel] [PATCH v3 02/11] pc: remove has_igd_gfx_passthru global

2016-01-19 Thread Eduardo Habkost
On Tue, Jan 05, 2016 at 12:41:29PM +0100, Gerd Hoffmann wrote: > Signed-off-by: Gerd Hoffmann Reviewed-by: Eduardo Habkost -- Eduardo ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH XEN v8 24/29] tools/libs/call: linux: touch newly allocated pages after madvise lockdown

2016-01-19 Thread Ian Campbell
On Tue, 2016-01-19 at 14:58 +, Wei Liu wrote: > On Tue, Jan 19, 2016 at 03:54:54PM +0100, Roger Pau Monné wrote: > > El 19/01/16 a les 14.24, Wei Liu ha escrit: > > > On Fri, Jan 15, 2016 at 01:23:03PM +, Ian Campbell wrote: > > > > This avoids a potential issue with a fork after allocation

Re: [Xen-devel] Error booting Xen

2016-01-19 Thread Andrew Cooper
On 19/01/16 14:51, Dario Faggioli wrote: > On Tue, 2016-01-19 at 06:36 -0700, Jan Beulich wrote: > On 18.01.16 at 19:29, wrote: >>> >>> Yeah, but again, it was booting as dom0 with another Xen... doesn't >>> that mean that Xen at least has a role in exposing it? >>> >>> Also, what would you

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

2016-01-19 Thread Wei Liu
On Tue, Jan 19, 2016 at 02:47:40PM +, Paul Durrant wrote: [...] > > ranges so perhaps the parameter name could be > > 'max_wp_memory_ranges'? > > > > > > > What does "WP" mean? "Write Protected"? > > > > Yes. > > > Is this parameter closely related to IOREQ server? Should it contain > > "io

Re: [Xen-devel] [PATCH v4.2] libxc: Defer initialization of start_page for HVM guests

2016-01-19 Thread Boris Ostrovsky
On 01/08/2016 10:11 AM, Ian Campbell wrote: BTW, it might be possible to use xc_dom_seg_to_ptr instead of an open coded xc_map_foreign? Although it wasn't used before and maybe there is a reason for that which still applies. Is this waiting for me to replace xc_map_foreign() with xc_dom_seg_

Re: [Xen-devel] [PATCH v2 03/16] Xen: xlate: Use page_to_xen_pfn instead of page_to_pfn

2016-01-19 Thread Stefano Stabellini
On Mon, 18 Jan 2016, Shannon Zhao wrote: > On 2016/1/16 1:08, Stefano Stabellini wrote: > > On Fri, 15 Jan 2016, Shannon Zhao wrote: > >> From: Shannon Zhao > >> > >> Use page_to_xen_pfn in case of 64KB page. > >> > >> Signed-off-by: Shannon Zhao > >> --- > >> drivers/xen/xlate_mmu.c | 2 +- > >>

Re: [Xen-devel] [PATCH OSSTEST v2] cs-adjust-flight: Add jobs-rename command which applies a perlop to job names

2016-01-19 Thread Ian Campbell
On Tue, 2016-01-19 at 14:44 +, Ian Jackson wrote: > Ian Campbell writes ("Re: [PATCH OSSTEST v2] cs-adjust-flight: Add jobs- > rename command which applies a perlop to job names"): > > On Tue, 2016-01-19 at 14:25 +, Ian Jackson wrote: > > > And if that happens, there might be entries in the

Re: [Xen-devel] [PATCH XEN v8 24/29] tools/libs/call: linux: touch newly allocated pages after madvise lockdown

2016-01-19 Thread Wei Liu
On Tue, Jan 19, 2016 at 03:54:54PM +0100, Roger Pau Monné wrote: > El 19/01/16 a les 14.24, Wei Liu ha escrit: > > On Fri, Jan 15, 2016 at 01:23:03PM +, Ian Campbell wrote: > >> This avoids a potential issue with a fork after allocation but before > >> madvise. > >> > >> Signed-off-by: Ian Camp

Re: [Xen-devel] [PATCH v2 10/13] xen_hello_world.xsplice: Test payload for patching 'xen_extra_version'.

2016-01-19 Thread Ross Lagerwall
On 01/14/2016 09:47 PM, Konrad Rzeszutek Wilk wrote: This change demonstrates how to generate an xSplice ELF payload. The idea here is that we want to patch in the hypervisor the 'xen_version_extra' function with an function that will return 'Hello World'. The 'xl info | grep extraversion' will

Re: [Xen-devel] [PATCH XEN v8 24/29] tools/libs/call: linux: touch newly allocated pages after madvise lockdown

2016-01-19 Thread Roger Pau Monné
El 19/01/16 a les 14.24, Wei Liu ha escrit: > On Fri, Jan 15, 2016 at 01:23:03PM +, Ian Campbell wrote: >> This avoids a potential issue with a fork after allocation but before >> madvise. >> >> Signed-off-by: Ian Campbell >> --- >> v7: New, replacing "tools/libs/call: linux: avoid forking bet

Re: [Xen-devel] Error booting Xen

2016-01-19 Thread Dario Faggioli
On Tue, 2016-01-19 at 06:36 -0700, Jan Beulich wrote: > > > > On 18.01.16 at 19:29, wrote: > >  > > Yeah, but again, it was booting as dom0 with another Xen... doesn't > > that mean that Xen at least has a role in exposing it? > > > > Also, what would you think it's better to try next... try anot

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

2016-01-19 Thread Paul Durrant
> -Original Message- > From: Wei Liu [mailto:wei.l...@citrix.com] > Sent: 19 January 2016 14:37 > To: Paul Durrant > Cc: Wei Liu; Yu Zhang; xen-devel@lists.xen.org; Kevin Tian; Keir (Xen.org); > Stefano Stabellini; Andrew Cooper; zhiyuan...@intel.com; > jbeul...@suse.com > Subject: Re: [Xen

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

2016-01-19 Thread osstest service owner
flight 78522 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/78522/ 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 v2 11/13] xsplice: Add support for bug frames. (v2)

2016-01-19 Thread Ross Lagerwall
On 01/14/2016 09:47 PM, Konrad Rzeszutek Wilk wrote: From: Ross Lagerwall Add support for handling bug frames contained with xsplice modules. If a trap occurs search either the kernel bug table or an applied payload's bug table depending on the instruction pointer. Signed-off-by: Ross Lagerwal

Re: [Xen-devel] [PATCH v2 09/13] xsplice: Implement support for applying/reverting/replacing patches. (v2)

2016-01-19 Thread Ross Lagerwall
On 01/14/2016 09:47 PM, Konrad Rzeszutek Wilk wrote: From: Ross Lagerwall Implement support for the apply, revert and replace actions. snip +#include #include #include #include @@ -10,25 +11,38 @@ #include #include #include +#include #include +#include #include #

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

2016-01-19 Thread Wei Liu
On Tue, Jan 19, 2016 at 01:54:42PM +, Paul Durrant wrote: > > -Original Message- > > From: Wei Liu [mailto:wei.l...@citrix.com] > > Sent: 19 January 2016 11:54 > > To: Yu Zhang > > Cc: xen-devel@lists.xen.org; Kevin Tian; Keir (Xen.org); Stefano Stabellini; > > Andrew Cooper; Paul Durra

Re: [Xen-devel] [PATCH OSSTEST v2] cs-adjust-flight: Add jobs-rename command which applies a perlop to job names

2016-01-19 Thread Ian Campbell
On Tue, 2016-01-19 at 14:25 +, Ian Jackson wrote: > Ian Campbell writes ("[PATCH OSSTEST v2] cs-adjust-flight: Add jobs- > rename command which applies a perlop to job names"): > > My intention was to allow creation of adhoc jobs based on a template > > but modified e.g. to enable/disable XSM w

Re: [Xen-devel] [PATCH] xen/gntdev: Don't allocate struct gntdev_copy_batch on stack

2016-01-19 Thread Boris Ostrovsky
On 01/18/2016 06:11 AM, David Vrabel wrote: On 15/01/16 19:43, Boris Ostrovsky wrote: struct gntdev_copy_batch is over 1300 bytes in size, we shouldn't put it on stack. Some compilers (e.g. 5.2.1) complain: drivers/xen/gntdev.c: In function ‘gntdev_ioctl_grant_copy.isra.5’: drivers/xen/gntd

Re: [Xen-devel] [PATCH v2 5/5] libxl: Add explicit cast to libxl_psr_cat_set_cbm

2016-01-19 Thread Ian Campbell
On Tue, 2016-01-19 at 14:06 +, Ian Jackson wrote: > Chester Lin writes ("[PATCH v2 5/5] libxl: Add explicit cast to > libxl_psr_cat_set_cbm"): > > Fixes Coverity CID 1343299. The call to xc_psr_cat_set_domain_data() > > expects type xc_psr_cat_type but is provided libxl_psr_cbm_type which > > i

Re: [Xen-devel] [PATCH] xen/gntdev: Don't allocate struct gntdev_copy_batch on stack

2016-01-19 Thread David Vrabel
On 19/01/16 14:26, Boris Ostrovsky wrote: > On 01/18/2016 06:11 AM, David Vrabel wrote: >> On 15/01/16 19:43, Boris Ostrovsky wrote: >>> struct gntdev_copy_batch is over 1300 bytes in size, we shouldn't >>> put it on stack. >>> >>> Some compilers (e.g. 5.2.1) complain: >>> drivers/xen/gntdev.c: I

Re: [Xen-devel] [PATCH v2 08/13] xsplice: Implement payload loading (v2)

2016-01-19 Thread Ross Lagerwall
On 01/14/2016 09:47 PM, Konrad Rzeszutek Wilk wrote: From: Ross Lagerwall Add support for loading xsplice payloads. This is somewhat similar to the Linux kernel module loader, implementing the following steps: - Verify the elf file. - Parse the elf file. - Allocate a region of memory mapped wit

Re: [Xen-devel] [PATCH v2 5/5] libxl: Add explicit cast to libxl_psr_cat_set_cbm

2016-01-19 Thread Ian Jackson
Ian Campbell writes ("Re: [PATCH v2 5/5] libxl: Add explicit cast to libxl_psr_cat_set_cbm"): > On Tue, 2016-01-19 at 14:06 +, Ian Jackson wrote: > >  * XEN_DOMCTL_PSR_CAT_OP_SET_L3_* (public/domctl.h) > >  * enum xc_psr_cat_type (xenctrl.h) > >  * Enumeration("psr_cbm_type",...) (libxl_types.

Re: [Xen-devel] [PATCH v2 5/5] libxl: Add explicit cast to libxl_psr_cat_set_cbm

2016-01-19 Thread Ian Jackson
Dario Faggioli writes ("Re: [PATCH v2 5/5] libxl: Add explicit cast to libxl_psr_cat_set_cbm"): > On Tue, 2016-01-19 at 14:21 +, Ian Campbell wrote: > > libxl_shutdown_reason has the same issues, libxl_tsc_mode also might, > > as might libxl_timer_mode. I'm not sure if there are others. > > >

Re: [Xen-devel] [PATCH v2 07/13] xsplice: Add helper elf routines (v2)

2016-01-19 Thread Ross Lagerwall
On 01/14/2016 09:47 PM, Konrad Rzeszutek Wilk wrote: From: Ross Lagerwall Add Elf routines and data structures in preparation for loading an xSplice payload. We also add an macro that will print where we failed during the ELF parsing. Signed-off-by: Ross Lagerwall Signed-off-by: Konrad Rzesz

Re: [Xen-devel] [PATCH v2 01/13] xsplice: Design document (v5).

2016-01-19 Thread Ross Lagerwall
On 01/14/2016 09:46 PM, Konrad Rzeszutek Wilk wrote: +## Workflow + +The expected workflows of higher-level tools that manage multiple patches +on production machines would be: + + * The first obvious task is loading all available / suggested + hotpatches around system start. I'd expect that

  1   2   3   >