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

2010-04-27 Thread Daniel Veillard
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

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

2010-04-27 Thread Chris Lalancette
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

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

2010-04-23 Thread Chris Lalancette
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.