Kevin Wolf <kw...@redhat.com> writes: > Am 13.12.2019 um 21:59 hat Eric Blake geschrieben: >> On 12/9/19 10:06 AM, Kevin Wolf wrote: >> > Am 28.11.2019 um 11:41 hat Sergio Lopez geschrieben: >> > > bdrv_try_set_aio_context() requires that the old context is held, and >> > > the new context is not held. Fix all the occurrences where it's not >> > > done this way. >> > > >> > > Suggested-by: Max Reitz <mre...@redhat.com> >> > > Signed-off-by: Sergio Lopez <s...@redhat.com> >> > > --- >> >> > Or in fact, I think you need to hold the AioContext of a bs to >> > bdrv_unref() it, so maybe 'goto out' is right, but you need to unref >> > target_bs while you still hold old_context. >> >> I suspect https://bugzilla.redhat.com/show_bug.cgi?id=1779036 is also a >> symptom of this. The v5 patch did not fix this simple test case: > > Speaking of a test case... I think this series should probably add > something to iotests.
Okay, I'll try to add one. Thanks, Sergio.
signature.asc
Description: PGP signature
