qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH V7 09/14] block: move bdrv_snapshot_find() to bl


From: Wenchao Xia
Subject: Re: [Qemu-devel] [PATCH V7 09/14] block: move bdrv_snapshot_find() to block.c
Date: Mon, 04 Mar 2013 11:20:49 +0800
User-agent: Mozilla/5.0 (Windows NT 5.1; rv:17.0) Gecko/20130215 Thunderbird/17.0.3

于 2013-3-1 9:51, Wenchao Xia 写道:
于 2013-2-28 0:22, Kevin Wolf 写道:
Am 27.02.2013 um 17:04 hat Markus Armbruster geschrieben:
Wenchao Xia <address@hidden> writes:

Signed-off-by: Wenchao Xia <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
---
  block.c               |   24 ++++++++++++++++++++++++
  include/block/block.h |    2 ++
  savevm.c              |   22 ----------------------
  3 files changed, 26 insertions(+), 22 deletions(-)

Perhaps savevm.c isn't the best home for snapshot stuff, but I don't
like it in block.c any better.  block.c is already unwieldy, and I'd
like us to try limiting it to core block stuff.

Kevin, Stefan, any ideas?

Take some more snapshot related functions from block.c and move them to
block/snapshot.c?

Kevin

Hi, Stefan
   Do you also agree about adding new file block/snapshot.c?


Hi,
  I am going to add include/block/snapshot.h and block/snapshot.c,
all internal snapshot function in block.c will be moved there.
external backing file related function will not be touched and
remain in block.c. This consume many effort and once done it will
be hard to revert back to form an incremental commit in this
serials, so before coding, please give your opinion if u don't agree
about it.

--
Best Regards

Wenchao Xia




reply via email to

[Prev in Thread] Current Thread [Next in Thread]