Re: [Qemu-devel] [PATCH 1/3] migration/savevm: flush file for iterable_only case

2019-07-19 Thread Wei Yang
On Fri, Jul 19, 2019 at 05:47:59PM +0100, Dr. David Alan Gilbert wrote: >* Wei Yang (richardw.y...@linux.intel.com) wrote: >> It would be proper to flush file even for iterable_only case. >> >> Signed-off-by: Wei Yang > >OK, I don't think this is actually necessary; but it's safe. >We only really

Re: [Qemu-devel] [PATCH 1/3] migration/savevm: flush file for iterable_only case

2019-07-19 Thread Dr. David Alan Gilbert
* Wei Yang (richardw.y...@linux.intel.com) wrote: > It would be proper to flush file even for iterable_only case. > > Signed-off-by: Wei Yang OK, I don't think this is actually necessary; but it's safe. We only really need the flush at the end of the file, and in the non-iterable-only case it's

[Qemu-devel] [PATCH 1/3] migration/savevm: flush file for iterable_only case

2019-07-09 Thread Wei Yang
It would be proper to flush file even for iterable_only case. Signed-off-by: Wei Yang --- migration/savevm.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/migration/savevm.c b/migration/savevm.c index c0e557b4c2..becedcc1c6 100644 --- a/migration/savevm.c +++ b/migration/