Re: [Qemu-devel] [PATCH V7 0/8] add internal snapshot support at block device level

[Wenchao Xia]

于 2013-8-16 23:36, Stefan Hajnoczi 写道:

On Wed, Aug 07, 2013 at 11:00:11AM +0800, Wenchao Xia wrote:

v6:
   Address Stefan's comments:
   2/8: macro STR_PRINT_CHAR was renamed as STR_OR_NULL, and moved into patch 5,
since implement function in this patch do not printf snapshot id any more, as
Kevin's suggestion.

Great.

   Address Kevin's comments:
   2/8: remove device, id, name info in the error message, use error message in
existing caller. A new function bdrv_snapshot_delete_by_id_or_name() is added
to make the usage clear while keep logic unchanged.
   3/8: remove device info in error message when name is empty. Use else if
after call of bdrv_snapshot_find_by_id_and_name().
   Other:
   2/8: refined the comments in code for bdrv_snapshot_delete().
   3/8: in error reporting, change format from "reason is: '%s'" to
"reason is: %s".

Kevin: do you have time to review these changes?

Stefan

Hi, Kevin
  Do you have comments for it?

--
Best Regards

Wenchao Xia