Re: [Qemu-devel] [PATCH v4 09/11] DumpState: adding total_size and written_size fields

2015-12-02 Thread Peter Xu
On Wed, Dec 02, 2015 at 08:51:48PM +0800, Fam Zheng wrote: > On Wed, 12/02 18:41, Peter Xu wrote: > > On Wed, Dec 02, 2015 at 05:49:18PM +0800, Fam Zheng wrote: > > Label "out" is out of the loop. So, when error happens, it sets the > > errp and directly jump out of the loop. Did I miss anything? >

Re: [Qemu-devel] [PATCH v4 09/11] DumpState: adding total_size and written_size fields

2015-12-02 Thread Fam Zheng
On Wed, 12/02 18:41, Peter Xu wrote: > On Wed, Dec 02, 2015 at 05:49:18PM +0800, Fam Zheng wrote: > > On Wed, 12/02 16:49, Peter Xu wrote: > > > On Wed, Dec 02, 2015 at 09:32:57AM +0800, Fam Zheng wrote: > > > > > @@ -1301,6 +1303,7 @@ static void write_dump_pages(DumpState *s, > > > > > Error **e

Re: [Qemu-devel] [PATCH v4 09/11] DumpState: adding total_size and written_size fields

2015-12-02 Thread Peter Xu
On Wed, Dec 02, 2015 at 05:49:18PM +0800, Fam Zheng wrote: > On Wed, 12/02 16:49, Peter Xu wrote: > > On Wed, Dec 02, 2015 at 09:32:57AM +0800, Fam Zheng wrote: > > > > @@ -1301,6 +1303,7 @@ static void write_dump_pages(DumpState *s, Error > > > > **errp) > > > > goto out; > > > >

Re: [Qemu-devel] [PATCH v4 09/11] DumpState: adding total_size and written_size fields

2015-12-02 Thread Fam Zheng
On Wed, 12/02 16:49, Peter Xu wrote: > On Wed, Dec 02, 2015 at 09:32:57AM +0800, Fam Zheng wrote: > > > @@ -1301,6 +1303,7 @@ static void write_dump_pages(DumpState *s, Error > > > **errp) > > > goto out; > > > } > > > } > > > +s->written_size += TARG

Re: [Qemu-devel] [PATCH v4 09/11] DumpState: adding total_size and written_size fields

2015-12-02 Thread Peter Xu
On Wed, Dec 02, 2015 at 09:32:57AM +0800, Fam Zheng wrote: > On Tue, 12/01 21:28, Peter Xu wrote: > > @@ -333,6 +333,8 @@ static void write_data(DumpState *s, void *buf, int > > length, Error **errp) > > if (ret < 0) { > > error_setg(errp, "dump: failed to save memory"); > > } >

Re: [Qemu-devel] [PATCH v4 09/11] DumpState: adding total_size and written_size fields

2015-12-01 Thread Fam Zheng
On Tue, 12/01 21:28, Peter Xu wrote: > Here, total_size is the size in bytes to be dumped (raw data, which > means before compression), while written_size are bytes handled (raw > size too). > > Signed-off-by: Peter Xu > --- > dump.c| 32 > includ

[Qemu-devel] [PATCH v4 09/11] DumpState: adding total_size and written_size fields

2015-12-01 Thread Peter Xu
Here, total_size is the size in bytes to be dumped (raw data, which means before compression), while written_size are bytes handled (raw size too). Signed-off-by: Peter Xu --- dump.c| 32 include/sysemu/dump.h | 9 + 2 files changed, 41 i