Re: [PATCH 2/3] libxl_driver: Resolve Coverity errors

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Eric Blake wrote:
> On 03/05/2013 05:43 AM, John Ferlan wrote:
>   
>> 1. The virObjectLock() call was unconditional, but Unlock was conditional
>>    on vm being valid.  Removed the check
>>
>> 2. A call to virDomainEventNewFromObj() isn't guaranteed to return an
>>    event - that check needs to be made prior to libxlDomainEventQueue()
>>    of the event. Did not add libxlDriverLock/Unlock around the call since
>>    some callers already have lock taken
>>     
>
> Someday, we should remove the big libxl lock in the same way that danpb
> just removed the big qemu lock; but that's an independent project.
>   

Agreed.  I added that to the libxl todo list when I saw his excellent work.

Jim

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]