Il 13/03/2013 10:31, Kevin Wolf ha scritto:
> Isn't the real problem that I/O requests for _this_specific_ iscsi BDS
> must not be in flight? So what you reall need is bdrv_drain(iscsi_bs)?
>
> If I understand the code correctly, this boils down to:
>
> while (iscsi_process_flush(iscsilun)) {
> qemu_aio_wait();
> }
Yes, but this function is not doing a truncate at all. It is simply
updating the cached result of bdrv_getlength. I think it makes sense
that this function (which should not be truncate, but something else)
expects to be called with no pending requests. It shouldn't be its task
to drain them.
I'll look at fixing this as mentioned in the earlier thread.
Paolo
