|
| From: | Paolo Bonzini |
| Subject: | Re: [PATCH v2] async: the main AioContext is only "current" if under the BQL |
| Date: | Mon, 14 Jun 2021 13:34:01 +0200 |
| User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.10.1 |
On 11/06/21 13:13, Paolo Bonzini wrote:
The commit message doesn't specify, but in the buggy case, are we talking about calling aio_co_wake() for a coroutine in the main context specifically, right? Could we have a unit test for this scenario?Yes, that's the scenario. I will try to write a unit test indeed.
Done ("tests: cover aio_co_enter from a worker thread without BQL taken"). aio_co_enter() is the function that "does the work" for aio_co_wake(). Eric, you may want to review and pick that up as well.
Paolo
| [Prev in Thread] | Current Thread | [Next in Thread] |