Re: [libvirt] [PATCH] revert commit baade4cd2bf partly

2012-01-05 Thread Eric Blake
On 01/04/2012 11:36 PM, Alex Jia wrote: > On 01/05/2012 09:13 AM, Hu Tao wrote: >> On Tue, Jan 03, 2012 at 11:14:15AM +0800, Hu Tao wrote: >>> This is not a memory leak. See line 8029 and 8030 of qemu_driver.c. >>> >>> To ensure this, I tested twice following these steps: >>> >>>1. set bandwidt

Re: [libvirt] [PATCH] revert commit baade4cd2bf partly

2012-01-04 Thread Alex Jia
On 01/05/2012 09:13 AM, Hu Tao wrote: On Tue, Jan 03, 2012 at 11:14:15AM +0800, Hu Tao wrote: This is not a memory leak. See line 8029 and 8030 of qemu_driver.c. To ensure this, I tested twice following these steps: 1. set bandwidth lively (--live) 2. query bandwidth (--live) 3. set b

Re: [libvirt] [PATCH] revert commit baade4cd2bf partly

2012-01-04 Thread Hu Tao
On Tue, Jan 03, 2012 at 11:14:15AM +0800, Hu Tao wrote: > This is not a memory leak. See line 8029 and 8030 of qemu_driver.c. > > To ensure this, I tested twice following these steps: > > 1. set bandwidth lively (--live) > 2. query bandwidth (--live) > 3. set bandwidth lively (--live) > >

[libvirt] [PATCH] revert commit baade4cd2bf partly

2012-01-02 Thread Hu Tao
This is not a memory leak. See line 8029 and 8030 of qemu_driver.c. To ensure this, I tested twice following these steps: 1. set bandwidth lively (--live) 2. query bandwidth (--live) 3. set bandwidth lively (--live) The first time libvirtd crashed at step 2. The second time on step 2 I got