[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3] bugfix: vm halt when in reset looping
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH v3] bugfix: vm halt when in reset looping |
Date: |
Fri, 16 Dec 2016 11:54:58 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.5.1 |
On 16/12/2016 09:23, hangaohuai wrote:
> reset mc146818rtc device when RESET event happens.
>
> Fix the problem:
> 1. Guest boot the second cpu, set CMOS_RESET_CODE 0x0a to protect selfboot;
> 2. VM being reset by others, hmp_system_reset;
> 3. seabios resume check the CMOS_RESET_CODE, if 0x0a, jump to the BDA
> resume execution by jump via 40h:0067h;
> 4. Guest halt;
>
> Signed-off-by: hangaohuai <address@hidden>
> ---
> hw/timer/mc146818rtc.c | 11 +++++++++++
> 1 file changed, 11 insertions(+)
>
> diff --git a/hw/timer/mc146818rtc.c b/hw/timer/mc146818rtc.c
> index da209d0..341b233 100644
> --- a/hw/timer/mc146818rtc.c
> +++ b/hw/timer/mc146818rtc.c
> @@ -946,11 +946,22 @@ static Property mc146818rtc_properties[] = {
> DEFINE_PROP_END_OF_LIST(),
> };
>
> +static void rtc_resetdev(DeviceState *d)
> +{
> + RTCState *s = MC146818_RTC(d);
> +
> + /* Reason: VM do suspend self will set 0xfe
> + * Only handle the VM boot case */
> + if (s->cmos_data[0x0f] == 0x0a)
> + s->cmos_data[0x0f] = 0x00;
> +}
Let's just do it for all values other than 0xfe. Also, please run
checkpatch.pl on your patch.
Paolo
> static void rtc_class_initfn(ObjectClass *klass, void *data)
> {
> DeviceClass *dc = DEVICE_CLASS(klass);
>
> dc->realize = rtc_realizefn;
> + dc->reset = rtc_resetdev;
> dc->vmsd = &vmstate_rtc;
> dc->props = mc146818rtc_properties;
> /* Reason: needs to be wired up by rtc_init() */
>