Re: [Qemu-devel] [PATCH 24/51] ram: Move migration_bitmap_mutex into RAMState

2017-03-30 Thread Dr. David Alan Gilbert
* Juan Quintela (quint...@redhat.com) wrote: > Signed-off-by: Juan Quintela I'm still pretty convinced that there's an existing problem with this mutex can get init'd twice with no destroy on a second migration, however you're not changing that (and it's not actually failed as far as I can tell)

Re: [Qemu-devel] [PATCH 24/51] ram: Move migration_bitmap_mutex into RAMState

2017-03-29 Thread Peter Xu
On Thu, Mar 23, 2017 at 09:45:17PM +0100, Juan Quintela wrote: > Signed-off-by: Juan Quintela Reviewed-by: Peter Xu

[Qemu-devel] [PATCH 24/51] ram: Move migration_bitmap_mutex into RAMState

2017-03-23 Thread Juan Quintela
Signed-off-by: Juan Quintela --- migration/ram.c | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/migration/ram.c b/migration/ram.c index a890179..ae2b89f 100644 --- a/migration/ram.c +++ b/migration/ram.c @@ -184,6 +184,8 @@ struct RAMState { uint64_t xbzr