Re: [libvirt] [Xen-devel] [PATCH] libxl: fix dom0 balloon logic

2015-03-24 Thread Stefano Stabellini
On Mon, 23 Mar 2015, Ian Campbell wrote: (just ccing the other tools maintainers, in particular Stefano who knows what this stuff is supposed to do...) On Fri, 2015-03-20 at 17:10 -0600, Jim Fehlig wrote: Recent testing on large memory systems revealed a bug in the Xen xl tool's freemem()

Re: [libvirt] [PATCH V2] libxl: fix dom0 balloon logic

2015-03-24 Thread Stefano Stabellini
. It was a bit cleaner IMO and will make it easier to spot future, potentially interesting divergences. Signed-off-by: Jim Fehlig jfeh...@suse.com Reviewed-by: Stefano Stabellini stefano.stabell...@eu.citrix.com V2: Actually use libxl_wait_for_memory_target(), instead

Re: [libvirt] [Xen-devel] [PATCH] libxl: prefer qdisk for driver name='file'

2014-06-21 Thread Stefano Stabellini
jfeh...@suse.com Acked-by: Stefano Stabellini stefano.stabell...@eu.citrix.com src/libxl/libxl_conf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/libxl/libxl_conf.c b/src/libxl/libxl_conf.c index fdbb522..4b6b5c0 100644 --- a/src/libxl/libxl_conf.c +++ b/src

Re: [libvirt] [Xen-devel] [PATCH LIBVIRT] libxl: Recognise ARM architectures

2014-02-28 Thread Stefano Stabellini
On Fri, 28 Feb 2014, Daniel P. Berrange wrote: On Fri, Feb 28, 2014 at 02:12:30AM +, Ian Campbell wrote: On Wed, 2014-02-26 at 15:01 +, Daniel P. Berrange wrote: Yep, if ARM has a PV console, then we'd need to add tiny bit to the XML to allow us to configure that explicitly,

Re: [libvirt] [Xen-devel] [PATCH RESENT 04/12] libxl: populate xenstore memory entries at startup\

2013-05-22 Thread Stefano Stabellini
On Wed, 22 May 2013, Jim Fehlig wrote: Marek Marczykowski wrote: On 19.04.2013 13:10, Stefano Stabellini wrote: On Thu, 11 Apr 2013, Marek Marczykowski wrote: On 11.04.2013 09:52, Ian Campbell wrote: On Thu, 2013-04-11 at 05:09 +0100, Jim Fehlig wrote

Re: [libvirt] [Xen-devel] [PATCH RESENT 04/12] libxl: populate xenstore memory entries at startup

2013-04-19 Thread Stefano Stabellini
On Thu, 11 Apr 2013, Ian Campbell wrote: On Thu, 2013-04-11 at 05:09 +0100, Jim Fehlig wrote: +/* This will fill xenstore info about free and dom0 memory - if missing, + * should be called before starting first domain */ +if (libxl_get_free_memory(libxl_driver-ctx,

Re: [libvirt] [Xen-devel] [PATCH RESENT 04/12] libxl: populate xenstore memory entries at startup

2013-04-19 Thread Stefano Stabellini
On Thu, 11 Apr 2013, Marek Marczykowski wrote: On 11.04.2013 09:52, Ian Campbell wrote: On Thu, 2013-04-11 at 05:09 +0100, Jim Fehlig wrote: +/* This will fill xenstore info about free and dom0 memory - if missing, + * should be called before starting first domain */ +if

Re: [libvirt] [Xen-devel] [PATCH RESENT 04/12] libxl: populate xenstore memory entries at startup

2013-04-19 Thread Stefano Stabellini
On Fri, 19 Apr 2013, Marek Marczykowski wrote: And to answer you question - libvirt rely on libxl autoballoon. Could we introduce something similar to autoballoon=auto to libvirt? Maybe we should push down the autoballoon option to libxl: we should probably rename autoballoon to

Re: [libvirt] [Xen-devel] [PATCH] Convert libxl driver to Xen 4.2

2012-11-27 Thread Stefano Stabellini
On Tue, 27 Nov 2012, George Dunlap wrote: On Mon, Nov 26, 2012 at 10:28 PM, Jim Fehlig jfeh...@suse.com wrote: Based on a patch originally authored by Daniel De Graaf   http://lists.xen.org/archives/html/xen-devel/2012-05/msg00565.html This patch converts the Xen libxl

Re: [libvirt] [PATCH V5] Add libxenlight driver

2011-03-11 Thread Stefano Stabellini
On Fri, 11 Mar 2011, Jim Fehlig wrote: Daniel Veillard wrote: Okay, I tried t make sure libxl_domain_shutdown() is really asynchronous in all cases, I somehow failed, can you confirm ? Yes, AFAIK that is the case. Stefano, is that true? Yes, libxl_domain_shutdown is

Re: [libvirt] [Xen-devel] [PATCH V2] Add libxenlight driver

2011-02-25 Thread Stefano Stabellini
) - Rebased to current master - Plug memory leaks found by Stefano Stabellini and valgrind - Handle SHUTDOWN_crash domain death event It looks good to me. +if (l_disks[i]-driverName) { +if (STREQ(l_disks[i]-driverName, tap) || +STREQ(l_disks[i]-driverName

Re: [libvirt] [Xen-devel] [PATCH] libxenlight driver

2011-02-18 Thread Stefano Stabellini
On Fri, 18 Feb 2011, Jim Fehlig wrote: +static int +libxlMakeDiskList(virDomainDefPtr def, libxl_domain_config *d_config) +{ +virDomainDiskDefPtr *l_disks = def-disks; +int ndisks = def-ndisks; +libxl_device_disk *x_disks; +int i; + +if (VIR_ALLOC_N(x_disks, ndisks)

Re: [libvirt] [Xen-devel] [PATCH] libxenlight driver

2011-02-14 Thread Stefano Stabellini
On Mon, 14 Feb 2011, Jim Fehlig wrote: Hi All, Here's a first cut of libxenlight driver for libvirt. The driver is stateful and provides functionality for managed (persistent) domains. The driver only maintains state for and manages domains under its control, ignoring domains created by

Re: [libvirt] [Xen-devel] Re: [RFC] libxenlight driver

2011-01-21 Thread Stefano Stabellini
On Fri, 21 Jan 2011, Jim Fehlig wrote: Daniel P. Berrange wrote: On Thu, Jan 20, 2011 at 04:49:25PM -0700, Jim Fehlig wrote: I'm looking into creating a driver for the new Xen xl/libxl toolstack (aka libxenlight [1]), set to become the default in upcoming Xen 4.1.0 release. My

Re: [libvirt] [Xen-devel] Re: [RFC] libxenlight driver

2011-01-21 Thread Stefano Stabellini
On Fri, 21 Jan 2011, Daniel P. Berrange wrote: Ok, so basically the mgmt app that is using the libxenlight API is taking the place of XenD. Thus in the new scheme libvirtd would have to handle those kind of events. Right. Libxenlight provides all the functions needed to do that and you can