Re: [Qemu-devel] [PATCH 6/6] Don't abort on memory allocation error

2014-02-04 Thread Juan Quintela
Orit Wasserman wrote: > It is better to fail migration in case of failure to > allocate new cache item > > Signed-off-by: Orit Wasserman Reviewed-by: Juan Quintela

Re: [Qemu-devel] [PATCH 6/6] Don't abort on memory allocation error

2014-01-30 Thread Orit Wasserman
On 01/30/2014 06:08 PM, Dr. David Alan Gilbert wrote: * Orit Wasserman (owass...@redhat.com) wrote: It is better to fail migration in case of failure to allocate new cache item Does this actually fail migration or just drop back to sending an uncompressed page? (I think the latter, and that's

Re: [Qemu-devel] [PATCH 6/6] Don't abort on memory allocation error

2014-01-30 Thread Dr. David Alan Gilbert
* Orit Wasserman (owass...@redhat.com) wrote: > It is better to fail migration in case of failure to > allocate new cache item Does this actually fail migration or just drop back to sending an uncompressed page? (I think the latter, and that's an even better result). > Signed-off-by: Orit Wasserm

[Qemu-devel] [PATCH 6/6] Don't abort on memory allocation error

2014-01-29 Thread Orit Wasserman
It is better to fail migration in case of failure to allocate new cache item Signed-off-by: Orit Wasserman --- arch_init.c| 4 +++- include/migration/page_cache.h | 4 +++- page_cache.c | 16 +++- 3 files changed, 17 insertions(+), 7 deletions(