Re: [Xen-devel] The Bitdefender virtual machine introspection library is now on GitHub

2015-07-28 Thread Razvan Cojocaru
Hello Tamas, > I've pinged the other LibVMI maintainers as well to take a look. In > summary, what do you see as the benefit in libbdvmi over LibVMI? Well, libbdvmi addresses a slightly different architectural problem: it needs to provide a very efficient way of working with Xen to access guest i

Re: [Xen-devel] [PATCH v4 0/4] virtio: Clean up scatterlists and use the DMA API

2015-07-28 Thread Christian Borntraeger
Am 28.07.2015 um 03:08 schrieb Andy Lutomirski: > On Mon, Sep 1, 2014 at 10:39 AM, Andy Lutomirski wrote: >> This fixes virtio on Xen guests as well as on any other platform >> that uses virtio_pci on which physical addresses don't match bus >> addresses. >> >> This can be tested with: >> >> v

[Xen-devel] [PATCH] tools/hvmloader: sync memory map[]

2015-07-28 Thread Tiejun Chen
Currently we always use memory map[] to help hvmloader construct e820 table but hvmloader may have relocated RAM to support mmio allocation or just populated ram to ensure we can have enough room to load ovmf. Anyway we need to sync these changes into memory map[]. CC: Keir Fraser CC: Jan Beulich

Re: [Xen-devel] [v4 11/17] vt-d: Add API to update IRTE when VT-d PI is used

2015-07-28 Thread Wu, Feng
> -Original Message- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: Friday, July 24, 2015 11:28 PM > To: Wu, Feng > Cc: Andrew Cooper; Tian, Kevin; Zhang, Yang Z; xen-devel@lists.xen.org; Keir > Fraser > Subject: Re: [v4 11/17] vt-d: Add API to update IRTE when VT-d PI is used >

Re: [Xen-devel] Regression in OVMF + RMRR series

2015-07-28 Thread Chen, Tiejun
Jul 25 17:48:51.685107 (d1) BIOS map: Jul 25 17:48:51.685145 (d1) ffe0-: Main BIOS Jul 25 17:48:51.693030 (d1) *** HVMLoader bug at e820.c:262 Jul 25 17:48:51.693064 (d1) *** HVMLoader crashed. Git blame shows that the change that crashes hvmloader was part of the RMRR series. Tieju

[Xen-devel] [linux-next test] 60025: tolerable FAIL

2015-07-28 Thread osstest service owner
flight 60025 linux-next real [real] http://logs.test-lab.xenproject.org/osstest/logs/60025/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-i386-xl-xsm 13 guest-saverestore fail baseline untested test-amd64-amd64-rumpuserxen-am

Re: [Xen-devel] [PATCH v4 0/4] virtio: Clean up scatterlists and use the DMA API

2015-07-28 Thread Paolo Bonzini
On 28/07/2015 03:08, Andy Lutomirski wrote: > On Mon, Sep 1, 2014 at 10:39 AM, Andy Lutomirski wrote: >> This fixes virtio on Xen guests as well as on any other platform >> that uses virtio_pci on which physical addresses don't match bus >> addresses. >> >> This can be tested with: >> >> vir

Re: [Xen-devel] Xen on ARM: Problems on fast model

2015-07-28 Thread Ian Campbell
On Tue, 2015-07-28 at 14:28 +0800, harry wrote: > Hi, > > I want to simulate xen on ARM with fast model, but I failed to download > git://xenbits.xen.org/people/ianc/boot-wrapper.git which is lost. > > Who has this file can send me one or who can tell me where I can download > it? Cloning tha

Re: [Xen-devel] [PATCH for-4.6 v2 1/8] libxl: properly clean up array in libxl_list_cpupool failure path

2015-07-28 Thread Dario Faggioli
On Mon, 2015-07-27 at 18:45 +0100, Wei Liu wrote: > Document how cpupool_info works. Distinguish success (ERROR_FAIL + > ENOENT) vs failure in libxl_list_cpupool and properly clean up the array > in failure path. > > Also switch to libxl__realloc and call libxl_cpupool_{init,dispose} > where appr

Re: [Xen-devel] Xen on ARM: Problems on fast model

2015-07-28 Thread harry
I've got the boot-wrapper already. And I found that in git://linux-arm.org/arm-dts.git, the fast model tree has been deleted.I want to know how I can get the corresponding file. Thanks, Harry At 2015-07-28 16:25:57, "Ian Campbell" wrote: >On Tue, 2015-07-28 at 14:28 +0800, harry wrote: >> Hi

Re: [Xen-devel] [PATCH v4 for Xen 4.6 3/4] libxl: enable per-VCPU parameter settings for RTDS scheduler

2015-07-28 Thread Dario Faggioli
On Fri, 2015-07-10 at 23:52 -0500, Chong Li wrote: > Add libxl_vcpu_sched_params_get/set and sched_rtds_vcpu_get/set > functions to support per-VCPU settings. > This patch looks mostly fine. A few comments. So, as for the other patches, I'd mention here in the changelog, that only RTDS supports t

Re: [Xen-devel] Help with reading from hvc console device on a PV guest

2015-07-28 Thread Wei Liu
Please don't top-post. On Fri, Jul 24, 2015 at 10:05:20AM -0700, sainath grandhi wrote: > Hi Wei, >Thanks for your reply. Sorry I did not change the comment properly. > These are the two methods I tried separately. > > First method: > I see /dev/hvc1 and /dev/hvc2 being created when I gi

[Xen-devel] [PATCH v3] xen/events: Support event channel rebind on ARM

2015-07-28 Thread Julien Grall
Currently, the event channel rebind code is gated with the presence of the vector callback. The virtual interrupt controller on ARM has the concept of per-CPU interrupt (PPI) which allow us to support per-VCPU event channel. Therefore there is no need of vector callback for ARM. Xen is already us

Re: [Xen-devel] [PATCH v4 for Xen 4.6 4/4] xl: enable per-VCPU parameter settings for RTDS scheduler

2015-07-28 Thread Dario Faggioli
On Fri, 2015-07-10 at 23:52 -0500, Chong Li wrote: > Change main_sched_rtds and related output functions to support > per-VCPU settings. > This patch also looks nice. A few comments provided inline. > diff --git a/tools/libxl/xl_cmdimpl.c b/tools/libxl/xl_cmdimpl.c > index c858068..da7c8a6 10064

Re: [Xen-devel] Xen on ARM: Problems on fast model

2015-07-28 Thread Ian Campbell
Please do not top post and do not post in HTML (use plain text instead). On Tue, 2015-07-28 at 16:57 +0800, harry wrote: > I've got the boot-wrapper already. > And I found that in git://linux-arm.org/arm-dts.git, the fast model tree > has been deleted. I believe the DTS for the FVP trees (which

Re: [Xen-devel] PV-vNUMA issue: topology is misinterpreted by the guest

2015-07-28 Thread Andrew Cooper
On 28/07/15 04:52, Juergen Gross wrote: > On 07/28/2015 01:19 AM, Andrew Cooper wrote: >> On 27/07/2015 18:42, Dario Faggioli wrote: >>> On Mon, 2015-07-27 at 17:33 +0100, Andrew Cooper wrote: On 27/07/15 17:31, David Vrabel wrote: > >> Yeah, indeed. >> That's the downside of Juerg

Re: [Xen-devel] [PATCH for-4.6 v2 5/8] xl: call libxl_dominfo_{init, dispose} in main_cpupoolnumasplit

2015-07-28 Thread Dario Faggioli
On Mon, 2015-07-27 at 18:45 +0100, Wei Liu wrote: > Signed-off-by: Wei Liu > Reviewed-by: Dario Faggioli Regards, Dario -- <> (Raistlin Majere) - Dario Faggioli, Ph.D, http://about.me/dario.faggioli Senior Software Engineer, Citrix

Re: [Xen-devel] [PATCH v3] xen/events: Support event channel rebind on ARM

2015-07-28 Thread David Vrabel
On 28/07/15 10:10, Julien Grall wrote: > Currently, the event channel rebind code is gated with the presence of > the vector callback. > > The virtual interrupt controller on ARM has the concept of per-CPU > interrupt (PPI) which allow us to support per-VCPU event channel. > Therefore there is no

Re: [Xen-devel] PV-vNUMA issue: topology is misinterpreted by the guest

2015-07-28 Thread Wei Liu
On Fri, Jul 24, 2015 at 06:05:59PM +0200, Dario Faggioli wrote: > On Fri, 2015-07-24 at 17:14 +0200, Juergen Gross wrote: > > On 07/24/2015 04:44 PM, Dario Faggioli wrote: > > > > Ok. And I already have a question (as I lost track of things a bit). > > > What you just said about ACPI tables is cer

Re: [Xen-devel] [PATCH 2/6] tools/libx{l, c}: Remove the toolstack_{save, restore} callbacks

2015-07-28 Thread Ian Campbell
On Wed, 2015-07-22 at 12:14 +0100, Ian Campbell wrote: > > > +> development. It is should not be used.* > > > Does it needs to be accepted on restore for N->N+1 purposes (or N+M > > > for > > > M>1 since you care about that)? Or does the conversion routine turn > > > it > > > into something el

Re: [Xen-devel] [qemu-mainline test] 59908: regressions - FAIL

2015-07-28 Thread Anthony PERARD
On Sat, Jul 25, 2015 at 05:34:34PM +0100, Ian Campbell wrote: > On Sat, 2015-07-25 at 16:15 +, osstest service owner wrote: > > flight 59908 qemu-mainline real [real] > > http://secure-web.cisco.com/1u4kwl1o3Dc6FDtVhFHrYLmyppwmEL8O6TSGGkDPdNzjeTSjZl_GhmvpBKlKsBPNbIlSfLqlqdopz-VgXgkCmm0eXivzV7hD

Re: [Xen-devel] [PATCH 1/4] libxl: CODING_STYLE: Much new material

2015-07-28 Thread Ian Campbell
On Wed, 2014-11-05 at 14:43 +, Ian Jackson wrote: Sorry to be a bit late to the game, do I win a prize? ;-) [...] > + Usually, don't use: Instead, use (see libxl_internal.h): > + libxl__log[v] LOG, LOGE, LOGEV > + libxl__sprintf GCSPRINTF > + libxl__*alloc et al.

Re: [Xen-devel] [PATCH v2 for-4.6 1/3] tools/libxl: Do not set stream->rc in stream_complete()

2015-07-28 Thread Ian Jackson
Andrew Cooper writes ("[PATCH v2 for-4.6 1/3] tools/libxl: Do not set stream->rc in stream_complete()"): > Only ever set stream->rc in check_all_finished(). The first version of > the migration v2 series had separate rc and joined_rc parameters, where > this logic worked. However when combining

Re: [Xen-devel] [PATCH v2 for-4.6 2/3] tools/libxl: Do not fire the stream callback multiple times

2015-07-28 Thread Ian Jackson
Andrew Cooper writes ("[PATCH v2 for-4.6 2/3] tools/libxl: Do not fire the stream callback multiple times"): > Avoid stacking of check_all_finished() via synchronous teardown of > tasks. If the _abort() functions call back synchronously, > stream->completion_callback() ends up getting called twic

Re: [Xen-devel] [PATCH for-4.6 v2 2/8] xl: lockdir should be lockfile in error message

2015-07-28 Thread Ian Campbell
On Mon, 2015-07-27 at 18:45 +0100, Wei Liu wrote: > Signed-off-by: Wei Liu Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH for-4.6 v2 1/8] libxl: properly clean up array in libxl_list_cpupool failure path

2015-07-28 Thread Ian Campbell
On Mon, 2015-07-27 at 18:45 +0100, Wei Liu wrote: > Document how cpupool_info works. Distinguish success (ERROR_FAIL + > ENOENT) vs failure in libxl_list_cpupool and properly clean up the array > in failure path. > > Also switch to libxl__realloc and call libxl_cpupool_{init,dispose} > where appr

Re: [Xen-devel] [PATCH for-4.6 v2 4/8] xl: valid fd can be 0 in main_loadpolicy

2015-07-28 Thread Ian Campbell
On Mon, 2015-07-27 at 18:45 +0100, Wei Liu wrote: > Initialise polFd to -1 before hand to avoid closing 0 by accident. > > Also fixed some style problems while I was there. > > Signed-off-by: Wei Liu Acked-by: Ian Campbell ___ Xen-devel mailing lis

Re: [Xen-devel] [PATCH for-4.6 v2 3/8] xl: call libxl_dominfo_init in main_list

2015-07-28 Thread Ian Campbell
On Mon, 2015-07-27 at 18:45 +0100, Wei Liu wrote: > Always call init and dispose function on info_buf though it's not > always used. > > Signed-off-by: Wei Liu Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen

Re: [Xen-devel] [PATCH v4 0/3] x86: modify_ldt improvement, test, and config option

2015-07-28 Thread Andrew Cooper
On 28/07/15 04:16, Andy Lutomirski wrote: > On Mon, Jul 27, 2015 at 7:20 PM, Andy Lutomirski wrote: >> On Mon, Jul 27, 2015 at 9:18 AM, Boris Ostrovsky >> wrote: >>> On 07/27/2015 11:53 AM, Andy Lutomirski wrote: On Mon, Jul 27, 2015 at 8:36 AM, Boris Ostrovsky wrote: > On 07/25/20

Re: [Xen-devel] [PATCH for-4.6 v2 5/8] xl: call libxl_dominfo_{init, dispose} in main_cpupoolnumasplit

2015-07-28 Thread Ian Campbell
On Mon, 2015-07-27 at 18:45 +0100, Wei Liu wrote: > Signed-off-by: Wei Liu Acked-by: Ian Campbell > --- > tools/libxl/xl_cmdimpl.c | 9 + > 1 file changed, 9 insertions(+) > > diff --git a/tools/libxl/xl_cmdimpl.c b/tools/libxl/xl_cmdimpl.c > index 77cf603..70ed675 100644 > --- a/tool

Re: [Xen-devel] [PATCH for-4.6 v2 6/8] libxlu: free buffer in failure path for PCI related functions

2015-07-28 Thread Ian Campbell
On Mon, 2015-07-27 at 18:45 +0100, Wei Liu wrote: > Signed-off-by: Wei Liu > --- > tools/libxl/libxlu_pci.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/tools/libxl/libxlu_pci.c b/tools/libxl/libxlu_pci.c > index 026413b..16ddaf7 100644 > --- a/tools/libxl/libxlu_pci.c > +++ b/tool

Re: [Xen-devel] [PATCH for-4.6 v2 7/8] python/xc: reinstate original implementation of next_bdf

2015-07-28 Thread Ian Campbell
On Mon, 2015-07-27 at 18:45 +0100, Wei Liu wrote: > I missed the fact that next_bdf is used to parsed user supplied > strings when reviewing. The user supplied string is a NULL-terminated > string separated by comma. User can supply several PCI devices in that > string. There is, however, no delimi

Re: [Xen-devel] [PATCH for-4.6 v2] libxl: check nesthvm and altp2m in libxl

2015-07-28 Thread Ian Campbell
On Mon, 2015-07-27 at 15:01 +0100, Wei Liu wrote: > In ea214001 ("x86/altp2m: add altp2mhvm HVM domain parameter"), a > check was added to ensure nestedhvm and altp2m cannot be enabled at > the same time. That check was added in xl, but in fact it should be in > libxl because it should be the entit

Re: [Xen-devel] [PATCH for 4.6 0/6] Prune legacy migration and move migration v2 out of daft status

2015-07-28 Thread Ian Campbell
On Wed, 2015-07-22 at 13:20 +0100, Wei Liu wrote: > > Then this series needs to wait until the window for applying patches > with freeze exception is closed. Those series have priority in this > week. I applied, having checked with you on IRC. ___ Xen

Re: [Xen-devel] [PATCH for-4.6 v2 7/8] python/xc: reinstate original implementation of next_bdf

2015-07-28 Thread Wei Liu
On Tue, Jul 28, 2015 at 11:38:29AM +0100, Ian Campbell wrote: > On Mon, 2015-07-27 at 18:45 +0100, Wei Liu wrote: > > I missed the fact that next_bdf is used to parsed user supplied > > strings when reviewing. The user supplied string is a NULL-terminated > > string separated by comma. User can sup

Re: [Xen-devel] [PATCH] tools/hvmloader: sync memory map[]

2015-07-28 Thread Andrew Cooper
On 28/07/15 08:27, Tiejun Chen wrote: > Currently we always use memory map[] to help hvmloader construct e820 table > but hvmloader may have relocated RAM to support mmio allocation or just > populated ram to ensure we can have enough room to load ovmf. Anyway we > need to sync these changes into m

Re: [Xen-devel] [PATCH for-4.6 v2 8/8] libxl: remove dead code libxl__domain_shutdown_reason

2015-07-28 Thread Ian Campbell
On Mon, 2015-07-27 at 18:45 +0100, Wei Liu wrote: > There is no user in tree. git log -S libxl__domain_shutdown_reason git log -S get_shutdown_reason (the later being a rename that the former picks up on) suggests it has _never_ been used! > Signed-off-by: Wei Liu Acked-by: Ian Campbell > --

Re: [Xen-devel] [PATCH for-4.6 v2 0/8] tools: fixes inspired by Coverity scan

2015-07-28 Thread Ian Campbell
On Mon, 2015-07-27 at 18:45 +0100, Wei Liu wrote: > Wei Liu (8): Applied 7 of these, but not: > libxlu: free buffer in failure path for PCI related functions which I have commented on. Ian. ___ Xen-devel mailing list Xen-devel@lists.xen.org http://

Re: [Xen-devel] [PATCH] blkif.h: document physical-device node

2015-07-28 Thread Ian Campbell
On Mon, 2015-07-20 at 13:04 +0200, Roger Pau Monné wrote: > > Acked-by: Roger Pau Monné Applied. Since I don't think it conflicts with... > Although I plan to expand this field so it's also used by FreeBSD > blkback with a different format. ...this. Ian. _

Re: [Xen-devel] [PATCH] xenbackendd: remove xenbackendd

2015-07-28 Thread Ian Campbell
On Thu, 2015-07-23 at 16:13 +0200, Roger Pau Monne wrote: > The functionality provided by xenbackendd has been integrated into the xl > toolstack under the devd command on all platforms, because of that it no > longer makes sense to maintain xenbackendd. > > Init scripts have also been provided fo

Re: [Xen-devel] [PATCH v3 04/32] libxc: introduce a domain loader for HVM guest firmware

2015-07-28 Thread Wei Liu
On Fri, Jul 03, 2015 at 01:34:42PM +0200, Roger Pau Monne wrote: > Introduce a very simple (and dummy) domain loader to be used to load the > firmware (hvmloader) into HVM guests. Since hmvloader is just a 32bit elf > executable the loader is fairly simple. > > Signed-off-by: Roger Pau Monné > Cc

Re: [Xen-devel] [PATCH v3 03/32] libxc: introduce the notion of a container type

2015-07-28 Thread Wei Liu
On Fri, Jul 03, 2015 at 01:34:41PM +0200, Roger Pau Monne wrote: > Introduce the notion of a container type into xc_dom_image. This will be > needed by later changes that will also use xc_dom_image in order to build > HVM guests. > > Signed-off-by: Roger Pau Monné > Cc: Ian Jackson > Cc: Stefano

Re: [Xen-devel] [PATCH v3 02/32] libxc: unify xc_dom_p2m_{host/guest}

2015-07-28 Thread Wei Liu
On Fri, Jul 03, 2015 at 01:34:40PM +0200, Roger Pau Monne wrote: > Unify both functions into xc_dom_p2m. Should not introduce any functional > change. > > Signed-off-by: Roger Pau Monné > Cc: Ian Jackson > Cc: Stefano Stabellini > Cc: Ian Campbell > Cc: Wei Liu > Cc: Samuel Thibault Acked-b

Re: [Xen-devel] [PATCH v3 05/32] libxc: make arch_setup_meminit a xc_dom_arch hook

2015-07-28 Thread Wei Liu
On Fri, Jul 03, 2015 at 01:34:43PM +0200, Roger Pau Monne wrote: > This allows having different arch_setup_meminit implementations based on the > guest type. It should not introduce any functional changes. > > Signed-off-by: Roger Pau Monné > Cc: Ian Jackson > Cc: Stefano Stabellini > Cc: Ian C

Re: [Xen-devel] [PATCH v3 06/32] libxc: make arch_setup_boot{init/late} xc_dom_arch hooks

2015-07-28 Thread Wei Liu
On Fri, Jul 03, 2015 at 01:34:44PM +0200, Roger Pau Monne wrote: > This should not introduce any functional change. > > Signed-off-by: Roger Pau Monné > Cc: Ian Jackson > Cc: Stefano Stabellini > Cc: Ian Campbell > Cc: Wei Liu Acked-by: Wei Liu > --- > +/*

Re: [Xen-devel] [PATCH v3 09/32] libxl: switch HVM domain building to use xc_dom_* helpers

2015-07-28 Thread Wei Liu
On Fri, Jul 03, 2015 at 01:34:47PM +0200, Roger Pau Monne wrote: > Now that we have all the code in place HVM domain building in libxl can be > switched to use the xc_dom_* family of functions, just like they are used in > order to build PV guests. > > Signed-off-by: Roger Pau Monné > Cc: Ian Jac

Re: [Xen-devel] [PATCH v3 23/32] libxc: allow creating domains without emulated devices.

2015-07-28 Thread Wei Liu
On Fri, Jul 03, 2015 at 01:35:01PM +0200, Roger Pau Monne wrote: > Introduce a new flag in xc_dom_image that turns on and off the emulated > devices. This prevents creating the VGA hole, the hvm_info page and the > ioreq server pages. libxl unconditionally sets it to true for all HVM > domains at t

Re: [Xen-devel] [PATCH v3 01/32] libxc: split x86 HVM setup_guest into smaller logical functions

2015-07-28 Thread Wei Liu
On Fri, Jul 03, 2015 at 01:34:39PM +0200, Roger Pau Monne wrote: > This is just a preparatory change to clean up the code in setup_guest. > Should not introduce any functional changes. > > Signed-off-by: Roger Pau Monné > Cc: Ian Jackson > Cc: Stefano Stabellini > Cc: Ian Campbell > Cc: Wei Li

Re: [Xen-devel] [PATCH v3 32/32] libxl: allow the creation of HVM domains without a device model.

2015-07-28 Thread Wei Liu
On Fri, Jul 03, 2015 at 01:35:10PM +0200, Roger Pau Monne wrote: > Replace the firmware loaded into HVM guests with an OS kernel. Since the HVM > builder now uses the PV xc_dom_* set of functions this kernel will be parsed > and loaded inside the guest like on PV, but the container is a pure HVM >

Re: [Xen-devel] [PATCH v4 0/4] virtio: Clean up scatterlists and use the DMA API

2015-07-28 Thread Benjamin Herrenschmidt
On Tue, 2015-07-28 at 10:16 +0200, Paolo Bonzini wrote: > > On 28/07/2015 03:08, Andy Lutomirski wrote: > > On Mon, Sep 1, 2014 at 10:39 AM, Andy Lutomirski > > wrote: > >> This fixes virtio on Xen guests as well as on any other platform > >> that uses virtio_pci on which physical addresses don'

[Xen-devel] [xen-unstable test] 60023: regressions - FAIL

2015-07-28 Thread osstest service owner
flight 60023 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/60023/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-ovmf-amd64 9 debian-hvm-install fail REGR. vs. 59817 test-amd64-i386-xl-q

Re: [Xen-devel] [PATCH v4 0/4] virtio: Clean up scatterlists and use the DMA API

2015-07-28 Thread Paolo Bonzini
On 28/07/2015 12:12, Benjamin Herrenschmidt wrote: >> > That is an experimental feature (it's x-iommu), so it can change. >> > >> > The plan was: >> > >> > - for PPC, virtio never honors IOMMU >> > >> > - for non-PPC, either have virtio always honor IOMMU, or enforce that >> > virtio is not un

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

2015-07-28 Thread Ian Campbell
On Tue, 2015-07-28 at 12:17 +, osstest service owner wrote: > > Tests which did not succeed and are blocking, > including tests which could not be run: > test-amd64-amd64-xl-qemuu-ovmf-amd64 9 debian-hvm-install fail REGR. vs. > 59817 > test-amd64-i386-xl-qemuu-ovmf-amd64 9 debian-hvm-inst

Re: [Xen-devel] [PATCH v4 0/4] virtio: Clean up scatterlists and use the DMA API

2015-07-28 Thread Michael S. Tsirkin
On Tue, Jul 28, 2015 at 02:46:20PM +0200, Paolo Bonzini wrote: > > > On 28/07/2015 12:12, Benjamin Herrenschmidt wrote: > >> > That is an experimental feature (it's x-iommu), so it can change. > >> > > >> > The plan was: > >> > > >> > - for PPC, virtio never honors IOMMU > >> > > >> > - for no

Re: [Xen-devel] [PATCH v4 0/4] virtio: Clean up scatterlists and use the DMA API

2015-07-28 Thread Michael S. Tsirkin
On Mon, Jul 27, 2015 at 06:08:59PM -0700, Andy Lutomirski wrote: > On Mon, Sep 1, 2014 at 10:39 AM, Andy Lutomirski wrote: > > This fixes virtio on Xen guests as well as on any other platform > > that uses virtio_pci on which physical addresses don't match bus > > addresses. > > > > This can be te

[Xen-devel] Dom0 crash with apache bench (ab)

2015-07-28 Thread Christoffer Dall
Hi, I've been doing some performance comparisons lately, and wanted to compare the performance overhead of using Xen with apache bench, but unfortunately the Dom0 kernel crashes when hitting it with ab from a remote machine. Most other workloads seem to be stable, however, I do see similar crashes

Re: [Xen-devel] [PATCH v4 0/4] virtio: Clean up scatterlists and use the DMA API

2015-07-28 Thread Jan Kiszka
On 2015-07-28 15:06, Michael S. Tsirkin wrote: > On Tue, Jul 28, 2015 at 02:46:20PM +0200, Paolo Bonzini wrote: >> >> >> On 28/07/2015 12:12, Benjamin Herrenschmidt wrote: > That is an experimental feature (it's x-iommu), so it can change. > > The plan was: > > - for PPC, virtio

Re: [Xen-devel] [PATCH v5 01/22] xen/arm: Return success if dt node does not have irq mapping

2015-07-28 Thread Julien Grall
Hi Vijay, I think the title is too wide for the patch. I would rename to "xen/dt: Handle correctly node with interrupt-map in dt_for_each_irq_map" On 27/07/15 12:11, vijay.kil...@gmail.com wrote: > From: Vijaya Kumar K > > dt_for_each_irq_map() returns error if no irq mapping is found. > With t

Re: [Xen-devel] [PATCH v5 04/22] xen/arm: Set nr_cpu_ids to available number of cpus

2015-07-28 Thread Julien Grall
Hi Vijay, On 27/07/15 12:11, vijay.kil...@gmail.com wrote: > From: Vijaya Kumar K > > nr_cpu_ids for arm platforms is set to 128 irrespective of NIT: s/128/NR_CPUS/ > number of cpus supported by platform. > > Signed-off-by: Vijaya Kumar K > --- > xen/arch/arm/setup.c |1 + > xen/ar

Re: [Xen-devel] [PATCH v5 01/22] xen/arm: Return success if dt node does not have irq mapping

2015-07-28 Thread Ian Campbell
On Tue, 2015-07-28 at 14:13 +0100, Julien Grall wrote: > Hi Vijay, > > I think the title is too wide for the patch. I would rename to > "xen/dt: Handle correctly node with interrupt-map in dt_for_each_irq_map" > > On 27/07/15 12:11, vijay.kil...@gmail.com wrote: > > From: Vijaya Kumar K > > > >

Re: [Xen-devel] [PATCH] tools/hvmloader: sync memory map[]

2015-07-28 Thread Ian Campbell
On Tue, 2015-07-28 at 11:49 +0100, Andrew Cooper wrote: > On 28/07/15 08:27, Tiejun Chen wrote: > > Currently we always use memory map[] to help hvmloader construct e820 > > table > > but hvmloader may have relocated RAM to support mmio allocation or just > > populated ram to ensure we can have en

Re: [Xen-devel] [PATCH v5 01/22] xen/arm: Return success if dt node does not have irq mapping

2015-07-28 Thread Julien Grall
On 28/07/15 14:23, Ian Campbell wrote: > On Tue, 2015-07-28 at 14:13 +0100, Julien Grall wrote: >> Hi Vijay, >> >> I think the title is too wide for the patch. I would rename to >> "xen/dt: Handle correctly node with interrupt-map in dt_for_each_irq_map" >> >> On 27/07/15 12:11, vijay.kil...@gmail.

Re: [Xen-devel] [PATCH v2 for-4.6 0/3] Fixes to stream v2 task joining logic

2015-07-28 Thread Ian Campbell
On Mon, 2015-07-27 at 17:47 +0100, Andrew Cooper wrote: > Split the original patch into 3 distinct fixes. > > Andrew Cooper (3): > tools/libxl: Do not set stream->rc in stream_complete() > tools/libxl: Do not fire the stream callback multiple times Applied with Ian's ack. > tools/libxl: On

[Xen-devel] [PATCH v10 01/11] xen: introduce SHUTDOWN_soft_reset shutdown reason

2015-07-28 Thread Vitaly Kuznetsov
This special type of shutdown is supposed to be used by PVHVM guests when they want to perform some sort of kexec/kdump. Signed-off-by: Vitaly Kuznetsov Acked-by: Jan Beulich Reviewed-by: Konrad Rzeszutek Wilk --- Changes since v9: - add Reviewed-by: [Konrad Rzeszutek Wilk] - minor comment chan

[Xen-devel] [PATCH v10 07/11] flask: DOMCTL_soft_reset support

2015-07-28 Thread Vitaly Kuznetsov
Add new soft_reset vector to domain2 class, add it to create_domain in the default policy. Signed-off-by: Vitaly Kuznetsov Acked-by: Daniel De Graaf --- Changes since v9: - None. --- tools/flask/policy/policy/modules/xen/xen.if | 2 +- xen/xsm/flask/hooks.c| 3 +++ xen/x

[Xen-devel] [PATCH v10 00/11] toolstack-assisted approach to PVHVM guest kexec

2015-07-28 Thread Vitaly Kuznetsov
This patch series provides x86 PVHVM domains with an ability to perform kexec/kdump-style operations. Changes since v9 (also mentioned in individual patches) grouped by reviewer's name: [Konrad Rzeszutek Wilk] - PATCH 01: add Reviewed-by: - PATCH 04: add Reviewed-by: - PATCH 05: add Reviewed-by: -

[Xen-devel] [PATCH v10 06/11] xen: Introduce XEN_DOMCTL_soft_reset

2015-07-28 Thread Vitaly Kuznetsov
New domctl resets state for a domain allowing it to 'start over': register vcpu_info, switch to FIFO ABI for event channels. Still active grants are being logged to help debugging misbehaving backends. Signed-off-by: Vitaly Kuznetsov Acked-by: Jan Beulich --- Changes since v9: - Unlock shutdown_

[Xen-devel] [PATCH v10 04/11] xen: evtchn: make evtchn_reset() ready for soft reset

2015-07-28 Thread Vitaly Kuznetsov
We need to close all event channel so the domain performing soft reset will be able to open them back. Signed-off-by: Vitaly Kuznetsov Reviewed-by: Konrad Rzeszutek Wilk Acked-by: Jan Beulich --- Changes since v9: - Add Reviewed-by: [Konrad Rzeszutek Wilk] - Add Acked-by: [Jan Beulich] --- xen

[Xen-devel] [PATCH v10 09/11] libxc: support XEN_DOMCTL_soft_reset operation

2015-07-28 Thread Vitaly Kuznetsov
Introduce xc_domain_soft_reset() function supporting XEN_DOMCTL_soft_reset. Signed-off-by: Vitaly Kuznetsov Acked-by: Wei Liu Reviewed-by: Konrad Rzeszutek Wilk Acked-by: Ian Jackson --- Changes since v9: - None. --- tools/libxc/include/xenctrl.h | 3 +++ tools/libxc/xc_domain.c | 9 +++

[Xen-devel] [PATCH v10 11/11] (lib)xl: soft reset support

2015-07-28 Thread Vitaly Kuznetsov
Use existing create/restore path to perform 'soft reset' for HVM domains. Tear everything down, e.g. destroy domain's device model, remove the domain from xenstore, save toolstack record and start over. Signed-off-by: Vitaly Kuznetsov --- Changes since v9: - s,reset,Reset, in xl.cfg.pod.5 [Konrad

[Xen-devel] [PATCH v10 03/11] xl: introduce enum domain_restart_type

2015-07-28 Thread Vitaly Kuznetsov
As a preparation before adding new restart type (soft reset) put all restart types into an enum. Signed-off-by: Vitaly Kuznetsov Acked-by: Ian Campbell Reviewed-by: Konrad Rzeszutek Wilk --- Changes since v9: - None. --- tools/libxl/xl.h | 6 ++ tools/libxl/xl_cmdimpl.c | 23 +

[Xen-devel] [PATCH v10 08/11] xen: arch-specific hooks for domain_soft_reset()

2015-07-28 Thread Vitaly Kuznetsov
x86-specific hook cleans up the pirq-emuirq mappings, destroys all ioreq servers and and replaces the shared_info frame with an empty page to support subsequent XENMAPSPACE_shared_info call. ARM-specific hook is -ENOSYS for now. Signed-off-by: Vitaly Kuznetsov --- Changes since v9: - is_hvm_doma

[Xen-devel] [PATCH v10 05/11] xen: grant_table: implement grant_table_warn_active_grants()

2015-07-28 Thread Vitaly Kuznetsov
Log first 10 active grants for a domain. This function is going to be used for soft reset, active grants on this path usually mean misbehaving backends refusing to release their mappings on shutdown. We need that in addition to the already existent 'g' keyhandler as such misbehaving backends can ca

[Xen-devel] [PATCH v10 10/11] libxc: add XC_DEVICE_MODEL_SAVE_FILE

2015-07-28 Thread Vitaly Kuznetsov
Use this in libxl_dm instead of hard-coding. Signed-off-by: Vitaly Kuznetsov Acked-by: Ian Campbell --- Changes since v9: - None --- tools/libxc/include/xenguest.h | 2 ++ tools/libxl/libxl_dm.c | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/tools/libxc/include/xe

[Xen-devel] [PATCH v10 02/11] libxl: support SHUTDOWN_soft_reset shutdown reason

2015-07-28 Thread Vitaly Kuznetsov
Use letter 'S' to indicate a domain in such state. Introduce new 'on_soft_reset' action and default it to 'restart' for now. Signed-off-by: Vitaly Kuznetsov --- Changes since v9: - None --- docs/man/xl.cfg.pod.5 | 5 + tools/libxl/libxl_types.idl | 2 ++ tools/libxl/xl_cm

Re: [Xen-devel] [PATCH v2 for-4.6 3/3] tools/libxl: Only continue stream operations if the stream is still in progress

2015-07-28 Thread Ian Jackson
Andrew Cooper writes ("[PATCH v2 for-4.6 3/3] tools/libxl: Only continue stream operations if the stream is still in progress"): > Part of the callback contract with check_all_finished() is that each > running parallel task shall call it exactly once. ... > @@ -738,14 +738,16 @@ void libxl__xc_dom

Re: [Xen-devel] [PATCH] xenbackendd: remove xenbackendd

2015-07-28 Thread Roger Pau Monné
El 28/07/15 a les 13.12, Ian Campbell ha escrit: > On Thu, 2015-07-23 at 16:13 +0200, Roger Pau Monne wrote: >> The functionality provided by xenbackendd has been integrated into the xl >> toolstack under the devd command on all platforms, because of that it no >> longer makes sense to maintain xen

Re: [Xen-devel] [PATCH] xenbackendd: remove xenbackendd

2015-07-28 Thread George Dunlap
On 07/28/2015 02:49 PM, Roger Pau Monné wrote: > El 28/07/15 a les 13.12, Ian Campbell ha escrit: >> On Thu, 2015-07-23 at 16:13 +0200, Roger Pau Monne wrote: >>> The functionality provided by xenbackendd has been integrated into the xl >>> toolstack under the devd command on all platforms, because

Re: [Xen-devel] [PATCH v2 for-4.6 3/3] tools/libxl: Only continue stream operations if the stream is still in progress

2015-07-28 Thread Andrew Cooper
On 28/07/15 14:41, Ian Jackson wrote: > Andrew Cooper writes ("[PATCH v2 for-4.6 3/3] tools/libxl: Only continue > stream operations if the stream is still in progress"): >> Part of the callback contract with check_all_finished() is that each >> running parallel task shall call it exactly once. >

Re: [Xen-devel] [PATCH v4 0/3] x86: modify_ldt improvement, test, and config option

2015-07-28 Thread Boris Ostrovsky
On 07/28/2015 06:29 AM, Andrew Cooper wrote: After forward-porting my virtio patches, I got this thing to run on Xen. After several tries, I got: [ 53.985707] [ cut here ] [ 53.986314] kernel BUG at arch/x86/xen/enlighten.c:496! [ 53.986677] invalid opcode:

Re: [Xen-devel] [v4 16/17] vmx: Add some scheduler hooks for VT-d posted interrupts

2015-07-28 Thread Dario Faggioli
On Thu, 2015-07-23 at 19:35 +0800, Feng Wu wrote: > This patch adds the following arch hooks in scheduler: > - vmx_pre_ctx_switch_pi(): > It is called before context switch, we update the posted > interrupt descriptor when the vCPU is preempted, go to sleep, > or is blocked. > > - vmx_post_ctx_swi

Re: [Xen-devel] [PATCH v3 09/32] libxl: switch HVM domain building to use xc_dom_* helpers

2015-07-28 Thread Roger Pau Monné
El 28/07/15 a les 13.22, Wei Liu ha escrit: > On Fri, Jul 03, 2015 at 01:34:47PM +0200, Roger Pau Monne wrote: >> Now that we have all the code in place HVM domain building in libxl can be >> switched to use the xc_dom_* family of functions, just like they are used in >> order to build PV guests. >

Re: [Xen-devel] [PATCH v3 09/32] libxl: switch HVM domain building to use xc_dom_* helpers

2015-07-28 Thread Wei Liu
On Tue, Jul 28, 2015 at 04:26:15PM +0200, Roger Pau Monné wrote: > El 28/07/15 a les 13.22, Wei Liu ha escrit: > > On Fri, Jul 03, 2015 at 01:34:47PM +0200, Roger Pau Monne wrote: > >> Now that we have all the code in place HVM domain building in libxl can be > >> switched to use the xc_dom_* famil

Re: [Xen-devel] Xen master hangs

2015-07-28 Thread Doug Goldstein
On Mon, Jul 27, 2015 at 4:11 PM, Doug Goldstein wrote: > On Mon, Jul 27, 2015 at 4:55 AM, Andrew Cooper > wrote: >> On 24/07/15 17:52, Doug Goldstein wrote: >> So now I've got a build of staging 8398ec706ff60e17a5044470fa2e90a1b081f37a with debug=y and frame_pointer=n so that we can get a good

Re: [Xen-devel] [PATCH v12 0/8] Xen VMware tools support

2015-07-28 Thread Konrad Rzeszutek Wilk
On Mon, Jul 27, 2015 at 11:18:03PM -0400, Don Slutz wrote: > On 07/10/15 05:07, Wei Liu wrote: > > On Fri, Jul 10, 2015 at 09:40:59AM +0100, Jan Beulich wrote: > > On 28.06.15 at 01:27, wrote: > >>> Changes v11 to v12: > >>> Rebased on staging. > >>> > > > >> > >> I was actually meaning to

Re: [Xen-devel] [PATCH v4 0/3] x86: modify_ldt improvement, test, and config option

2015-07-28 Thread Andrew Cooper
On 28/07/15 15:05, Boris Ostrovsky wrote: > On 07/28/2015 06:29 AM, Andrew Cooper wrote: >> After forward-porting my virtio patches, I got this thing to run on Xen. After several tries, I got: [ 53.985707] [ cut here ] [ 53.986314] kernel BUG at

Re: [Xen-devel] The Bitdefender virtual machine introspection library is now on GitHub

2015-07-28 Thread Dario Faggioli
[Moving to publicity@, xen-devel@ on Bcc] On Tue, 2015-07-28 at 10:04 +0300, Razvan Cojocaru wrote: > Hello Tamas, > > > I've pinged the other LibVMI maintainers as well to take a look. In > > summary, what do you see as the benefit in libbdvmi over LibVMI? > > Well, libbdvmi addresses a slightl

Re: [Xen-devel] Dom0 crash with apache bench (ab)

2015-07-28 Thread Konrad Rzeszutek Wilk
On Tue, Jul 28, 2015 at 03:09:31PM +0200, Christoffer Dall wrote: > Hi, > > I've been doing some performance comparisons lately, and wanted to compare > the performance overhead of using Xen with apache bench, but unfortunately > the Dom0 kernel crashes when hitting it with ab from a remote machin

Re: [Xen-devel] [PATCH v4 0/3] x86: modify_ldt improvement, test, and config option

2015-07-28 Thread Boris Ostrovsky
On 07/28/2015 10:35 AM, Andrew Cooper wrote: On 28/07/15 15:05, Boris Ostrovsky wrote: On 07/28/2015 06:29 AM, Andrew Cooper wrote: After forward-porting my virtio patches, I got this thing to run on Xen. After several tries, I got: [ 53.985707] [ cut here ] [ 53.9

Re: [Xen-devel] [PATCHv2 01/10] mm: memory hotplug with an existing resource

2015-07-28 Thread Daniel Kiper
On Fri, Jul 24, 2015 at 12:47:39PM +0100, David Vrabel wrote: > Add add_memory_resource() to add memory using an existing "System RAM" > resource. This is useful if the memory region is being located by > finding a free resource slot with allocate_resource(). > > Xen guests will make use of this i

Re: [Xen-devel] Dom0 crash with apache bench (ab)

2015-07-28 Thread Ian Campbell
On Tue, 2015-07-28 at 10:50 -0400, Konrad Rzeszutek Wilk wrote: > On Tue, Jul 28, 2015 at 03:09:31PM +0200, Christoffer Dall wrote: > > Hi, > > > > I've been doing some performance comparisons lately, and wanted to > > compare > > the performance overhead of using Xen with apache bench, but > >

Re: [Xen-devel] Xen master hangs

2015-07-28 Thread Konrad Rzeszutek Wilk
On Tue, Jul 28, 2015 at 09:30:59AM -0500, Doug Goldstein wrote: > On Mon, Jul 27, 2015 at 4:11 PM, Doug Goldstein wrote: > > On Mon, Jul 27, 2015 at 4:55 AM, Andrew Cooper > > wrote: > >> On 24/07/15 17:52, Doug Goldstein wrote: > >> > > > > So now I've got a build of staging > 8398ec706ff60e1

[Xen-devel] [PATCH 8/8] xen/privcmd: Further s/MFN/GFN/ clean-up

2015-07-28 Thread Julien Grall
The privcmd code is mixing the usage of GFN and MFN within the same functions which make the code difficult to understand when you only work with auto-translated guests. The privcmd driver is only dealing with GFN so replace all the mention of MFN into GFN. The ioctl structure used to map foreign

[Xen-devel] [PATCH 6/8] video/xen-fbfront: Further s/MFN/GFN clean-up

2015-07-28 Thread Julien Grall
The PV driver xen-fbfront is only dealing with GFN and not MFN. Rename all the occurence of MFN to GFN. Also take the opportunity to replace to usage of pfn_to_gfn by page_to_pfn. Signed-off-by: Julien Grall Cc: Jean-Christophe Plagniol-Villard Cc: Tomi Valkeinen Cc: David Vrabel Cc: linux-fb

[Xen-devel] [PATCH 1/8] arm/xen: Remove helpers which are PV specific

2015-07-28 Thread Julien Grall
ARM guests are assimilated to HVM guest on ARM. The current implementation is assuming a 1:1 mapping which is only true for DOM0 and may not be at all in the future. Furthermore, all the helpers but arbitrary_virt_to_machine are used in x86 specific code (or only compiled for). The helper arbitra

[Xen-devel] [PATCH 5/8] xen/tmem: Use page_to_gfn rather than pfn_to_gfn

2015-07-28 Thread Julien Grall
All the caller of xen_tmem_{get,put}_page have a struct page * in hand and call pfn_to_gfn for the only benefits of these 2 functions. Rather than passing the pfn in parameter, pass directly the page and use directly page_to_gfn. Signed-off-by: Julien Grall Cc: Konrad Rzeszutek Wilk Cc: Boris O

[Xen-devel] [PATCH 7/8] hvc/xen: Further s/MFN/GFN clean-up

2015-07-28 Thread Julien Grall
HVM_PARAM_CONSOLE_PFN is used to retrieved the console PFN for HVM guest. It returns a PFN (aka GFN) and not a MFN. Furthermore, use directly virt_to_gfn for both PV and HVM domain rather than doing a special case for each of the them. Signed-off-by: Julien Grall Cc: Greg Kroah-Hartman Cc: Jiri

[Xen-devel] [PATCH 4/8] xen: Use the correctly the Xen memory terminologies

2015-07-28 Thread Julien Grall
Based on include/xen/mm.h [1], Linux is mistakenly using MFN when GFN is meant, I suspect this is because the first support for Xen was for PV. This brough some misimplementation of helpers on ARM and make the developper confused the expected behavior. For instance, with pfn_to_mfn, we expect to g

[Xen-devel] [PATCH 0/8] Use correctly the Xen memory terminologies in Linux

2015-07-28 Thread Julien Grall
Hi all, This patch series aims to use the memory terminologies described in include/linux/mm.h [1] for Linux xen code. Linux is using mistakenly MFN when GFN is meant, I suspect this is because the first support of Xen was for PV. This has brought some misimplementation of memory helpers on ARM a

[Xen-devel] [PATCH 2/8] xen: Make clear that swiotlb and biomerge are dealing with DMA address

2015-07-28 Thread Julien Grall
The swiotlb is required when programming a DMA address on ARM when a device is not protected by an IOMMU. In this case, the DMA address should always be equal to the machine address. For DOM0 memory, Xen ensure it by have an identity mapping between the guest address and host address. However, whe

  1   2   >