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

Reply via email to