On Wed, 04/19 18:38, Paolo Bonzini wrote: > > > On 19/04/2017 11:17, Fam Zheng wrote: > > Mirror calculates job len from current I/O progress: > > > > s->common.len = s->common.offset + > > (cnt + s->sectors_in_flight) * BDRV_SECTOR_SIZE; > > > > The final "len" of a failed mirror job in iotests 109 depends on the > > subtle timing of the completion of read and write issued in the first > > mirror iteration. The second iteration may or may not have run when the > > I/O error happens, resulting in non-deterministic output of the > > BLOCK_JOB_COMPLETED event text. > > > > Similar to what was done in a752e4786, filter out the field to make the > > test robust. > > There are more occurrences:
Yes. I'll convert them too.. Fam