On Wed, Nov 04, 2015 at 02:31:55PM +0300, Denis V. Lunev wrote: > The patch enforces proper locking for this operation. > > Signed-off-by: Denis V. Lunev <d...@openvz.org> > CC: Stefan Hajnoczi <stefa...@redhat.com> > CC: Juan Quintela <quint...@redhat.com> > --- > Patches are compile-tested only. Sent to check the approach, naming and > functions placement. Functions are returning bad BlockDriver via > parameter to make clear distinction when I'll have to return BS to write. > > block/snapshot.c | 35 +++++++++++++++++++++++++++++++++++ > include/block/snapshot.h | 9 +++++++++ > migration/savevm.c | 17 ++++------------- > 3 files changed, 48 insertions(+), 13 deletions(-)
Kevin is the maintainer of block/snapshot.c. Please include him in future revisions. Looks fine to me, besides the comments that Juan already raised. Stefan
