From 39ed143e275f208501469d484bb7104503fd6f1d Mon Sep 17 00:00:00 2001 From: Thomas Schoebel-Theuer Date: Tue, 18 Jun 2013 10:57:32 +0200 Subject: [PATCH] logger: fix potential race on replay_end_pos The variable should only be set from outside, e.g. mars_light. This should also lead to more correct replay_code==2 results. --- kernel/mars_trans_logger.c | 1 - 1 file changed, 1 deletion(-) diff --git a/kernel/mars_trans_logger.c b/kernel/mars_trans_logger.c index 34f61c8a..6c5020b4 100644 --- a/kernel/mars_trans_logger.c +++ b/kernel/mars_trans_logger.c @@ -2882,7 +2882,6 @@ void trans_logger_replay(struct trans_logger_brick *brick) MARS_DBG("EOF at %lld (old = %lld, end_pos = %lld)\n", new_finished_pos, finished_pos, brick->replay_end_pos); if (!brick->continuous_replay_mode) { // notice: finished_pos remains at old value here! - brick->replay_end_pos = finished_pos; break; } brick_msleep(1000);