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 <d...@openvz.org> > CC: Vladimir Sementsov-Ogievskiy<vsement...@virtuozzo.com> > CC: Stefan Hajnoczi <stefa...@redhat.com> > CC: Fam Zheng <f...@redhat.com> > CC: Kevin Wolf <kw...@redhat.com> > CC: Max Reitz <mre...@redhat.com>
Three good catches! Thanks for these patches. I think 2 and 3 should be Cced to qemu-stable too. Reviewed-by: Paolo Bonzini <pbonz...@redhat.com> Paolo