[libvirt] Availability of 0.9.4 candidate release 2 rc1
ajia
ajia at redhat.com
Mon Aug 1 14:09:59 UTC 2011
On 08/01/2011 10:05 PM, ajia wrote:
> On 08/01/2011 08:38 PM, Guannan Ren wrote:
>> On 08/01/2011 05:04 PM, Daniel Veillard wrote:
>>> On Mon, Aug 01, 2011 at 04:51:38PM +0800, Wen Congyang wrote:
>>>> At 07/30/2011 03:02 PM, Daniel Veillard Write:
>>>>> I actually tagged and pushed the rc2 tarball and rpms yesterday
>>>>> but completely forgot to send the associated mail, oops !
>>>>>
>>>>> ftp://libvirt.org/libvirt/libvirt-0.9.4-rc2.tar.gz
>>>>>
>>>>> Hopefully it fixes most of the problems raised with rc1, including
>>>>> a number of leaks. Please report and if you had an issue with rc1
>>>>> which is still not fixed there (or in git) please raise it ASAP.
>>>>> I'm planning for the final release early Tuesday 2 morning (i.e.
>>>>> late Monday for most :-)
>>>> If client(for example: virsh) exits unexpectedly, it will cause
>>>> libvirtd
>>>> crashed.
>>>>
>>>> Steps to reproduce this problem(vm1 does not run):
>>>> 1. for ((i=0; i< 50; i++)); do virsh managedsave vm1& done;
>>>> killall virsh
>>>>
>>>> The reason is that we free virNetServerClient when the refs is not 0.
>>>>
>>>> I read the code under the directory src/rpc/, and find we have
>>>> xxxRef(), but
>>>> we do not have xxxUnref(). And sometimes we free the data structure
>>>> if ref is
>>>> not 0. We add an reference of the data structure, but sometimes we
>>>> forget to
>>>> unref it.
>>> Okay, so I count 3 issues we should fix before pushing 0.9.4 out
>>> - this one (can you open a bugzilla for tracking), are you gonna
>>> provide a patch ? Dan Berrange won't be around today
>>> - fix crash when mixing sync and async monitor jobs, we are waiting
>>> on Eric's finding of his v3 version
>>> - domabort seems broken in rc2
>>> https://bugzilla.redhat.com/show_bug.cgi?id=727047
>>>
>>> thanks for the heads-up !
>>>
>>> Daniel
>>>
>> There is another issues needed to be fixed before pushing
>> 0.9.4
>> The "lock_manager " in qemu.conf still uses old default value .
>> When it is uncommented, it will lead to qemu driver
>> initialization failure:
>>
>> 20:47:45.227: 23830: error : virLockManagerPluginNew:147 :
>> Plugin /usr/lib64/libvirt/lock-driver/fcntl.so not accessible: No
>> such file or directory
>> 20:47:45.227: 23830: error : qemudLoadDriverConfig:457 :
>> Failed to load lock manager fcntl
>> 20:47:45.227: 23830: error : qemudStartup:540 : Missing lock
>> manager implementation
>> ...
>> 20:47:45.227: 23830: error : virStateInitialize:846 :
>> Initialization of QEMU state driver failed
> I have ever discussed this issue with danpb, the above errror is a
> expected result, because libvirt indeed
> hasn't implemented 'fcntl' lock at present, in addition, the biggest
> issue is if we change 'fcntl' to 'sanlock' in qemu.conf then restart
> libvirtd service, virsh command will be hung, danpb has fixed this
> issue in recent, and commit id is
> 92509413e2da3260b99177de82c85a59cf8ddfd9, so IMHO, this is not a big
> deal, here is a known bug about this issue.
https://bugzilla.redhat.com/show_bug.cgi?id=723967
>
> Regards,
> Alex
>>
>> I proposed a patch that needs an ACK:
>>
>> https://www.redhat.com/archives/libvir-list/2011-July/msg02097.html
>>
>> --
>> libvir-list mailing list
>> libvir-list at redhat.com
>> https://www.redhat.com/mailman/listinfo/libvir-list
>
> --
> libvir-list mailing list
> libvir-list at redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list
More information about the libvir-list
mailing list