On 11/07/2017 10:28, Kevin Wolf wrote: > Am 10.07.2017 um 18:58 hat Paolo Bonzini geschrieben: >> It is called from qcow2_invalidate_cache in coroutine context, so always >> load metadata from a coroutine. >> --- >> block/qcow2.c | 45 ++++++++++++++++++++++++++++++++++++++++----- >> 1 file changed, 40 insertions(+), 5 deletions(-) > > Missing S-o-b on patches 1 and 2?
Oops, yes. Though I don't think these patches are for 2.10. They are on top of Stefan's bdrv_co_create series, by the way. Paolo > These patches suggest that .bdrv_co_open() might be the next thing to > get rid of the coroutine wrappers again in driver code. But we'll always > have some wrappers in the drivers in an intermediate state, so doing > .bdrv_invalidate_cache first is fine. > > Kevin >