
6 Mar
2013
6 Mar
'13
3:50 a.m.
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