[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 11/12] mc146818rtc: add missed field to vmstate
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 11/12] mc146818rtc: add missed field to vmstate |
Date: |
Tue, 26 Aug 2014 10:58:26 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.7.0 |
Il 26/08/2014 09:15, Pavel Dovgalyuk ha scritto:
> +static int rtc_pre_load(void *opaque)
> +{
> + RTCState *s = (RTCState *)opaque;
> + s->irq_reinject_on_ack_count = 0;
> + return 0;
> +}
> +
You found a real bug here, in that the field has to be cleared at reset
time. But reinitializing stuff in pre-load hooks is another quirk of
your record/replay patches that does not belong in these patches (and
that you have not justified yet, either).
Paolo
- Re: [Qemu-devel] [PATCH 09/12] rtl8139: adding new fields to vmstate, (continued)
- Re: [Qemu-devel] [PATCH 09/12] rtl8139: adding new fields to vmstate, Paolo Bonzini, 2014/08/27
- Re: [Qemu-devel] [PATCH 09/12] rtl8139: adding new fields to vmstate, Pavel Dovgaluk, 2014/08/27
- Re: [Qemu-devel] [PATCH 09/12] rtl8139: adding new fields to vmstate, Paolo Bonzini, 2014/08/27
- Re: [Qemu-devel] [PATCH 09/12] rtl8139: adding new fields to vmstate, Pavel Dovgaluk, 2014/08/27
- Message not available
- Re: [Qemu-devel] [PATCH 09/12] rtl8139: adding new fields to vmstate, Paolo Bonzini, 2014/08/27
- Re: [Qemu-devel] [PATCH 09/12] rtl8139: adding new fields to vmstate, Pavel Dovgaluk, 2014/08/28
- Re: [Qemu-devel] [PATCH 09/12] rtl8139: adding new fields to vmstate, Paolo Bonzini, 2014/08/28
[Qemu-devel] [PATCH 10/12] piix: do not raise irq while loading vmstate, Pavel Dovgalyuk, 2014/08/26
[Qemu-devel] [PATCH 11/12] mc146818rtc: add missed field to vmstate, Pavel Dovgalyuk, 2014/08/26
- Re: [Qemu-devel] [PATCH 11/12] mc146818rtc: add missed field to vmstate,
Paolo Bonzini <=
[Qemu-devel] [PATCH 12/12] pl031: add missed field to vmstate, Pavel Dovgalyuk, 2014/08/26