[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 2/7] hpet: Save/restore cached RTC IRQ level
From: |
Jan Kiszka |
Subject: |
Re: [Qemu-devel] [PATCH v3 2/7] hpet: Save/restore cached RTC IRQ level |
Date: |
Tue, 31 Jan 2012 23:05:33 +0100 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666 |
On 2012-01-31 22:02, Anthony Liguori wrote:
> On 01/31/2012 11:41 AM, Jan Kiszka wrote:
>> In legacy mode, the HPET suppresses the RTC interrupt delivery via IRQ
>> 8 but keeps track of the RTC output level and applies it when legacy
>> mode is turned off again. This value has to be preserved across save/
>> restore as it cannot be reconstructed otherwise.
>>
>> Signed-off-by: Jan Kiszka<address@hidden>
>> ---
>> hw/hpet.c | 26 ++++++++++++++++++++++++++
>> 1 files changed, 26 insertions(+), 0 deletions(-)
>>
>> diff --git a/hw/hpet.c b/hw/hpet.c
>> index aba9ea9..39b291f 100644
>> --- a/hw/hpet.c
>> +++ b/hw/hpet.c
>> @@ -240,6 +240,24 @@ static int hpet_post_load(void *opaque, int
>> version_id)
>> return 0;
>> }
>>
>> +static bool hpet_rtc_irq_level_needed(void *opaque)
>> +{
>> + HPETState *s = opaque;
>> +
>> + return s->rtc_irq_level != 0;
>> +}
>> +
>> +static const VMStateDescription vmstate_hpet_rtc_irq_level = {
>> + .name = "hpet/rtc_irq_level",
>> + .version_id = 1,
>> + .minimum_version_id = 1,
>> + .minimum_version_id_old = 1,
>> + .fields = (VMStateField[]) {
>> + VMSTATE_UINT8(rtc_irq_level, HPETState),
>> + VMSTATE_END_OF_LIST()
>> + }
>> +};
>> +
>
> This won't work. We don't clear rtc_irq_level on reset so rtc_irq_level
> may be high or low after reset. As such, we can't use a subsection
> here. We need to bump the savevm state.
rtc_irq_level is updated during reset by the connected RTC device. It
clears its IRQ line.
Jan
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH v3 4/7] i8254: Pass alternative IRQ output object on initialization, (continued)
[Qemu-devel] [PATCH v3 2/7] hpet: Save/restore cached RTC IRQ level, Jan Kiszka, 2012/01/31
[Qemu-devel] [PATCH v3 5/7] i8254: Rework & fix interaction with HPET in legacy mode, Jan Kiszka, 2012/01/31