On 14/10/2016 12:33, Fam Zheng wrote: >> > + bdrv_aio_flush(s->bs->file->bs, qed_clear_need_check, s); > If this one has to change, what about the other bdrv_aio_flush(s->bs, ...) > down > in this call path: > > qed_need_check_timer_cb > qed_clear_need_check > qed_write_header > qed_flush_after_clear_need_check
It was really just for clarity, so I guess I can change all four of them in a separate patch. Paolo