[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 09/11] DumpState: adding total_size and writt
From: |
Peter Xu |
Subject: |
Re: [Qemu-devel] [PATCH v4 09/11] DumpState: adding total_size and written_size fields |
Date: |
Wed, 2 Dec 2015 18:41:54 +0800 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
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;
> > > > }
> > > > }
> > > > + s->written_size += TARGET_PAGE_SIZE;
> > >
> > > The same question applies here.
> >
> > For kdump case, it is using "goto out" when error happens. So it
> > seems to have no problem here?
>
> write_cache can fail after you increment it here, no?
I am adding it at the end of loop. It looks like:
while (get_next_page(&block_iter, &pfn_iter, &buf, s)) {
if (is_zero_page(buf, TARGET_PAGE_SIZE)) {
ret = write_cache(&page_desc, &pd_zero, sizeof(PageDescriptor),
false);
if (ret < 0) {
error_setg(errp, "XXXXXXXX");
goto out;
}
} else {
...
ret = write_cache(&page_desc, &pd, sizeof(PageDescriptor), false);
if (ret < 0) {
error_setg(errp, "XXXXXXXX");
goto out;
}
...
}
s->written_size += TARGET_PAGE_SIZE;
}
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?
Thanks!
Peter
>
> Fam
[Qemu-devel] [PATCH v4 09/11] DumpState: adding total_size and written_size fields, Peter Xu, 2015/12/01
[Qemu-devel] [PATCH v4 10/11] Dump: add qmp command "query-dump", Peter Xu, 2015/12/01
[Qemu-devel] [PATCH v4 11/11] Dump: add hmp command "info dump", Peter Xu, 2015/12/01