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

Re: [libvirt] [PATCH 1/8] Fix up the locking in the snapshot code.



On 04/27/2010 08:26 AM, Daniel Veillard wrote:
> On Fri, Apr 23, 2010 at 01:27:45PM -0400, Chris Lalancette wrote:
>> In particular I was forgetting to take the qemuMonitorPrivatePtr
>> lock (via qemuDomainObjBeginJob), which would cause problems
>> if two users tried to access the same domain at the same time.
>> This patch also fixes a problem where I was forgetting to remove
>> a transient domain from the list of domains.
>>
>> Thanks to Stephen Shaw for pointing out the problem and testing
>> out the initial patch.
>>
>> Signed-off-by: Chris Lalancette <clalance redhat com>

<snip>

> 
>   ACK, looks fine to me,

Thanks, I pushed this now.

-- 
Chris Lalancette


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