[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 03/11] dump-guest-memory: using static DumpSt
From: |
Peter Xu |
Subject: |
Re: [Qemu-devel] [PATCH v4 03/11] dump-guest-memory: using static DumpState, add DumpStatus |
Date: |
Wed, 2 Dec 2015 11:00:27 +0800 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Wed, Dec 02, 2015 at 08:46:49AM +0800, Fam Zheng wrote:
> On Tue, 12/01 21:28, Peter Xu wrote:
> > + if (*errp) {
> > + s->status = DUMP_STATUS_FAILED;
> > + } else {
> > + s->status = DUMP_STATUS_COMPLETED;
> > + }
> > +
>
> To detect error, it's better to use local_err plus error_propagate like a few
> lines above. errp _can_ be NULL depending on callers, though in practice qmp
> functions should get a non-NULL.
Yes, you are right. I will make sure all the error handlings in the
patch set are using local error variables, and call
error_propagate() afterward.
Thanks.
Peter
> > Fam
- [Qemu-devel] [PATCH v4 00/11] Add basic "detach" support for dump-guest-memory, Peter Xu, 2015/12/01
- [Qemu-devel] [PATCH v4 01/11] dump-guest-memory: cleanup: removing dump_{error|cleanup}()., Peter Xu, 2015/12/01
- [Qemu-devel] [PATCH v4 02/11] dump-guest-memory: add "detach" flag for QMP/HMP interfaces., Peter Xu, 2015/12/01
- [Qemu-devel] [PATCH v4 03/11] dump-guest-memory: using static DumpState, add DumpStatus, Peter Xu, 2015/12/01
- [Qemu-devel] [PATCH v4 04/11] dump-guest-memory: add dump_in_progress() helper function, Peter Xu, 2015/12/01
- [Qemu-devel] [PATCH v4 05/11] dump-guest-memory: introduce dump_process() helper function., Peter Xu, 2015/12/01
- [Qemu-devel] [PATCH v4 06/11] dump-guest-memory: disable dump when in INMIGRATE state, Peter Xu, 2015/12/01
- [Qemu-devel] [PATCH v4 07/11] dump-guest-memory: add "detach" support, Peter Xu, 2015/12/01
- [Qemu-devel] [PATCH v4 08/11] dump-guest-memory: add qmp event DUMP_COMPLETED, Peter Xu, 2015/12/01