[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v9 19/21] replay: add BH oneshot event for block
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH v9 19/21] replay: add BH oneshot event for block layer |
Date: |
Fri, 11 Jan 2019 11:49:23 +0100 |
User-agent: |
Mutt/1.10.1 (2018-07-13) |
Am 09.01.2019 um 13:13 hat Pavel Dovgalyuk geschrieben:
> Replay is capable of recording normal BH events, but sometimes
> there are single use callbacks scheduled with aio_bh_schedule_oneshot
> function. This patch enables recording and replaying such callbacks.
> Block layer uses these events for calling the completion function.
> Replaying these calls makes the execution deterministic.
>
> Signed-off-by: Pavel Dovgalyuk <address@hidden>
This still doesn't come even close to catching all BHs that need to be
caught. While you managed to show a few BHs that actually don't need to
be considered for recording when I asked for this in v7, most BHs in the
block layer can in some way lead to device callbacks and must therefore
be recorded.
How bad would it be to record some BHs even if recording them isn't
necessary? I'd definitely try to err on the safe side here. Having two
different sets of BH functions, you can't expect that people always use
the right one (especially if you don't even make the existing code base
consistently use the right one intially).
Kevin
- Re: [Qemu-devel] [PATCH v9 12/21] replay: introduce breakpoint at the specified step, (continued)
- [Qemu-devel] [PATCH v9 13/21] replay: implement replay-seek command to proceed to the desired step, Pavel Dovgalyuk, 2019/01/09
- [Qemu-devel] [PATCH v9 14/21] replay: refine replay-time module, Pavel Dovgalyuk, 2019/01/09
- [Qemu-devel] [PATCH v9 15/21] replay: flush rr queue before loading the vmstate, Pavel Dovgalyuk, 2019/01/09
- [Qemu-devel] [PATCH v9 16/21] gdbstub: add reverse step support in replay mode, Pavel Dovgalyuk, 2019/01/09
- [Qemu-devel] [PATCH v9 17/21] gdbstub: add reverse continue support in replay mode, Pavel Dovgalyuk, 2019/01/09
- [Qemu-devel] [PATCH v9 18/21] replay: describe reverse debugging in docs/replay.txt, Pavel Dovgalyuk, 2019/01/09
- [Qemu-devel] [PATCH v9 19/21] replay: add BH oneshot event for block layer, Pavel Dovgalyuk, 2019/01/09
- Re: [Qemu-devel] [PATCH v9 19/21] replay: add BH oneshot event for block layer,
Kevin Wolf <=
[Qemu-devel] [PATCH v9 20/21] replay: init rtc after enabling the replay, Pavel Dovgalyuk, 2019/01/09
[Qemu-devel] [PATCH v9 21/21] replay: document development rules, Pavel Dovgalyuk, 2019/01/09
Re: [Qemu-devel] [PATCH v9 00/21] Fixing record/replay and adding reverse debugging, no-reply, 2019/01/13