|
From: | Juan Quintela |
Subject: | Re: [Qemu-devel] [PATCH 03/11] snapshot: create bdrv_all_delete_snapshot helper |
Date: | Wed, 18 Nov 2015 12:01:05 +0100 |
User-agent: | Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
"Denis V. Lunev" <address@hidden> wrote: > to delete snapshots from all loaded block drivers. > > The patch also ensures proper locking. > > Signed-off-by: Denis V. Lunev <address@hidden> > Reviewed-by: Stefan Hajnoczi <address@hidden> > Reviewed-by: Fam Zheng <address@hidden> > CC: Juan Quintela <address@hidden> > CC: Kevin Wolf <address@hidden> Reviewed-by: Juan Quintela <address@hidden> I will still suggest to rename it to brdv_delete_all_snapshots() or bdrv_delete_snapshot_everywhere() that makes is more clear what it does. A big THANKS for removing this from the migration side of the equation O:-) Later, Juan.
[Prev in Thread] | Current Thread | [Next in Thread] |