[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH 04/15] Improve vm_stop reason declarations
From: |
Marcelo Tosatti |
Subject: |
[Qemu-devel] Re: [PATCH 04/15] Improve vm_stop reason declarations |
Date: |
Wed, 9 Feb 2011 12:17:00 -0200 |
User-agent: |
Mutt/1.5.20 (2009-08-17) |
On Wed, Feb 09, 2011 at 09:07:01AM +0100, Jan Kiszka wrote:
> On 2011-02-08 19:59, Marcelo Tosatti wrote:
> > On Mon, Feb 07, 2011 at 12:19:15PM +0100, Jan Kiszka wrote:
> >> index d6556c9..3397566 100644
> >> --- a/gdbstub.c
> >> +++ b/gdbstub.c
> >> @@ -2194,14 +2194,14 @@ static void gdb_vm_state_change(void *opaque, int
> >> running, int reason)
> >> const char *type;
> >> int ret;
> >>
> >> - if (running || (reason != EXCP_DEBUG && reason != EXCP_INTERRUPT) ||
> >> - s->state == RS_INACTIVE || s->state == RS_SYSCALL)
> >> + if (running || (reason != VMSTOP_DEBUG && reason != VMSTOP_INTERRUPT)
> >> ||
> >> + s->state == RS_INACTIVE || s->state == RS_SYSCALL) {
> >> return;
> >
> > What about VMSTOP_USER ?
> >
> > VMSTOP_INTERRUPT -> "the VM is stopped by an interrupt".
> >
> > VMSTOP_USER -> "the VM is stopped by the user".
>
> Makes a lot of sense, will change.
>
> >
> >> diff --git a/migration.c b/migration.c
> >> index 3612572..20ea113 100644
> >> --- a/migration.c
> >> +++ b/migration.c
> >> @@ -378,7 +378,7 @@ void migrate_fd_put_ready(void *opaque)
> >> int old_vm_running = vm_running;
> >>
> >> DPRINTF("done iterating\n");
> >> - vm_stop(0);
> >> + vm_stop(VMSTOP_RWSTATE);
> >
> > VMSTOP_RWSTATE is cryptic. What about VMSTOP_SAVEVM, MIGRATE, etc.
>
> Both alternatives are not completely accurate as we also stop of vmload.
> What about VMSTOP_SYNC_STATE or UPDATE_STATE?
IMO it would be better to state the specific reason why the vm stopped,
not some generic action. VMSTOP_SAVEVM, VMSTOP_VMLOAD and
VMSTOP_MIGRATE.
- [Qemu-devel] [PATCH 10/15] kvm: x86: Catch and report failing IRQ and NMI injections, (continued)
- [Qemu-devel] [PATCH 10/15] kvm: x86: Catch and report failing IRQ and NMI injections, Jan Kiszka, 2011/02/07
- [Qemu-devel] [PATCH 03/15] Fix a few coding style violations in cpus.c, Jan Kiszka, 2011/02/07
- [Qemu-devel] [PATCH 01/15] Refactor kvm&tcg function names in cpus.c, Jan Kiszka, 2011/02/07
- [Qemu-devel] [PATCH 08/15] kvm: x86: Prepare VCPU loop for in-kernel irqchip, Jan Kiszka, 2011/02/07
- [Qemu-devel] [PATCH 09/15] kvm: Drop return values from kvm_arch_pre/post_run, Jan Kiszka, 2011/02/07
- [Qemu-devel] [PATCH 04/15] Improve vm_stop reason declarations, Jan Kiszka, 2011/02/07
- [Qemu-devel] [PATCH v2 04/15] Improve vm_stop reason declarations, Jan Kiszka, 2011/02/09
[Qemu-devel] [PATCH 12/15] Introduce log_start/log_stop in CPUPhysMemoryClient, Jan Kiszka, 2011/02/07
[Qemu-devel] [PATCH 06/15] Move debug exception handling out of cpu_exec, Jan Kiszka, 2011/02/07
[Qemu-devel] [PATCH 15/15] kvm: x86: Introduce kvmclock device to save/restore its state, Jan Kiszka, 2011/02/07