Re: [Qemu-devel] [PATCH v2 6/6] migration: Route more error paths

2017-09-27 Thread Juan Quintela
"Dr. David Alan Gilbert (git)" wrote: > From: "Dr. David Alan Gilbert" > > vmstate_save is called in a few places, and vmstate_save_state is > called in lots of places. > > Route error returns from the easier cases back up; there are lots > of more

Re: [Qemu-devel] [PATCH v2 6/6] migration: Route more error paths

2017-09-26 Thread Cornelia Huck
On Mon, 25 Sep 2017 12:29:17 +0100 "Dr. David Alan Gilbert (git)" wrote: > From: "Dr. David Alan Gilbert" > > vmstate_save is called in a few places, and vmstate_save_state is > called in lots of places. > > Route error returns from the easier cases

Re: [Qemu-devel] [PATCH v2 6/6] migration: Route more error paths

2017-09-26 Thread Dr. David Alan Gilbert
* Peter Xu (pet...@redhat.com) wrote: > On Mon, Sep 25, 2017 at 12:29:17PM +0100, Dr. David Alan Gilbert (git) wrote: > > From: "Dr. David Alan Gilbert" > > > > vmstate_save is called in a few places, and vmstate_save_state is > > called in lots of places. > >

Re: [Qemu-devel] [PATCH v2 6/6] migration: Route more error paths

2017-09-25 Thread Peter Xu
On Mon, Sep 25, 2017 at 12:29:17PM +0100, Dr. David Alan Gilbert (git) wrote: > From: "Dr. David Alan Gilbert" > > vmstate_save is called in a few places, and vmstate_save_state is > called in lots of places. vmstate_save() should have been processed with previous patch? >

[Qemu-devel] [PATCH v2 6/6] migration: Route more error paths

2017-09-25 Thread Dr. David Alan Gilbert (git)
From: "Dr. David Alan Gilbert" vmstate_save is called in a few places, and vmstate_save_state is called in lots of places. Route error returns from the easier cases back up; there are lots of more complex cases where their own error paths need fixing. Signed-off-by: Dr.