On Wed 27 Apr 2016 02:04:33 PM CEST, Max Reitz wrote:
>> - bs = NULL;
>> - while ((bs = bdrv_next(bs))) {
>> - AioContext *aio_context = bdrv_get_aio_context(bs);
>> + while ((job = block_job_next(job))) {
>> + AioContext *aio_context = bdrv_get_aio_context(job->bs);
>
> Technically, the "bs = NULL;" before didn't do anything either. But in
> my opinion, it made the code more readable, therefore I'd really like
> a "job = NULL;" before this loop, too.
Looks reasonable, I'll update it in the next revision.
Berto