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(a)redhat.com>
<snip>
ACK, looks fine to me,
Thanks, I pushed this now.
--
Chris Lalancette