diff options
| author | Nicholas Piggin <npiggin@gmail.com> | 2024-08-13 21:23:21 +0100 |
|---|---|---|
| committer | Alex Bennée <alex.bennee@linaro.org> | 2024-08-16 14:04:19 +0100 |
| commit | 94962ff00d09674047aed896e87ba09736cd6941 (patch) | |
| tree | 60cdb127e8f461885f926931eaf6c05c04a1ea64 /replay | |
| parent | 9dbab31d9eb710b433f93e9e35962fbfd29f4c3f (diff) | |
| download | qemu-94962ff00d09674047aed896e87ba09736cd6941.zip qemu-94962ff00d09674047aed896e87ba09736cd6941.tar.gz qemu-94962ff00d09674047aed896e87ba09736cd6941.tar.bz2 | |
Revert "replay: stop us hanging in rr_wait_io_event"
This reverts commit 1f881ea4a444ef36a8b6907b0b82be4b3af253a2.
That commit causes reverse_debugging.py test failures, and does
not seem to solve the root cause of the problem x86-64 still
hangs in record/replay tests.
The problem with short-cutting the iowait that was taken during
record phase is that related events will not get consumed at the
same points (e.g., reading the clock).
A hang with zero icount always seems to be a symptom of an earlier
problem that has caused the recording to become out of synch with
the execution and consumption of events by replay.
Acked-by: Alex Bennée <alex.bennee@linaro.org>
Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
Message-Id: <20240813050638.446172-6-npiggin@gmail.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20240813202329.1237572-14-alex.bennee@linaro.org>
Diffstat (limited to 'replay')
| -rw-r--r-- | replay/replay.c | 21 |
1 files changed, 0 insertions, 21 deletions
diff --git a/replay/replay.c b/replay/replay.c index b8564a4..895fa6b 100644 --- a/replay/replay.c +++ b/replay/replay.c @@ -451,27 +451,6 @@ void replay_start(void) replay_enable_events(); } -/* - * For none/record the answer is yes. - */ -bool replay_can_wait(void) -{ - if (replay_mode == REPLAY_MODE_PLAY) { - /* - * For playback we shouldn't ever be at a point we wait. If - * the instruction count has reached zero and we have an - * unconsumed event we should go around again and consume it. - */ - if (replay_state.instruction_count == 0 && replay_state.has_unread_data) { - return false; - } else { - replay_sync_error("Playback shouldn't have to iowait"); - } - } - return true; -} - - void replay_finish(void) { if (replay_mode == REPLAY_MODE_NONE) { |