From nobody Tue Feb 10 19:17:10 2026 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=gmail.com ARC-Seal: i=1; a=rsa-sha256; t=1563957911; cv=none; d=zoho.com; s=zohoarc; b=XNjF6bpR4F9sFxiF32SAWbWsk4gZoTgsHBk71Q+ouqXyu4CTmRNn8dJS58MtGZFeznVt2DNp/BicYtpWF0oMczH4eVkt4ZWNoQTC8jFA/E66APLu4QgWnR9r5Rg1SRYSlOAgfWur5wmxwAxw9A8mBAgDB/9DDqAgNJ4GQj3hjuw= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1563957911; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To:ARC-Authentication-Results; bh=Mgi/5UNE+eRHpGWwQEiI1T9f+4HDw8WY3Y+35VaZEAw=; b=hiktbhsMZUTgtiUOBGAHKNvMoqDQT1bGjBPmYLcwlKbW5e1aYKlt77IBYeSGLBZi4tydsx4tVFodY8QrIq69zyekIEhIn6/2Tun4HiQJ5MpKYPMaEYWZ/VlW+Re0TBbZr4xAn0ZLaDvAxKC2UbU1lqP7tIaYjNed8kNjlBUNvUI= ARC-Authentication-Results: i=1; mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1563957911875437.2390381244902; Wed, 24 Jul 2019 01:45:11 -0700 (PDT) Received: from localhost ([::1]:49766 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hqCta-0000Co-HU for importer@patchew.org; Wed, 24 Jul 2019 04:45:10 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:34406) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hqCsy-0006di-RL for qemu-devel@nongnu.org; Wed, 24 Jul 2019 04:44:34 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hqCsw-0002dU-TR for qemu-devel@nongnu.org; Wed, 24 Jul 2019 04:44:32 -0400 Received: from mail.ispras.ru ([83.149.199.45]:59078) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1hqCsw-0002c0-IG for qemu-devel@nongnu.org; Wed, 24 Jul 2019 04:44:30 -0400 Received: from [127.0.1.1] (unknown [85.142.117.226]) by mail.ispras.ru (Postfix) with ESMTPSA id 30E8754006A; Wed, 24 Jul 2019 11:44:27 +0300 (MSK) From: Pavel Dovgalyuk To: qemu-devel@nongnu.org Date: Wed, 24 Jul 2019 11:44:27 +0300 Message-ID: <156395786700.510.11133540400113889902.stgit@pasha-Precision-3630-Tower> In-Reply-To: <156395778867.510.17588721322993616668.stgit@pasha-Precision-3630-Tower> References: <156395778867.510.17588721322993616668.stgit@pasha-Precision-3630-Tower> User-Agent: StGit/0.17.1-dirty MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 3.x X-Received-From: 83.149.199.45 Subject: [Qemu-devel] [PATCH for-4.2 13/14] replay: rename step-related variables and functions X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kwolf@redhat.com, peter.maydell@linaro.org, pavel.dovgaluk@ispras.ru, pbonzini@redhat.com, quintela@redhat.com, ciro.santilli@gmail.com, jasowang@redhat.com, crosthwaite.peter@gmail.com, armbru@redhat.com, mreitz@redhat.com, alex.bennee@linaro.org, maria.klimushenkova@ispras.ru, mst@redhat.com, kraxel@redhat.com, boost.lists@gmail.com, thomas.dullien@googlemail.com, dovgaluk@ispras.ru, artem.k.pisarenko@gmail.com, dgilbert@redhat.com, rth@twiddle.net Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" From: Pavel Dovgalyuk This patch renames replay_get_current_step() and related variables to make these names consistent with existing 'icount' command line option and future record/replay hmp/qmp commands. Signed-off-by: Pavel Dovgalyuk --- include/sysemu/replay.h | 2 +- replay/replay-events.c | 4 ++-- replay/replay-internal.c | 10 +++++----- replay/replay-internal.h | 10 +++++----- replay/replay-snapshot.c | 6 +++--- replay/replay-time.c | 2 +- replay/replay.c | 22 +++++++++++----------- 7 files changed, 28 insertions(+), 28 deletions(-) diff --git a/include/sysemu/replay.h b/include/sysemu/replay.h index 503bb6258d..6ed4322447 100644 --- a/include/sysemu/replay.h +++ b/include/sysemu/replay.h @@ -77,7 +77,7 @@ const char *replay_get_filename(void); /* Processing the instructions */ =20 /*! Returns number of executed instructions. */ -uint64_t replay_get_current_step(void); +uint64_t replay_get_current_icount(void); /*! Returns number of instructions to execute in replay mode. */ int replay_get_instructions(void); /*! Updates instructions counter in replay mode. */ diff --git a/replay/replay-events.c b/replay/replay-events.c index e5bf1bc2e6..302b84043a 100644 --- a/replay/replay-events.c +++ b/replay/replay-events.c @@ -127,7 +127,7 @@ void replay_add_event(ReplayAsyncEventKind event_kind, void replay_bh_schedule_event(QEMUBH *bh) { if (events_enabled) { - uint64_t id =3D replay_get_current_step(); + uint64_t id =3D replay_get_current_icount(); replay_add_event(REPLAY_ASYNC_EVENT_BH, bh, NULL, id); } else { qemu_bh_schedule(bh); @@ -138,7 +138,7 @@ void replay_bh_schedule_oneshot_event(AioContext *ctx, QEMUBHFunc *cb, void *opaque) { if (events_enabled) { - uint64_t id =3D replay_get_current_step(); + uint64_t id =3D replay_get_current_icount(); replay_add_event(REPLAY_ASYNC_EVENT_BH_ONESHOT, cb, opaque, id); } else { aio_bh_schedule_oneshot(ctx, cb, opaque); diff --git a/replay/replay-internal.c b/replay/replay-internal.c index 979f3a0b39..ac6c9017fc 100644 --- a/replay/replay-internal.c +++ b/replay/replay-internal.c @@ -172,7 +172,7 @@ void replay_fetch_data_kind(void) if (!replay_state.has_unread_data) { replay_state.data_kind =3D replay_get_byte(); if (replay_state.data_kind =3D=3D EVENT_INSTRUCTION) { - replay_state.instructions_count =3D replay_get_dword(); + replay_state.instruction_count =3D replay_get_dword(); } replay_check_error(); replay_state.has_unread_data =3D 1; @@ -226,9 +226,9 @@ void replay_mutex_unlock(void) } } =20 -void replay_advance_current_step(uint64_t current_step) +void replay_advance_current_icount(uint64_t current_icount) { - int diff =3D (int)(current_step - replay_state.current_step); + int diff =3D (int)(current_icount - replay_state.current_icount); =20 /* Time can only go forward */ assert(diff >=3D 0); @@ -236,7 +236,7 @@ void replay_advance_current_step(uint64_t current_step) if (diff > 0) { replay_put_event(EVENT_INSTRUCTION); replay_put_dword(diff); - replay_state.current_step +=3D diff; + replay_state.current_icount +=3D diff; } } =20 @@ -245,6 +245,6 @@ void replay_save_instructions(void) { if (replay_file && replay_mode =3D=3D REPLAY_MODE_RECORD) { g_assert(replay_mutex_locked()); - replay_advance_current_step(replay_get_current_step()); + replay_advance_current_icount(replay_get_current_icount()); } } diff --git a/replay/replay-internal.h b/replay/replay-internal.h index f3cdcfd93e..55fca1ac6b 100644 --- a/replay/replay-internal.h +++ b/replay/replay-internal.h @@ -65,10 +65,10 @@ typedef enum ReplayAsyncEventKind ReplayAsyncEventKind; typedef struct ReplayState { /*! Cached clock values. */ int64_t cached_clock[REPLAY_CLOCK_COUNT]; - /*! Current step - number of processed instructions and timer events. = */ - uint64_t current_step; + /*! Current icount - number of processed instructions. */ + uint64_t current_icount; /*! Number of instructions to be executed before other events happen. = */ - int instructions_count; + int instruction_count; /*! Type of the currently executed event. */ unsigned int data_kind; /*! Flag which indicates that event is not processed yet. */ @@ -123,8 +123,8 @@ void replay_finish_event(void); data_kind variable. */ void replay_fetch_data_kind(void); =20 -/*! Advance replay_state.current_step to the specified value. */ -void replay_advance_current_step(uint64_t current_step); +/*! Advance replay_state.current_icount to the specified value. */ +void replay_advance_current_icount(uint64_t current_icount); /*! Saves queued events (like instructions and sound). */ void replay_save_instructions(void); =20 diff --git a/replay/replay-snapshot.c b/replay/replay-snapshot.c index 756f48bc02..97d026af30 100644 --- a/replay/replay-snapshot.c +++ b/replay/replay-snapshot.c @@ -41,7 +41,7 @@ static int replay_post_load(void *opaque, int version_id) } else if (replay_mode =3D=3D REPLAY_MODE_RECORD) { /* This is only useful for loading the initial state. Therefore reset all the counters. */ - state->instructions_count =3D 0; + state->instruction_count =3D 0; state->block_request_id =3D 0; } =20 @@ -56,8 +56,8 @@ static const VMStateDescription vmstate_replay =3D { .post_load =3D replay_post_load, .fields =3D (VMStateField[]) { VMSTATE_INT64_ARRAY(cached_clock, ReplayState, REPLAY_CLOCK_COUNT), - VMSTATE_UINT64(current_step, ReplayState), - VMSTATE_INT32(instructions_count, ReplayState), + VMSTATE_UINT64(current_icount, ReplayState), + VMSTATE_INT32(instruction_count, ReplayState), VMSTATE_UINT32(data_kind, ReplayState), VMSTATE_UINT32(has_unread_data, ReplayState), VMSTATE_UINT64(file_offset, ReplayState), diff --git a/replay/replay-time.c b/replay/replay-time.c index 49c9e5d8b2..43357c9f24 100644 --- a/replay/replay-time.c +++ b/replay/replay-time.c @@ -24,7 +24,7 @@ int64_t replay_save_clock(ReplayClockKind kind, int64_t c= lock, * Due to the caller's locking requirements we get the icount from it * instead of using replay_save_instructions(). */ - replay_advance_current_step(raw_icount); + replay_advance_current_icount(raw_icount); replay_put_event(EVENT_CLOCK + kind); replay_put_qword(clock); =20 diff --git a/replay/replay.c b/replay/replay.c index 73c5d4e207..14ee2c4144 100644 --- a/replay/replay.c +++ b/replay/replay.c @@ -39,7 +39,7 @@ bool replay_next_event_is(int event) bool res =3D false; =20 /* nothing to skip - not all instructions used */ - if (replay_state.instructions_count !=3D 0) { + if (replay_state.instruction_count !=3D 0) { assert(replay_state.data_kind =3D=3D EVENT_INSTRUCTION); return event =3D=3D EVENT_INSTRUCTION; } @@ -62,7 +62,7 @@ bool replay_next_event_is(int event) return res; } =20 -uint64_t replay_get_current_step(void) +uint64_t replay_get_current_icount(void) { return cpu_get_icount_raw(); } @@ -72,7 +72,7 @@ int replay_get_instructions(void) int res =3D 0; replay_mutex_lock(); if (replay_next_event_is(EVENT_INSTRUCTION)) { - res =3D replay_state.instructions_count; + res =3D replay_state.instruction_count; } replay_mutex_unlock(); return res; @@ -82,16 +82,16 @@ void replay_account_executed_instructions(void) { if (replay_mode =3D=3D REPLAY_MODE_PLAY) { g_assert(replay_mutex_locked()); - if (replay_state.instructions_count > 0) { - int count =3D (int)(replay_get_current_step() - - replay_state.current_step); + if (replay_state.instruction_count > 0) { + int count =3D (int)(replay_get_current_icount() + - replay_state.current_icount); =20 /* Time can only go forward */ assert(count >=3D 0); =20 - replay_state.instructions_count -=3D count; - replay_state.current_step +=3D count; - if (replay_state.instructions_count =3D=3D 0) { + replay_state.instruction_count -=3D count; + replay_state.current_icount +=3D count; + if (replay_state.instruction_count =3D=3D 0) { assert(replay_state.data_kind =3D=3D EVENT_INSTRUCTION); replay_finish_event(); /* Wake up iothread. This is required because @@ -273,8 +273,8 @@ static void replay_enable(const char *fname, int mode) replay_mutex_init(); =20 replay_state.data_kind =3D -1; - replay_state.instructions_count =3D 0; - replay_state.current_step =3D 0; + replay_state.instruction_count =3D 0; + replay_state.current_icount =3D 0; replay_state.has_unread_data =3D 0; =20 /* skip file header for RECORD and check it for PLAY */