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