[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 01/13] qed: Don't yield in bdrv_qed_co_drain_begin()
From: |
Kevin Wolf |
Subject: |
Re: [PATCH 01/13] qed: Don't yield in bdrv_qed_co_drain_begin() |
Date: |
Wed, 9 Nov 2022 13:22:13 +0100 |
Am 09.11.2022 um 10:27 hat Vladimir Sementsov-Ogievskiy geschrieben:
> On 11/8/22 15:37, Kevin Wolf wrote:
> > int ret;
> > trace_qed_need_check_timer_cb(s);
> > @@ -310,9 +309,20 @@ static void coroutine_fn
> > qed_need_check_timer_entry(void *opaque)
> > (void) ret;
> > }
> > +static void coroutine_fn qed_need_check_timer_entry(void *opaque)
> > +{
> > + BDRVQEDState *s = opaque;
> > +
> > + qed_need_check_timer(opaque);
> > + bdrv_dec_in_flight(s->bs);
>
> hmm, one question: don't we need aio_wait_kick() call here?
bdrv_dec_in_flight() already calls aio_wait_kick() internally, so any
places that use it don't need a separate aio_wait_kick().
Kevin
- [PATCH 00/13] block: Simplify drain, Kevin Wolf, 2022/11/08
- [PATCH 01/13] qed: Don't yield in bdrv_qed_co_drain_begin(), Kevin Wolf, 2022/11/08
- Re: [PATCH 01/13] qed: Don't yield in bdrv_qed_co_drain_begin(), Vladimir Sementsov-Ogievskiy, 2022/11/09
- Re: [PATCH 01/13] qed: Don't yield in bdrv_qed_co_drain_begin(), Vladimir Sementsov-Ogievskiy, 2022/11/09
- Re: [PATCH 01/13] qed: Don't yield in bdrv_qed_co_drain_begin(),
Kevin Wolf <=
- Re: [PATCH 01/13] qed: Don't yield in bdrv_qed_co_drain_begin(), Stefan Hajnoczi, 2022/11/09
- Re: [PATCH 01/13] qed: Don't yield in bdrv_qed_co_drain_begin(), Emanuele Giuseppe Esposito, 2022/11/11
- Re: [PATCH 01/13] qed: Don't yield in bdrv_qed_co_drain_begin(), Hanna Reitz, 2022/11/14
- [PATCH 03/13] block: Revert .bdrv_drained_begin/end to non-coroutine_fn, Kevin Wolf, 2022/11/08
- [PATCH 05/13] block: Inline bdrv_drain_invoke(), Kevin Wolf, 2022/11/08