Re: [Xen-devel] [PATCH] Config.mk: update OVMF changeset

2015-10-23 Thread Ian Campbell
On Fri, 2015-10-23 at 02:01 -0600, Jan Beulich wrote: > > > > On 22.10.15 at 19:08, wrote: > > Stefano Stabellini writes ("Re: [PATCH] Config.mk: update OVMF > > changeset"): > > > On Thu, 22 Oct 2015, Ian Campbell wrote: > > > > This was discussed prior to Wei

[Xen-devel] Adding Xen development headers to XenServer

2015-10-23 Thread D'Mita Levy
Is it possible to add Xen development headers to XenServer? I would like to do a little bit of VM introspection with LibVMI on XenServer. I have all the other dependencies installed (glib, flex, etc.) just missing the ones for Xen...Any help is greatly appreciated. Thanks, D'Mita -- D'Mita Levy

Re: [Xen-devel] Adding Xen development headers to XenServer

2015-10-23 Thread Andrew Cooper
On 23/10/15 19:07, D'Mita Levy wrote: > Is it possible to add Xen development headers to XenServer? I would > like to do a little bit of VM introspection with LibVMI on XenServer. > I have all the other dependencies installed (glib, flex, etc.) just > missing the ones for Xen...Any help is greatly

Re: [Xen-devel] Adding Xen development headers to XenServer

2015-10-23 Thread Razvan Cojocaru
On 10/23/2015 09:07 PM, D'Mita Levy wrote: > Is it possible to add Xen development headers to XenServer? I would like > to do a little bit of VM introspection with LibVMI on XenServer. I have > all the other dependencies installed (glib, flex, etc.) just missing the > ones for Xen...Any help is

[Xen-devel] [PATCH 2/3] xl: convert vcpu related return codes to EXIT_[SUCCESS|FAILURE]

2015-10-23 Thread Harmandeep Kaur
turning vcpu manipulation functions xl exit codes toward using the EXIT_[SUCCESS|FAILURE] macros, instead of instead of arbitrary numbers or libxl return codes. Signed-off-by: Harmandeep Kaur --- tools/libxl/xl_cmdimpl.c | 44

Re: [Xen-devel] [PATCH] xl: log an error if libxl_cpupool_destroy() fails

2015-10-23 Thread Wei Liu
On Thu, Oct 22, 2015 at 07:14:20PM +0200, Dario Faggioli wrote: > In fact, right now, failing at destroying a cpupool is just > not reported to the user in any explicit way. Log an error, > as it is customary for xl in these cases. > > While there, take the chance to turn a couple of xl exit >

Re: [Xen-devel] [PATCH v8 00/17] Add VT-d Posted-Interrupts support

2015-10-23 Thread Dario Faggioli
On Fri, 2015-10-23 at 08:35 +, Wu, Feng wrote: > > > -Original Message- > > From: Jan Beulich [mailto:jbeul...@suse.com] > > > > > On 23.10.15 at 04:12, wrote: > > > Kindly ping ... > > > > I can understand that you want to get this to make progress, but as > >

[Xen-devel] [PATCH 3/3] xl: convert cpupool related return codes to EXIT_[SUCCESS|FAILURE]

2015-10-23 Thread Harmandeep Kaur
turning cpupools related functions xl exit codes towards using the EXIT_[SUCCESS|FAILURE] macros, instead of instead of arbitrary numbers or libxl return codes. Signed-off-by: Harmandeep Kaur --- tools/libxl/xl_cmdimpl.c | 44

Re: [Xen-devel] [PATCH 0/2] limit xen vnic max queues number to online cpu number

2015-10-23 Thread Paul Durrant
> -Original Message- > From: netdev-ow...@vger.kernel.org [mailto:netdev- > ow...@vger.kernel.org] On Behalf Of Joe Jin > Sent: 23 October 2015 08:54 > To: Wei Liu; Ian Campbell; Boris Ostrovsky; Konrad Rzeszutek Wilk; David S. > Miller > Cc: net...@vger.kernel.org;

Re: [Xen-devel] [PATCH v5 7/9] x86/intel_pstate: add a booting param to select the driver to load

2015-10-23 Thread Wang, Wei W
On 23/10/2015 16:36, Jan Beulich wrote: > >>> On 23.10.15 at 10:18, wrote: > > On 07/10/2015 23:46, Jan Beulich wrote: > >> >>> On 14.09.15 at 04:32, wrote: > >> > --- a/xen/arch/x86/acpi/cpufreq/cpufreq.c > >> > +++

[Xen-devel] [PATCH 1/3] xl: convert scheduling related return codes to EXIT_[SUCCESS|FAILURE]

2015-10-23 Thread Harmandeep Kaur
turning scheduling related functions xl exit codes towards using the EXIT_[SUCCESS|FAILURE] macros, instead of instead of arbitrary numbers or libxl return codes. Signed-off-by: Harmandeep Kaur --- tools/libxl/xl_cmdimpl.c | 67

[Xen-devel] [distros-debian-jessie test] 38201: tolerable all pass

2015-10-23 Thread Platform Team regression test user
flight 38201 distros-debian-jessie real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/38201/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-armhf-armhf-armhf-jessie-netboot-pygrub 12 saverestore-support-check fail never pass

Re: [Xen-devel] [PATCH v8 00/17] Add VT-d Posted-Interrupts support

2015-10-23 Thread Wu, Feng
> -Original Message- > From: Dario Faggioli [mailto:dario.faggi...@citrix.com] > Sent: Friday, October 23, 2015 4:46 PM > To: Wu, Feng ; Jan Beulich > Cc: Lars Kurth ; george.dun...@eu.citrix.com; Li, Susie >

Re: [Xen-devel] [PATCH 0/2] limit xen vnic max queues number to online cpu number

2015-10-23 Thread Joe Jin
On 10/23/2015 04:47 PM, Paul Durrant wrote: >> -Original Message- >> From: netdev-ow...@vger.kernel.org [mailto:netdev- >> ow...@vger.kernel.org] On Behalf Of Joe Jin >> Sent: 23 October 2015 08:54 >> To: Wei Liu; Ian Campbell; Boris Ostrovsky; Konrad Rzeszutek Wilk; David S. >> Miller >>

Re: [Xen-devel] [PATCH 0/2] limit xen vnic max queues number to online cpu number

2015-10-23 Thread Jan Beulich
>>> On 23.10.15 at 11:05, wrote: > On 10/23/2015 04:47 PM, Paul Durrant wrote: >>> -Original Message- >>> From: netdev-ow...@vger.kernel.org [mailto:netdev- >>> ow...@vger.kernel.org] On Behalf Of Joe Jin >>> Sent: 23 October 2015 08:54 >>> To: Wei Liu; Ian Campbell;

Re: [Xen-devel] [PATCH V3 1/2] xen-netback: limit xen vif max queues number to online cpus

2015-10-23 Thread Andrew Cooper
On 23/10/15 10:44, Joe Jin wrote: > Should not allocate xen vif queues number more than online cpus. > > Signed-off-by: Joe Jin > Cc: Jan Beulich > Cc: Wei Liu > Cc: Ian Campbell > Cc: Boris Ostrovsky

[Xen-devel] [PATCH v2] x86/mm: pod: Use the correct memory flags for alloc_domheap_page{, s}

2015-10-23 Thread Julien Grall
The last parameter of alloc_domheap_page{s,} contain the memory flags and not the order of the allocation. Use 0 for the call in p2m_pod_set_cache_target as it was before 1069d63c5ef2510d08b83b2171af660e5bb18c63 "x86/mm/p2m: use defines for page sizes". Note that PAGE_ORDER_4K is also equal to 0

[Xen-devel] [PATCHv1 2/2] passthrough: improve locking when iterating over interrupts bound to VMs

2015-10-23 Thread David Vrabel
radix_tree_gang_lookup() only requires a RCU read lock, not the per-domain event_lock. Introduce a new RCU read lock and take the per-interrupt lock before calling the callback instead. This eliminates all contention on the event_lock when injecting interrupts from passthrough devices.

[Xen-devel] [PATCHv1 1/2] passthrough: use per-interrupt lock when injecting an interrupt

2015-10-23 Thread David Vrabel
The use of the per-domain event_lock in hvm_dirq_assist() does not scale with many VCPUs or interrupts. Add a per-interrupt lock to reduce contention. When a interrupt for a passthrough device is being setup or teared down, we must take both the event_lock and this new lock. Signed-off-by:

Re: [Xen-devel] [PATCH] Config.mk: update OVMF changeset

2015-10-23 Thread Stefano Stabellini
On Fri, 23 Oct 2015, Ian Jackson wrote: > Stefano Stabellini writes ("Re: [PATCH] Config.mk: update OVMF changeset"): > > On Fri, 23 Oct 2015, Ian Campbell wrote: > > > Yes. Those (that?) and the reasons why we aren't just trivially taking > > > them > > > are explained in the referenced thread.

[Xen-devel] [PATCH V2 0/2] limit xen vnic max queues number to online cpus number

2015-10-23 Thread Joe Jin
Currently xen vnic allowed to create lots of queues by set module param max_queues(both netback and netfront), when queues number larger than cpus number, it does not help for performance but need more cpu time. This patchset limit netback and netfront max queues number to online cpus number.

Re: [Xen-devel] [PATCH 04/13] x86/hvm: Setup TSC scaling ratio

2015-10-23 Thread Jan Beulich
>>> On 23.10.15 at 10:40, wrote: > On Fri, Oct 23, 2015 at 02:31:11AM -0600, Jan Beulich wrote: >> >>> On 23.10.15 at 10:18, wrote: >> > On Fri, Oct 23, 2015 at 01:59:46AM -0600, Jan Beulich wrote: >> >> >>> On 23.10.15 at 09:44,

[Xen-devel] [PATCH V2 1/2] xen-netback: limit xen vif max queues number to online cpus

2015-10-23 Thread Joe Jin
Should not allocate xen vif queues number more than online cpus. Signed-off-by: Joe Jin Cc: Jan Beulich Cc: Wei Liu Cc: Ian Campbell Cc: Boris Ostrovsky Cc: Konrad Rzeszutek Wilk

Re: [Xen-devel] [PATCH v4 2/5] xen/arm: vgic-v2: Don't ignore a write in ITARGETSR if one field is 0

2015-10-23 Thread Ian Campbell
On Thu, 2015-10-22 at 17:51 +0100, Julien Grall wrote: > On 22/10/15 17:07, Ian Campbell wrote: > > > diff --git a/xen/arch/arm/vgic-v2.c b/xen/arch/arm/vgic-v2.c > > > index 665afeb..6b7eab3 100644 > > > --- a/xen/arch/arm/vgic-v2.c > > > +++ b/xen/arch/arm/vgic-v2.c > > > @@ -50,6 +50,94 @@ void

Re: [Xen-devel] [PATCH 0/2] limit xen vnic max queues number to online cpu number

2015-10-23 Thread Paul Durrant
> -Original Message- > From: Joe Jin [mailto:joe@oracle.com] > Sent: 23 October 2015 10:05 > To: Paul Durrant; Wei Liu; Ian Campbell; Boris Ostrovsky; Konrad Rzeszutek > Wilk; David S. Miller > Cc: net...@vger.kernel.org; xen-de...@lists.xenproject.org > Subject: Re: [PATCH 0/2] limit

Re: [Xen-devel] [PATCH 1/3] xl: convert scheduling related return codes to EXIT_[SUCCESS|FAILURE]

2015-10-23 Thread Ian Campbell
On Fri, 2015-10-23 at 10:30 +0200, Dario Faggioli wrote: > On Fri, 2015-10-23 at 13:18 +0530, Harmandeep Kaur wrote: > > turning scheduling related functions xl exit codes towards using the > > EXIT_[SUCCESS|FAILURE] macros, instead of instead of arbitrary > > numbers > > or libxl return codes. >

Re: [Xen-devel] [PATCH v4 3/5] xen/arm: vgic: Optimize the way to store the target vCPU in the rank

2015-10-23 Thread Julien Grall
On 23/10/15 11:14, Ian Campbell wrote: > On Fri, 2015-10-23 at 11:01 +0100, Julien Grall wrote: >> On 23/10/15 10:34, Ian Campbell wrote: >>> On Thu, 2015-10-22 at 18:15 +0100, Julien Grall wrote: Hi Ian, On 22/10/15 17:17, Ian Campbell wrote: > On Mon, 2015-10-12 at 15:22

Re: [Xen-devel] [PATCH 1/3] xl: convert scheduling related return codes to EXIT_[SUCCESS|FAILURE]

2015-10-23 Thread Ian Campbell
On Fri, 2015-10-23 at 12:03 +0200, Dario Faggioli wrote: > On Fri, 2015-10-23 at 10:56 +0100, Ian Campbell wrote: > > On Fri, 2015-10-23 at 10:30 +0200, Dario Faggioli wrote: > > > > Harman, when you send a patch series, as you did here (thanks and > > > good > > > job doing it so quickly :-) ),

Re: [Xen-devel] Work on a project: Refactor Linux hotplug scripts

2015-10-23 Thread Roger Pau Monné
El 23/10/15 a les 7.53, Надежда Ампилогова ha escrit: > Hi all, > > I am interested in the project Refactor Linux hotplug scripts for > Outreachy(Round 11). I am more of an intermediate-beginner in c. Please can > anyone provide some pointers and helper about the things I need to do to join >

Re: [Xen-devel] [PATCH for-4.6] xen/public: arm: Use __typeof__ rather than typeof

2015-10-23 Thread Jan Beulich
>>> On 23.10.15 at 15:58, wrote: > On 23/10/15 14:52, Jan Beulich wrote: > On 23.10.15 at 15:30, wrote: >>> On Fri, 2015-10-23 at 14:13 +0100, Julien Grall wrote: Hi, On 04/10/15 20:24, Julien Grall wrote: > The keyword

Re: [Xen-devel] [PATCH for-4.6] xen/public: arm: Use __typeof__ rather than typeof

2015-10-23 Thread Ian Campbell
On Fri, 2015-10-23 at 15:31 +0100, Ian Campbell wrote: > On Fri, 2015-10-23 at 08:16 -0600, Jan Beulich wrote: > > > > > On 23.10.15 at 15:58, wrote: > > > On 23/10/15 14:52, Jan Beulich wrote: > > > > > > > On 23.10.15 at 15:30, wrote: > > > > >

Re: [Xen-devel] [PATCH] xl: log an error if libxl_cpupool_destroy() fails

2015-10-23 Thread Dario Faggioli
On Fri, 2015-10-23 at 15:09 +0100, Ian Campbell wrote: > On Fri, 2015-10-23 at 09:43 +0100, Wei Liu wrote: > Looking at those links, I'm not sure that either you or myself was > thinking > of using EXIT_* as function return values, just that the eventual > process > exit would become one of those

Re: [Xen-devel] arm64: iomem_resource doesn't contain all the region used

2015-10-23 Thread Ian Campbell
On Fri, 2015-10-23 at 15:58 +0100, Julien Grall wrote: > Is there any way we could register the IO region used on ARM without > having to enforce it in all the drivers? This seems like an uphill battle to me. Why not do as I suggested IRL yesterday and expose the map of "potential RAM" addresses

[Xen-devel] [PATCH] xl: Die on unknown options

2015-10-23 Thread Ian Jackson
def_getopt would print a message to stderr, but blunder on anyway. Sadly this is probably not a backport candidate. Signed-off-by: Ian Jackson --- tools/libxl/xl_cmdimpl.c |1 + 1 file changed, 1 insertion(+) diff --git a/tools/libxl/xl_cmdimpl.c

Re: [Xen-devel] [PATCH] xl: log an error if libxl_cpupool_destroy() fails

2015-10-23 Thread Ian Campbell
On Fri, 2015-10-23 at 17:12 +0200, Dario Faggioli wrote: > On Fri, 2015-10-23 at 15:09 +0100, Ian Campbell wrote: > > On Fri, 2015-10-23 at 09:43 +0100, Wei Liu wrote: > > > Looking at those links, I'm not sure that either you or myself was > > thinking > > of using EXIT_* as function return

Re: [Xen-devel] [PATCH] xl: log an error if libxl_cpupool_destroy() fails

2015-10-23 Thread Dario Faggioli
On Fri, 2015-10-23 at 16:40 +0100, Ian Campbell wrote: > On Fri, 2015-10-23 at 17:12 +0200, Dario Faggioli wrote: > > On Fri, 2015-10-23 at 15:09 +0100, Ian Campbell wrote: > > > I think what I would have been expecting is for the xl internal > > > error > > > code > > Well, same here. Except,

[Xen-devel] [xen-4.5-testing test] 63214: regressions - trouble: blocked/broken/fail/pass

2015-10-23 Thread osstest service owner
flight 63214 xen-4.5-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/63214/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf 4 host-build-prep fail REGR. vs. 63099

Re: [Xen-devel] [PATCH V2 2/2] xen-netfront: limit vnic max_queues number to online cpus

2015-10-23 Thread Jan Beulich
>>> On 23.10.15 at 11:19, wrote: > > @@ -164,6 +166,19 @@ struct netfront_rx_info { > struct xen_netif_extra_info extras[XEN_NETIF_EXTRA_TYPE_MAX - 1]; > }; > > +static int xennet_set_max_queues(const char *val, struct kernel_param *kp) > +{ > + unsigned int

[Xen-devel] [PATCH V3 0/2] limit xen vnic max queues number to online cpus number

2015-10-23 Thread Joe Jin
Currently xen vnic allowed to create lots of queues by set module param max_queues(both netback and netfront), when queues number larger than cpus number, it does not help for performance but need more cpu time. This patchset limit netback and netfront max queues number to online cpus number.

Re: [Xen-devel] [PATCH] Config.mk: update OVMF changeset

2015-10-23 Thread Stefano Stabellini
On Fri, 23 Oct 2015, Ian Campbell wrote: > On Fri, 2015-10-23 at 02:01 -0600, Jan Beulich wrote: > > > > > On 22.10.15 at 19:08, wrote: > > > Stefano Stabellini writes ("Re: [PATCH] Config.mk: update OVMF > > > changeset"): > > > > On Thu, 22 Oct 2015, Ian Campbell

Re: [Xen-devel] [PATCH QEMU-XEN v4 9/9] xen: make it possible to build without the Xen PV domain builder

2015-10-23 Thread Ian Campbell
On Fri, 2015-10-23 at 12:35 +0100, Stefano Stabellini wrote: > On Fri, 23 Oct 2015, Ian Campbell wrote: > > On Fri, 2015-10-23 at 12:12 +0100, Stefano Stabellini wrote: > > > @@ -2113,6 +2117,15 @@ if test "$xen_pci_passthrough" != "no"; then > > > >fi > > > > fi > > > > > > > > +if test

Re: [Xen-devel] passthrough: improve interrupt injection locking

2015-10-23 Thread David Vrabel
On 23/10/15 13:37, Ian Campbell wrote: > On Fri, 2015-10-23 at 12:05 +0100, David Vrabel wrote: >> When injecting an interrupt for a passthrough device into a guest, the >> per-domain event_lock is held, reducing performance when a guest has >> many VCPUs and high interrupt rates. > > Did you CC

[Xen-devel] [PATCH 03/13] python: remove flask related libxc python bindings

2015-10-23 Thread Juergen Gross
Mostly for historical reasons Xen includes Python bindings for libxc. They have been used by xm/xend in the past but nowadays there is no user of any flask related python binding left. Remove them. Signed-off-by: Juergen Gross --- tools/python/xen/lowlevel/xc/xc.c | 238

[Xen-devel] [PATCH 09/13] python: remove domain handling related libxc python bindings

2015-10-23 Thread Juergen Gross
Mostly for historical reasons Xen includes Python bindings for libxc. They have been used by xm/xend in the past but nowadays there is no user of domain handling related python binding left. Remove them. Signed-off-by: Juergen Gross --- tools/python/xen/lowlevel/xc/xc.c | 358

[Xen-devel] [PATCH 01/13] libxc: remove most of tools/libxc/xc_dom_compat_linux.c

2015-10-23 Thread Juergen Gross
In tools/libxc/xc_dom_compat_linux.c xc_linux_build() is the only domain building function used by an in-tree component (qemu-xen) which is really necessary. Remove the other domain building functions and the unused python wrapper xc.linux_build() referencing one of the to be removed functions.

Re: [Xen-devel] [PATCH] Config.mk: update OVMF changeset

2015-10-23 Thread Fabio Fantoni
Il 23/10/2015 14:56, Ian Campbell ha scritto: On Fri, 2015-10-23 at 13:43 +0100, Stefano Stabellini wrote: We have no existing stable baseline for that arch, and no testing or reason to believe that cb9a7eb (the Config.mk version currently referenced by 4.6) as being any good at all on that

Re: [Xen-devel] [PATCH 3/3] xl: convert cpupool related return codes to EXIT_[SUCCESS|FAILURE]

2015-10-23 Thread Harmandeep Kaur
On Fri, Oct 23, 2015 at 5:40 PM, Dario Faggioli wrote: > > int main_cpupooldestroy(int argc, char **argv) > > @@ -7580,13 +7580,13 @@ int main_cpupooldestroy(int argc, char > > **argv) > > if (libxl_cpupool_qualifier_to_cpupoolid(ctx, pool, , > > NULL) || > >

[Xen-devel] [xen-4.3-testing test] 63212: tolerable FAIL - PUSHED

2015-10-23 Thread osstest service owner
flight 63212 xen-4.3-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/63212/ Failures :-/ but no regressions. Tests which are failing intermittently (not blocking): test-amd64-amd64-xl-qemuu-winxpsp3 15 guest-localmigrate.2 fail in 63151 pass in 63212

Re: [Xen-devel] [PATCH v2 3/6] xen: sched: clarify use cases of schedule_cpu_switch()

2015-10-23 Thread Dario Faggioli
On Fri, 2015-10-23 at 15:42 +0200, Juergen Gross wrote: > On 10/21/2015 07:10 PM, Dario Faggioli wrote: > > Also, all the operations done in schedule_cpu_switch() itself, > > depend > > either on per_cpu(scheduler) or on per_cpu(schedule_data) being > > updated > > properly, rather than on

Re: [Xen-devel] [PATCH v4 1/4] xen/arm: vgic-v2: Report the correct GICC size to the guest

2015-10-23 Thread Ian Campbell
On Fri, 2015-10-23 at 14:34 +0100, Julien Grall wrote: > Hi Ian, > > On 23/10/15 14:28, Ian Campbell wrote: > > On Thu, 2015-10-08 at 19:23 +0100, Julien Grall wrote: > > > The GICv2 DT node is usually used by the guest to know the > > > address/size > > > of the regions (GICD, GICC...) to map

Re: [Xen-devel] [PATCH for-4.6] xen/public: arm: Use __typeof__ rather than typeof

2015-10-23 Thread Ian Campbell
On Fri, 2015-10-23 at 14:13 +0100, Julien Grall wrote: > Hi, > > On 04/10/15 20:24, Julien Grall wrote: > > The keyword typeof is not portable: > > > > /usr/src/freebsd/sys/xen/hypervisor.h:93:2: error: implicit declaration > > of function 'typeof' is invalid in C99 > >

Re: [Xen-devel] [PATCH v4 3/4] xen/arm: gic-v2: Automatically detect aliased GIC400

2015-10-23 Thread Ian Campbell
On Thu, 2015-10-08 at 19:23 +0100, Julien Grall wrote: > We are currently using a per-platform quirk to know if the 2 4KB region > of > the GIC CPU interface are each aligned to 64KB. Although, it may be > possible to have different layout on a same platform (depending on the > firmware version).

Re: [Xen-devel] [PATCH] Config.mk: update OVMF changeset

2015-10-23 Thread Ian Campbell
On Fri, 2015-10-23 at 15:16 +0200, Fabio Fantoni wrote: > A recent ovmf patch (already tested) I think is good to backport is: Pointing out backport candidates in the depths of a thread such as this is a sure fire way to ensure they get missed I'm afraid. Please make such requests explicitly in

Re: [Xen-devel] [PATCH] mm: hotplug: Don't release twice the resource on error

2015-10-23 Thread David Vrabel
On 23/10/15 13:57, Julien Grall wrote: > The function add_memory_resource take in parameter a resource allocated > by the caller. On error, both add_memory_resource and the caller will > release the resource via release_memory_source. [...] > --- a/mm/memory_hotplug.c > +++ b/mm/memory_hotplug.c >

Re: [Xen-devel] [PATCH] xl: log an error if libxl_cpupool_destroy() fails

2015-10-23 Thread Ian Campbell
On Fri, 2015-10-23 at 09:43 +0100, Wei Liu wrote: > On Thu, Oct 22, 2015 at 07:14:20PM +0200, Dario Faggioli wrote: > > In fact, right now, failing at destroying a cpupool is just > > not reported to the user in any explicit way. Log an error, > > as it is customary for xl in these cases. > > > >

[Xen-devel] [PATCH] mm: hotplug: Don't release twice the resource on error

2015-10-23 Thread Julien Grall
The function add_memory_resource take in parameter a resource allocated by the caller. On error, both add_memory_resource and the caller will release the resource via release_memory_source. This will result to Linux crashing when the caller is trying to release the resource: CPU: 1 PID: 45 Comm:

Re: [Xen-devel] [PATCH v4 2/4] xen/arm: gic: Check the size of the CPU and vCPU interface retrieved from DT

2015-10-23 Thread Ian Campbell
On Thu, 2015-10-08 at 19:23 +0100, Julien Grall wrote: > The size of the CPU interface will used in a follow-up patch to map the ^be > region in Xen memory. > > Based on GICv2 spec, the CPU interface should at least be 8KB, although > most of the platform we

Re: [Xen-devel] [OSSTEST PATCH 26/26] ts-xen-install: networking: Rename `nodhcp' to `ensurebridge'

2015-10-23 Thread Hu, Robert
> -Original Message- > From: Ian Campbell [mailto:ian.campb...@citrix.com] > Sent: Friday, October 23, 2015 4:15 PM > To: Hu, Robert ; 'Ian Jackson' > ; 'xen-de...@lists.xenproject.org' > > Subject: Re:

Re: [Xen-devel] [PATCH v4 1/4] xen/arm: vgic-v2: Report the correct GICC size to the guest

2015-10-23 Thread Julien Grall
On 23/10/15 14:39, Ian Campbell wrote: > On Fri, 2015-10-23 at 14:34 +0100, Julien Grall wrote: >> Hi Ian, >> >> On 23/10/15 14:28, Ian Campbell wrote: >>> On Thu, 2015-10-08 at 19:23 +0100, Julien Grall wrote: The GICv2 DT node is usually used by the guest to know the address/size

Re: [Xen-devel] Work on a project: Refactor Linux hotplug scripts

2015-10-23 Thread Roger Pau Monné
El 23/10/15 a les 13.33, Roger Pau Monné ha escrit: > El 23/10/15 a les 7.53, Надежда Ампилогова ha escrit: >> Hi all, >> >> I am interested in the project Refactor Linux hotplug scripts for >> Outreachy(Round 11). I am more of an intermediate-beginner in c. Please can >> anyone provide some

Re: [Xen-devel] [PATCH v4 1/4] xen/arm: vgic-v2: Report the correct GICC size to the guest

2015-10-23 Thread Ian Campbell
On Thu, 2015-10-08 at 19:23 +0100, Julien Grall wrote: > The GICv2 DT node is usually used by the guest to know the address/size > of the regions (GICD, GICC...) to map into their virtual memory. > > While the GICv2 spec requires the size of the GICC to be 8KB, we > correctly do an 8KB stage-2

Re: [Xen-devel] [PATCH] Config.mk: update OVMF changeset

2015-10-23 Thread Stefano Stabellini
On Fri, 23 Oct 2015, Ian Campbell wrote: > On Fri, 2015-10-23 at 12:18 +0100, Ian Jackson wrote: > > Stefano Stabellini writes ("Re: [PATCH] Config.mk: update OVMF > > changeset"): > > > On Fri, 23 Oct 2015, Ian Campbell wrote: > > > > Yes. Those (that?) and the reasons why we aren't just

Re: [Xen-devel] passthrough: improve interrupt injection locking

2015-10-23 Thread Ian Campbell
On Fri, 2015-10-23 at 13:38 +0100, David Vrabel wrote: > On 23/10/15 13:37, Ian Campbell wrote: > > On Fri, 2015-10-23 at 12:05 +0100, David Vrabel wrote: > > > When injecting an interrupt for a passthrough device into a guest, > > > the > > > per-domain event_lock is held, reducing performance

Re: [Xen-devel] [PATCH for-4.6] xen/public: arm: Use __typeof__ rather than typeof

2015-10-23 Thread Julien Grall
Hi, On 04/10/15 20:24, Julien Grall wrote: > The keyword typeof is not portable: > > /usr/src/freebsd/sys/xen/hypervisor.h:93:2: error: implicit declaration > of function 'typeof' is invalid in C99 > [-Werror,-Wimplicit-function-declaration] Ping? Aside the fact that other bits of the header

Re: [Xen-devel] [PATCH v4 4/4] xen/arm: platform: Drop the quirks callback

2015-10-23 Thread Ian Campbell
On Thu, 2015-10-08 at 19:23 +0100, Julien Grall wrote: > All the quirks has been replaced by proper detection. Lets drop the have > callback and hope that no one will need new quirks. > > At the same time, remove the definition platform_dom0_evtchn_ppi with is

Re: [Xen-devel] [PATCH] libxl: Do not call assert() in signal handlers

2015-10-23 Thread Ian Campbell
On Thu, 2015-10-22 at 17:22 +0100, Ian Campbell wrote: > On Thu, 2015-10-22 at 16:39 +0100, Ian Jackson wrote: > > assert is not async-signal-safe. > > I don't doubt you, but I'm curious regarding a reference. > > http://pubs.opengroup.org/onlinepubs/9699919799/functions/assert.html doe > sn >

Re: [Xen-devel] [PATCH for-4.6] xen/public: arm: Use __typeof__ rather than typeof

2015-10-23 Thread Julien Grall
On 23/10/15 14:52, Jan Beulich wrote: On 23.10.15 at 15:30, wrote: >> On Fri, 2015-10-23 at 14:13 +0100, Julien Grall wrote: >>> Hi, >>> >>> On 04/10/15 20:24, Julien Grall wrote: The keyword typeof is not portable:

Re: [Xen-devel] [PATCH] tools/python: remove broken xl binding

2015-10-23 Thread Ian Campbell
On Thu, 2015-10-08 at 12:10 -0400, Zhigang Wang wrote: > On 10/08/2015 11:28 AM, Ian Campbell wrote: > > On Thu, 2015-10-08 at 10:58 -0400, Zhigang Wang wrote: > > > On 10/08/2015 10:40 AM, Ian Campbell wrote: > > > > On Tue, 2015-10-06 at 15:09 -0400, Konrad Rzeszutek Wilk wrote: > > > > > On

Re: [Xen-devel] [PATCH for-4.6] xen/public: arm: Use __typeof__ rather than typeof

2015-10-23 Thread Ian Campbell
On Fri, 2015-10-23 at 07:52 -0600, Jan Beulich wrote: > > > > On 23.10.15 at 15:30, wrote: > > On Fri, 2015-10-23 at 14:13 +0100, Julien Grall wrote: > > > Hi, > > > > > > On 04/10/15 20:24, Julien Grall wrote: > > > > The keyword typeof is not portable: > > > > > > > >

Re: [Xen-devel] [PATCH v2] x86/mm: pod: Use the correct memory flags for alloc_domheap_page{, s}

2015-10-23 Thread Dario Faggioli
On Fri, 2015-10-23 at 11:33 +0100, Julien Grall wrote: > The last parameter of alloc_domheap_page{s,} contain the memory flags > and > not the order of the allocation. > > Use 0 for the call in p2m_pod_set_cache_target as it was before > 1069d63c5ef2510d08b83b2171af660e5bb18c63 "x86/mm/p2m: use

Re: [Xen-devel] [PATCH QEMU-XEN v4 3/9] xen: Switch to libxengnttab interface for compat shims.

2015-10-23 Thread Ian Campbell
On Fri, 2015-10-23 at 12:06 +0100, Stefano Stabellini wrote: > On Wed, 21 Oct 2015, Ian Campbell wrote: > > In Xen 4.7 we are refactoring parts libxenctrl into a number of > > separate libraries which will provide backward and forward API and ABI > > compatiblity. > > > > One such library will be

[Xen-devel] [PATCH 12/13] python: remove permission related libxc python bindings

2015-10-23 Thread Juergen Gross
Mostly for historical reasons Xen includes Python bindings for libxc. They have been used by xm/xend in the past but nowadays there is no user of permission related python binding left. Remove them. Signed-off-by: Juergen Gross --- tools/python/xen/lowlevel/xc/xc.c | 97

[Xen-devel] [PATCH 13/13] python: remove unused other libxc python bindings

2015-10-23 Thread Juergen Gross
Mostly for historical reasons Xen includes Python bindings for libxc. They have been used by xm/xend in the past but nowadays there are only some few python bindings in use, most of the in out of tree tools. Remove all unused libxc python bindings. Signed-off-by: Juergen Gross

[Xen-devel] [PATCH 08/13] python: remove unused memory related libxc python bindings

2015-10-23 Thread Juergen Gross
Mostly for historical reasons Xen includes Python bindings for libxc. They have been used by xm/xend in the past but nowadays there is no user of most memory related python binding left. Remove them. Especially xc.domain_set_target_mem() and xc.domain_setmaxmem() will not be removed as there are

[Xen-devel] [PATCH 11/13] python: remove hvm related libxc python bindings

2015-10-23 Thread Juergen Gross
Mostly for historical reasons Xen includes Python bindings for libxc. They have been used by xm/xend in the past but nowadays there is no user of hvm related python binding left. Remove them. Signed-off-by: Juergen Gross --- tools/python/xen/lowlevel/xc/xc.c | 92

[Xen-devel] [PATCH 07/13] python: remove scheduler related libxc python bindings

2015-10-23 Thread Juergen Gross
Mostly for historical reasons Xen includes Python bindings for libxc. They have been used by xm/xend in the past but nowadays there is no user of any scheduler related python binding left. Remove them. Signed-off-by: Juergen Gross --- tools/python/xen/lowlevel/xc/xc.c | 133

[Xen-devel] [PATCH 02/13] libxc: remove xc_get_bit_size() from tools/libxc/xc_dom_compat_linux.c

2015-10-23 Thread Juergen Gross
xc_get_bit_size() is being used by the unused python wrapper xc.getBitSize() only. Remove the wrapper and xc_get_bit_size(). Signed-off-by: Juergen Gross --- tools/libxc/include/xenguest.h| 4 tools/libxc/xc_dom_compat_linux.c | 25 -

[Xen-devel] [PATCH 04/13] python: remove cpupool related libxc python bindings

2015-10-23 Thread Juergen Gross
Mostly for historical reasons Xen includes Python bindings for libxc. They have been used by xm/xend in the past but nowadays there is no user of any cpupool related python binding left. Remove them. Signed-off-by: Juergen Gross --- tools/python/xen/lowlevel/xc/xc.c | 224

[Xen-devel] [PATCH 05/13] python: remove cpuid related libxc python bindings

2015-10-23 Thread Juergen Gross
Mostly for historical reasons Xen includes Python bindings for libxc. They have been used by xm/xend in the past but nowadays there is no user of any cpuid related python binding left. Remove them. Signed-off-by: Juergen Gross --- tools/python/xen/lowlevel/xc/xc.c | 128

[Xen-devel] [PATCH 06/13] python: remove device related libxc python bindings

2015-10-23 Thread Juergen Gross
Mostly for historical reasons Xen includes Python bindings for libxc. They have been used by xm/xend in the past but nowadays there is no user of any device related python binding left. Remove them. Signed-off-by: Juergen Gross --- tools/python/xen/lowlevel/xc/xc.c | 226

[Xen-devel] [PATCH 10/13] python: remove vcpu related libxc python bindings

2015-10-23 Thread Juergen Gross
Mostly for historical reasons Xen includes Python bindings for libxc. They have been used by xm/xend in the past but nowadays there is no user of vcpu related python binding left. Remove them. Signed-off-by: Juergen Gross --- tools/python/xen/lowlevel/xc/xc.c | 134

[Xen-devel] [PATCH 00/13] tools: do cleanups related to libxc python bindings

2015-10-23 Thread Juergen Gross
This series is a combination of my previous patches: "libxc: remove most of tools/libxc/xc_dom_compat_linux.c" "tools: remove unused wrappers for python" I have split it up as requested by Ian Campbell, thus it consists of 13 patches instead just of 2, but the functionality is roughly the same.

Re: [Xen-devel] [PATCH v4 1/4] xen/arm: vgic-v2: Report the correct GICC size to the guest

2015-10-23 Thread Julien Grall
Hi Ian, On 23/10/15 14:28, Ian Campbell wrote: > On Thu, 2015-10-08 at 19:23 +0100, Julien Grall wrote: >> The GICv2 DT node is usually used by the guest to know the address/size >> of the regions (GICD, GICC...) to map into their virtual memory. >> >> While the GICv2 spec requires the size of

Re: [Xen-devel] [PATCH v4 0/4] xen/arm: gic-v2: Detect automatically aliased GIC 400

2015-10-23 Thread Ian Campbell
On Thu, 2015-10-08 at 19:23 +0100, Julien Grall wrote: > Hi all, > > Only patch #3 is related to the subject of the cover letter. The rest is > clean > up of code I looked while I was working on this series. Though, patch #1 > is a > new bug fix I noticed between v3 and v4. I fixed the typos I

Re: [Xen-devel] [PATCH v2 1/2] xen/arm: Add support of PSCI v1.0 for the host

2015-10-23 Thread Ian Campbell
On Fri, 2015-10-23 at 11:39 +0100, Julien Grall wrote: > On 23/10/15 11:39, Ian Campbell wrote: > > On Thu, 2015-10-22 at 18:17 +0100, Julien Grall wrote: > > > Hi, > > > > > > On 22/10/15 16:42, Ian Campbell wrote: > > > > On Mon, 2015-10-12 at 16:39 +0100, Julien Grall wrote: > > > > > > > >

Re: [Xen-devel] [PATCH for-4.6] xen/public: arm: Use __typeof__ rather than typeof

2015-10-23 Thread Jan Beulich
>>> On 23.10.15 at 15:30, wrote: > On Fri, 2015-10-23 at 14:13 +0100, Julien Grall wrote: >> Hi, >> >> On 04/10/15 20:24, Julien Grall wrote: >> > The keyword typeof is not portable: >> > >> > /usr/src/freebsd/sys/xen/hypervisor.h:93:2: error: implicit declaration >> >

Re: [Xen-devel] [PATCH v2 3/6] xen: sched: clarify use cases of schedule_cpu_switch()

2015-10-23 Thread Juergen Gross
On 10/21/2015 07:10 PM, Dario Faggioli wrote: On Thu, 2015-10-15 at 10:25 +0200, Juergen Gross wrote: Maybe it would be a good idea to move setting of per_cpu(cpupool, cpu) into schedule_cpu_switch(). Originally I didn't do that to avoid spreading too much cpupool related actions outside of

Re: [Xen-devel] [OSSTEST PATCH 26/26] ts-xen-install: networking: Rename `nodhcp' to `ensurebridge'

2015-10-23 Thread Ian Campbell
On Fri, 2015-10-23 at 13:25 +, Hu, Robert wrote: > > -Original Message- > > From: Ian Campbell [mailto:ian.campb...@citrix.com] > > Sent: Friday, October 23, 2015 4:15 PM > > To: Hu, Robert ; 'Ian Jackson' > > ;

[Xen-devel] [PATCH V3 2/2] xen-netfront: limit vnic max_queues number to online cpus

2015-10-23 Thread Joe Jin
Should not allocate vnic queues number more than online cpus. Signed-off-by: Joe Jin Cc: Jan Beulich Cc: Wei Liu Cc: Ian Campbell Cc: Boris Ostrovsky Cc: Konrad Rzeszutek Wilk

Re: [Xen-devel] [PATCH v4 1/5] xen/arm: vgic-v2: Handle correctly byte write in ITARGETSR

2015-10-23 Thread Julien Grall
On 23/10/15 10:33, Ian Campbell wrote: > On Thu, 2015-10-22 at 17:36 +0100, Julien Grall wrote: >> On 22/10/15 16:53, Ian Campbell wrote: >>> On Mon, 2015-10-12 at 15:22 +0100, Julien Grall wrote: [...] Furthermore, the body of the loop is retrieving the old target list using the

Re: [Xen-devel] [PATCH v2 1/2] xen/arm: Add support of PSCI v1.0 for the host

2015-10-23 Thread Julien Grall
On 23/10/15 11:39, Ian Campbell wrote: > On Thu, 2015-10-22 at 18:17 +0100, Julien Grall wrote: >> Hi, >> >> On 22/10/15 16:42, Ian Campbell wrote: >>> On Mon, 2015-10-12 at 16:39 +0100, Julien Grall wrote: >>> >>> Subject: support PSCI v1.0 for the host. >>> From Xen point of view, PSCI v0.2

Re: [Xen-devel] [PATCH 2/3] xl: convert vcpu related return codes to EXIT_[SUCCESS|FAILURE]

2015-10-23 Thread Dario Faggioli
On Fri, 2015-10-23 at 13:18 +0530, Harmandeep Kaur wrote: > turning vcpu manipulation functions xl exit codes toward using the > EXIT_[SUCCESS|FAILURE] macros, instead of instead of arbitrary > numbers > or libxl return codes. > So, this patch is, technically, mostly fine. The observations on the

Re: [Xen-devel] [PATCH QEMU-XEN v4 3/9] xen: Switch to libxengnttab interface for compat shims.

2015-10-23 Thread Stefano Stabellini
On Wed, 21 Oct 2015, Ian Campbell wrote: > In Xen 4.7 we are refactoring parts libxenctrl into a number of > separate libraries which will provide backward and forward API and ABI > compatiblity. > > One such library will be libxengnttab which provides access to grant > tables. > > In

Re: [Xen-devel] [PATCH QEMU-XEN v4 6/9] xen: Switch uses of xc_map_foreign_bulk to use libxenforeignmemory API.

2015-10-23 Thread Stefano Stabellini
On Wed, 21 Oct 2015, Ian Campbell wrote: > In Xen 4.7 we are refactoring parts libxenctrl into a number of > separate libraries which will provide backward and forward API and ABI > compatiblity. > > One such library will be libxenforeignmemory which provides access to > privileged foreign

Re: [Xen-devel] [PATCH QEMU-XEN v4 2/9] xen: Switch to libxenevtchn interface for compat shims.

2015-10-23 Thread Stefano Stabellini
On Wed, 21 Oct 2015, Ian Campbell wrote: > In Xen 4.7 we are refactoring parts libxenctrl into a number of > separate libraries which will provide backward and forward API and ABI > compatiblity. > > One such library will be libxenevtchn which provides access to event > channels. > > In

[Xen-devel] passthrough: improve interrupt injection locking

2015-10-23 Thread David Vrabel
When injecting an interrupt for a passthrough device into a guest, the per-domain event_lock is held, reducing performance when a guest has many VCPUs and high interrupt rates. By using a per-interrupt lock in the hot paths, this contention is eliminated and performance improves (a bit). For

Re: [Xen-devel] [PATCH] Config.mk: update OVMF changeset

2015-10-23 Thread Ian Jackson
Stefano Stabellini writes ("Re: [PATCH] Config.mk: update OVMF changeset"): > On Fri, 23 Oct 2015, Ian Campbell wrote: > > Yes. Those (that?) and the reasons why we aren't just trivially taking them > > are explained in the referenced thread. That explanation isn't very convincing to me. > I

Re: [Xen-devel] [PATCH QEMU-XEN v4 7/9] xen: Use stable library interfaces when they are available.

2015-10-23 Thread Stefano Stabellini
On Wed, 21 Oct 2015, Ian Campbell wrote: > In Xen 4.7 we are refactoring parts libxenctrl into a number of > separate libraries which will provide backward and forward API and ABI > compatiblity. > > Specifically libxenevtchn, libxengnttab and libxenforeignmemory. > > Previous patches have

Re: [Xen-devel] [PATCH v2 1/2] xen/arm: Add support of PSCI v1.0 for the host

2015-10-23 Thread Ian Campbell
On Thu, 2015-10-22 at 18:17 +0100, Julien Grall wrote: > Hi, > > On 22/10/15 16:42, Ian Campbell wrote: > > On Mon, 2015-10-12 at 16:39 +0100, Julien Grall wrote: > > > > Subject: support PSCI v1.0 for the host. > > > > > From Xen point of view, PSCI v0.2 and PSCI v1.0 are very similar. All > >

Re: [Xen-devel] [PATCH QEMU-XEN v4 9/9] xen: make it possible to build without the Xen PV domain builder

2015-10-23 Thread Stefano Stabellini
On Wed, 21 Oct 2015, Ian Campbell wrote: > Until the previous patch this relied on xc_fd(), which was only > implemented for Xen 4.0 and earlier. > > Given this wasn't working since Xen 4.0 I have marked this as disabled > by default. > > Removing this support drops the use of a bunch of symbols

  1   2   >