[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 17/21] virtio-net: Use virtual time for RSC timers
From: |
Michael S. Tsirkin |
Subject: |
Re: [PATCH v2 17/21] virtio-net: Use virtual time for RSC timers |
Date: |
Tue, 13 Aug 2024 16:49:24 -0400 |
On Tue, Aug 13, 2024 at 09:23:25PM +0100, Alex Bennée wrote:
> From: Nicholas Piggin <npiggin@gmail.com>
>
> Receive coalescing is visible to the target machine, so its timers
> should use virtual time like other timers in virtio-net, to be
> compatible with record-replay.
>
> Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
> Message-Id: <20240813050638.446172-10-npiggin@gmail.com>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Acked-by: Michael S. Tsirkin <mst@redhat.com>
> ---
> hw/net/virtio-net.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/hw/net/virtio-net.c b/hw/net/virtio-net.c
> index 10ebaae5e2..ed33a32877 100644
> --- a/hw/net/virtio-net.c
> +++ b/hw/net/virtio-net.c
> @@ -2124,7 +2124,7 @@ static void virtio_net_rsc_purge(void *opq)
> chain->stat.timer++;
> if (!QTAILQ_EMPTY(&chain->buffers)) {
> timer_mod(chain->drain_timer,
> - qemu_clock_get_ns(QEMU_CLOCK_HOST) + chain->n->rsc_timeout);
> + qemu_clock_get_ns(QEMU_CLOCK_VIRTUAL) + chain->n->rsc_timeout);
> }
> }
>
> @@ -2360,7 +2360,7 @@ static size_t
> virtio_net_rsc_do_coalesce(VirtioNetRscChain *chain,
> chain->stat.empty_cache++;
> virtio_net_rsc_cache_buf(chain, nc, buf, size);
> timer_mod(chain->drain_timer,
> - qemu_clock_get_ns(QEMU_CLOCK_HOST) + chain->n->rsc_timeout);
> + qemu_clock_get_ns(QEMU_CLOCK_VIRTUAL) + chain->n->rsc_timeout);
> return size;
> }
>
> @@ -2598,7 +2598,7 @@ static VirtioNetRscChain
> *virtio_net_rsc_lookup_chain(VirtIONet *n,
> chain->max_payload = VIRTIO_NET_MAX_IP6_PAYLOAD;
> chain->gso_type = VIRTIO_NET_HDR_GSO_TCPV6;
> }
> - chain->drain_timer = timer_new_ns(QEMU_CLOCK_HOST,
> + chain->drain_timer = timer_new_ns(QEMU_CLOCK_VIRTUAL,
> virtio_net_rsc_purge, chain);
> memset(&chain->stat, 0, sizeof(chain->stat));
>
> --
> 2.39.2
- Re: [PATCH v2 16/21] virtio-net: Use replay_schedule_bh_event for bhs that affect machine state, (continued)
- Re: [PATCH v2 16/21] virtio-net: Use replay_schedule_bh_event for bhs that affect machine state, Alex Bennée, 2024/08/14
- Re: [PATCH v2 16/21] virtio-net: Use replay_schedule_bh_event for bhs that affect machine state, Nicholas Piggin, 2024/08/15
- Re: [PATCH v2 16/21] virtio-net: Use replay_schedule_bh_event for bhs that affect machine state, Michael S. Tsirkin, 2024/08/15
- Re: [PATCH v2 16/21] virtio-net: Use replay_schedule_bh_event for bhs that affect machine state, Nicholas Piggin, 2024/08/15
- Re: [PATCH v2 16/21] virtio-net: Use replay_schedule_bh_event for bhs that affect machine state, Jason Wang, 2024/08/15
- Re: [PATCH v2 16/21] virtio-net: Use replay_schedule_bh_event for bhs that affect machine state, Alex Bennée, 2024/08/16
[PATCH v2 15/21] chardev: set record/replay on the base device of a muxed device, Alex Bennée, 2024/08/13
[PATCH v2 21/21] plugins: fix race condition with scoreboards, Alex Bennée, 2024/08/13
[PATCH v2 10/21] scripts/replay-dump.py: rejig decoders in event number order, Alex Bennée, 2024/08/13
[PATCH v2 17/21] virtio-net: Use virtual time for RSC timers, Alex Bennée, 2024/08/13
- Re: [PATCH v2 17/21] virtio-net: Use virtual time for RSC timers,
Michael S. Tsirkin <=
[PATCH v2 19/21] docs: Fix some typos (found by typos) and grammar issues, Alex Bennée, 2024/08/13
[PATCH v2 18/21] savevm: Fix load_snapshot error path crash, Alex Bennée, 2024/08/13
[PATCH v2 20/21] docs/devel: update tcg-plugins page, Alex Bennée, 2024/08/13