[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] pvpanic plans?
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] pvpanic plans? |
Date: |
Thu, 31 Oct 2013 17:17:24 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130923 Thunderbird/17.0.9 |
Il 31/10/2013 17:14, Michael S. Tsirkin ha scritto:
>> PANICKED->DEBUG was added by commit bc7d0e667. That commit can be
>> reverted if the panicked state is removed from runstate_needs_reset.
>
> Okay so let's drop the code duplication and explicitly make
> them the same?
>
> Signed-off-by: Michael S. Tsirkin <address@hidden>
>
>
> diff --git a/vl.c b/vl.c
> index 46c29c4..e12d317 100644
> --- a/vl.c
> +++ b/vl.c
> @@ -638,10 +638,6 @@ static const RunStateTransition
> runstate_transitions_def[] = {
> { RUN_STATE_WATCHDOG, RUN_STATE_RUNNING },
> { RUN_STATE_WATCHDOG, RUN_STATE_FINISH_MIGRATE },
>
> - { RUN_STATE_GUEST_PANICKED, RUN_STATE_PAUSED },
> - { RUN_STATE_GUEST_PANICKED, RUN_STATE_FINISH_MIGRATE },
> - { RUN_STATE_GUEST_PANICKED, RUN_STATE_DEBUG },
> -
> { RUN_STATE_MAX, RUN_STATE_MAX },
> };
>
> @@ -660,6 +656,12 @@ static void runstate_init(void)
>
> for (p = &runstate_transitions_def[0]; p->from != RUN_STATE_MAX; p++) {
> runstate_valid_transitions[p->from][p->to] = true;
> + /* Panicked state is same as paused, we only made it different so
> + * management can detect a panic.
> + */
> + if (p->from == RUN_STATE_PAUSED) {
> + runstate_valid_transitions[RUN_STATE_GUEST_PANICKED][p->to] =
> true;
It makes only sense to me if you do that for IO_ERROR and WATCHDOG as
well, and perhaps there are others I'm missing. Just add a comment
before runstate_transitions_def's entries for PANICKED, IO_ERROR and
WATCHDOG.
But again, it is somewhat separate from the issue at hand, which is to
finally make pvpanic usable and hopefully before 1.7.
Paolo
> + }
> }
> }
>
> @@ -686,8 +688,7 @@ int runstate_is_running(void)
> bool runstate_needs_reset(void)
> {
> return runstate_check(RUN_STATE_INTERNAL_ERROR) ||
> - runstate_check(RUN_STATE_SHUTDOWN) ||
> - runstate_check(RUN_STATE_GUEST_PANICKED);
> + runstate_check(RUN_STATE_SHUTDOWN);
> }
>
> StatusInfo *qmp_query_status(Error **errp)
>
- Re: [Qemu-devel] pvpanic plans?, (continued)
- Re: [Qemu-devel] pvpanic plans?, Eric Blake, 2013/10/31
- Re: [Qemu-devel] pvpanic plans?, Anthony Liguori, 2013/10/31
- Re: [Qemu-devel] pvpanic plans?, Paolo Bonzini, 2013/10/31
- Re: [Qemu-devel] pvpanic plans?, Michael S. Tsirkin, 2013/10/31
- Re: [Qemu-devel] pvpanic plans?, Paolo Bonzini, 2013/10/31
- Re: [Qemu-devel] pvpanic plans?, Michael S. Tsirkin, 2013/10/31
- Re: [Qemu-devel] pvpanic plans?, Paolo Bonzini, 2013/10/31
- Re: [Qemu-devel] pvpanic plans?, Michael S. Tsirkin, 2013/10/31
- Re: [Qemu-devel] pvpanic plans?, Paolo Bonzini, 2013/10/31
- Re: [Qemu-devel] pvpanic plans?, Michael S. Tsirkin, 2013/10/31
- Re: [Qemu-devel] pvpanic plans?,
Paolo Bonzini <=
- Re: [Qemu-devel] pvpanic plans?, Michael S. Tsirkin, 2013/10/31
- Re: [Qemu-devel] pvpanic plans?, Paolo Bonzini, 2013/10/31
- Re: [Qemu-devel] pvpanic plans?, Michael S. Tsirkin, 2013/10/31
- Re: [Qemu-devel] pvpanic plans?, Paolo Bonzini, 2013/10/31
- Re: [Qemu-devel] pvpanic plans?, Michael S. Tsirkin, 2013/10/31
- Re: [Qemu-devel] pvpanic plans?, Paolo Bonzini, 2013/10/31
- Re: [Qemu-devel] pvpanic plans?, Michael S. Tsirkin, 2013/10/31
- Re: [Qemu-devel] pvpanic plans?, Paolo Bonzini, 2013/10/31
- Re: [Qemu-devel] pvpanic plans?, Michael S. Tsirkin, 2013/10/31
- Re: [Qemu-devel] pvpanic plans?, Paolo Bonzini, 2013/10/31