On 16/06/2016 08:58, Denis V. Lunev wrote: > Actually I have found this problem running iotest 132 for active async > mirror I have sent yesturday. Anyway, the problem is actual for current > backup/mirror implementation. > > bdrv_co_discard must mark areas dirty after writing zeroes, it must call > before_write_notifier chain to push underlying data to backup and it also > must properly fill tracked request information. > > Signed-off-by: Denis V. Lunev <[email protected]> > CC: Vladimir Sementsov-Ogievskiy<[email protected]> > CC: Stefan Hajnoczi <[email protected]> > CC: Fam Zheng <[email protected]> > CC: Kevin Wolf <[email protected]> > CC: Max Reitz <[email protected]>
Three good catches! Thanks for these patches. I think 2 and 3 should be Cced to qemu-stable too. Reviewed-by: Paolo Bonzini <[email protected]> Paolo
