Re: [Xen-devel] [libvirt] [PATCH V2 1/3] xenconfig: support vif bandwidth in sexpr parser and formatter

2016-01-12 Thread John Ferlan
On 01/04/2016 08:08 PM, Jim Fehlig wrote: > The xen sexpr config format has long supported specifying vif rate > limiting, e.g. > > (device > (vif > (mac '00:16:3e:1b:b1:47') > (rate '10240KB/s') > ... > ) > ) > > Add support for mapping rate to and from in the xe

Re: [Xen-devel] [libvirt] [PATCH V2] libxl: don't overwrite error from virNetSocketNewConnectTCP()

2015-09-03 Thread John Ferlan
On 09/03/2015 01:40 PM, Jim Fehlig wrote: > Remove redundant error reporting in libxlDomainMigrationPerform(). > virNetSocketNewConnectTCP() is perfectly capable of reporting > sensible errors. > > Signed-off-by: Jim Fehlig > --- > > V2: > Actually try to compile the code and find saved_errno

Re: [Xen-devel] [libvirt] [PATCH V4 0/3] Parser for xen-xl config format

2015-01-13 Thread John Ferlan
On 01/13/2015 10:53 AM, Jim Fehlig wrote: > It's been a long, twisting road to V4 of the Xen xl parser. V3 [1] was > based on a flex-based parser that was copied from the Xen project and > proved to be a bit challenging to integrate properly with autotools. > But as it turns out, Xen provides an

Re: [Xen-devel] [libvirt] [PATCH 10/12] Introduce support for parsing/formatting Xen xl config format

2015-01-12 Thread John Ferlan
On 01/10/2015 12:03 AM, Jim Fehlig wrote: > Introduce a parser/formatter for the xl config format. Since the > deprecation of xm/xend, the VM config file format has diverged as > new features are added to libxl. This patch adds support for parsing > and formating the xl config format. It suppo

Re: [Xen-devel] [libvirt] [PATCH 09/12] Revert "src/xenconfig: Xen-xl parser"

2015-01-12 Thread John Ferlan
On 01/10/2015 12:03 AM, Jim Fehlig wrote: > This reverts commit 2c78051a14acfb7aba078d569b1632dfe0ca0853. > > Conflicts: > src/Makefile.am > > Signed-off-by: Jim Fehlig > --- > .gitignore| 1 - > cfg.mk| 3 +- > configure.ac

Re: [Xen-devel] [libvirt] [PATCH] xenconfig: fix boot device parsing

2014-12-15 Thread John Ferlan
On 12/14/2014 09:58 AM, Wei Liu wrote: > The original code always checked *boot which was in effect boot[0]. It > should use boot[i]. In the future, when you do the investigation, just reference the commit id that caused the regression. In this case it seems to be commit id '547bd71a' (7/25/08