[libvirt] [PATCH] openvz: fixed two memory leaks on migration code

2014-09-15 Thread Hongbin Lu
--- src/openvz/openvz_driver.c |8 ++-- 1 files changed, 6 insertions(+), 2 deletions(-) diff --git a/src/openvz/openvz_driver.c b/src/openvz/openvz_driver.c index 57b3c22..3147311 100644 --- a/src/openvz/openvz_driver.c +++ b/src/openvz/openvz_driver.c @@ -2364,7 +2364,10 @@

Re: [libvirt] [PATCH] add migration support for OpenVZ driver

2014-09-15 Thread Hongbin Lu
Hi John, Thanks for pointing that out. A fix was proposed in here https://www.redhat.com/archives/libvir-list/2014-September/msg01000.html. Best regards, Hongbin On Mon, Sep 15, 2014 at 11:46 AM, John Ferlan jfer...@redhat.com wrote: On 09/04/2014 10:25 PM, Hongbin Lu wrote: This patch

[libvirt] [PATCH] openvz: fixed an config file parsing error

2014-09-06 Thread Hongbin Lu
The OpenVZ driver reported an error on parsing some OpenVZ config parameters (e.g. diskspace). This issue is due to the driver made two incorrect assumptions about the value of the parameters: 1. Assume paramaeter is just a number (without unit suffix). 2. Assume the number is an integer.

Re: [libvirt] [PATCH] openvz: fixed an config file parsing error

2014-09-06 Thread Hongbin Lu
Thanks Eric. I modified the fix accordingly. https://www.redhat.com/archives/libvir-list/2014-September/msg00363.html Best regards, Hongbin On Wed, Sep 3, 2014 at 12:19 PM, Eric Blake ebl...@redhat.com wrote: On 09/03/2014 09:55 AM, Hongbin Lu wrote: The OpenVZ driver reported an error

Re: [libvirt] [PATCH] add migration support for OpenVZ driver

2014-09-04 Thread Hongbin Lu
It was resent in here https://www.redhat.com/archives/libvir-list/2014-September/msg00179.html. Best regards, Hongbin On Thu, Sep 4, 2014 at 7:56 AM, Michal Privoznik mpriv...@redhat.com wrote: On 03.09.2014 14:41, Hongbin Lu wrote: Resent the email below. Thanks. This is rather unusual

[libvirt] [PATCH] add migration support for OpenVZ driver

2014-09-04 Thread Hongbin Lu
This patch adds initial migration support to the OpenVZ driver, using the VIR_DRV_FEATURE_MIGRATION_PARAMS family of migration functions. --- src/openvz/openvz_conf.h |5 +- src/openvz/openvz_driver.c | 348 src/openvz/openvz_driver.h | 10 ++

Re: [libvirt] [PATCH] add migration support for OpenVZ driver

2014-09-04 Thread Hongbin Lu
Thanks Guido, Your comment is addressed: https://www.redhat.com/archives/libvir-list/2014-September/msg00284.html. Best regards, Hongbin On Thu, Sep 4, 2014 at 1:42 AM, Guido Günther a...@sigxcpu.org wrote: Hi, On Wed, Sep 03, 2014 at 11:07:20PM -0400, Hongbin Lu wrote: [..snip

Re: [libvirt] [PATCH] add migration support for OpenVZ driver

2014-09-03 Thread Hongbin Lu
Resent the email below. Thanks. On Sun, Aug 31, 2014 at 2:05 PM, Hongbin Lu hongbin...@gmail.com wrote: This patch adds initial migration support to the OpenVZ driver, using the VIR_DRV_FEATURE_MIGRATION_PARAMS family of migration functions. --- src/openvz/openvz_conf.h |5 +- src

Re: [libvirt] [PATCH] openvz: fixed an config file parsing error

2014-09-03 Thread Hongbin Lu
Resend the email below. On Mon, Sep 1, 2014 at 8:28 PM, Hongbin Lu hongbin...@gmail.com wrote: The OpenVZ driver reported an error on parsing some OpenVZ config parameters (e.g. diskspace). This issue is due to the driver made two incorrect assumptions about the value of the parameters: 1

[libvirt] [PATCH] add migration support for OpenVZ driver

2014-09-03 Thread Hongbin Lu
This patch adds initial migration support to the OpenVZ driver, using the VIR_DRV_FEATURE_MIGRATION_PARAMS family of migration functions. --- src/openvz/openvz_conf.h |5 +- src/openvz/openvz_driver.c | 348 src/openvz/openvz_driver.h | 10 ++