Re: [Qemu-devel] [PATCH 1/5] ram: Call migration_page_queue_free() at ram_migration_cleanup()

2017-06-06 Thread Peter Xu
On Fri, Jun 02, 2017 at 12:08:09AM +0200, Juan Quintela wrote: > We shouldn't be using memory later than that. > > Signed-off-by: Juan Quintela Reviewed-by: Peter Xu > --- > migration/migration.c | 2 -- > migration/ram.c | 5 +++-- >

Re: [Qemu-devel] [PATCH 1/5] ram: Call migration_page_queue_free() at ram_migration_cleanup()

2017-06-05 Thread Dr. David Alan Gilbert
* Juan Quintela (quint...@redhat.com) wrote: > We shouldn't be using memory later than that. > > Signed-off-by: Juan Quintela Yes, I think I agree: migration_completion calls await_return_path_close_on_source that makes sure there's no more incoming requests.

[Qemu-devel] [PATCH 1/5] ram: Call migration_page_queue_free() at ram_migration_cleanup()

2017-06-01 Thread Juan Quintela
We shouldn't be using memory later than that. Signed-off-by: Juan Quintela --- migration/migration.c | 2 -- migration/ram.c | 5 +++-- migration/ram.h | 1 - 3 files changed, 3 insertions(+), 5 deletions(-) diff --git a/migration/migration.c