From patchwork Wed Aug 3 15:17:19 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Luiz Capitulino X-Patchwork-Id: 108298 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from lists.gnu.org (lists.gnu.org [140.186.70.17]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id 7FB4AB71D7 for ; Thu, 4 Aug 2011 02:07:17 +1000 (EST) Received: from localhost ([::1]:36903 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QodDo-0008PQ-Ta for incoming@patchwork.ozlabs.org; Wed, 03 Aug 2011 11:19:00 -0400 Received: from eggs.gnu.org ([140.186.70.92]:39728) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QodDS-0005mj-VA for qemu-devel@nongnu.org; Wed, 03 Aug 2011 11:18:56 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1QodCd-0006aK-RC for qemu-devel@nongnu.org; Wed, 03 Aug 2011 11:17:52 -0400 Received: from mx1.redhat.com ([209.132.183.28]:2395) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QodCd-0006a5-BF for qemu-devel@nongnu.org; Wed, 03 Aug 2011 11:17:47 -0400 Received: from int-mx01.intmail.prod.int.phx2.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id p73FHjJq008081 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK); Wed, 3 Aug 2011 11:17:45 -0400 Received: from localhost (ovpn-113-124.phx2.redhat.com [10.3.113.124]) by int-mx01.intmail.prod.int.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id p73FHifM024703; Wed, 3 Aug 2011 11:17:45 -0400 From: Luiz Capitulino To: qemu-devel@nongnu.org Date: Wed, 3 Aug 2011 12:17:19 -0300 Message-Id: <1312384643-581-4-git-send-email-lcapitulino@redhat.com> In-Reply-To: <1312384643-581-1-git-send-email-lcapitulino@redhat.com> References: <1312384643-581-1-git-send-email-lcapitulino@redhat.com> X-Scanned-By: MIMEDefang 2.67 on 10.5.11.11 X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 209.132.183.28 Cc: blauwirbel@gmail.com, amit.shah@redhat.com, aliguori@us.ibm.com, jan.kiszka@web.de, avi@redhat.com Subject: [Qemu-devel] [PATCH 3/7] QemuState: Add additional states X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Sender: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Currently, only vm_start() and vm_stop() change the VM state. That's, the state is only changed when starting or stopping the VM. This commit adds the qemu_state_set() function, making it possible to also do state transitions when qemu is stopped or running. Additional states are also added and the current state is stored. This is going to be used by the next commits. Signed-off-by: Luiz Capitulino --- cpus.c | 1 + migration.c | 8 +++++++- sysemu.h | 10 +++++++++- vl.c | 20 ++++++++++++++++++++ 4 files changed, 37 insertions(+), 2 deletions(-) diff --git a/cpus.c b/cpus.c index ebbb8b9..48e6ca1 100644 --- a/cpus.c +++ b/cpus.c @@ -124,6 +124,7 @@ static void do_vm_stop(QemuState state) cpu_disable_ticks(); vm_running = 0; pause_all_vcpus(); + qemu_state_set(state); vm_state_notify(0, state); qemu_aio_flush(); bdrv_flush_all(); diff --git a/migration.c b/migration.c index 9724ce0..8aacf64 100644 --- a/migration.c +++ b/migration.c @@ -72,8 +72,11 @@ void process_incoming_migration(QEMUFile *f) incoming_expected = false; - if (autostart) + if (autostart) { vm_start(); + } else { + qemu_state_set(QSTATE_PRELAUNCH); + } } int do_migrate(Monitor *mon, const QDict *qdict, QObject **ret_data) @@ -394,6 +397,9 @@ void migrate_fd_put_ready(void *opaque) } state = MIG_STATE_ERROR; } + if (state == MIG_STATE_COMPLETED) { + qemu_state_set(QSTATE_POSTMIGRATE); + } s->state = state; notifier_list_notify(&migration_state_notifiers, NULL); } diff --git a/sysemu.h b/sysemu.h index 2c3ea3d..32c9abb 100644 --- a/sysemu.h +++ b/sysemu.h @@ -11,16 +11,22 @@ /* vl.c */ typedef enum { + QSTATE_NOSTATE, QSTATE_DEBUG, /* qemu is running under gdb */ + QSTATE_INMIGRATE, /* paused waiting for an incoming migration */ QSTATE_INTERROR, /* paused due to an internal error */ QSTATE_IOERROR, /* paused due to an I/O error */ QSTATE_PAUSED, /* paused by the user (ie. the 'stop' command) */ + QSTATE_POSTMIGRATE, /* paused following a successful migration */ + QSTATE_PRELAUNCH, /* qemu was started with -S and haven't started */ QSTATE_PREMIGRATE, /* paused preparing to finish migrate */ QSTATE_RESTVM, /* paused restoring the VM state */ + QSTATE_RESTVMFAILED, /* paused due to a failed attempt to load state */ QSTATE_RUNNING, /* qemu is running */ QSTATE_SAVEVM, /* paused saving VM state */ QSTATE_SHUTDOWN, /* guest shut down and -no-shutdown is in use */ - QSTATE_WATCHDOG /* watchdog fired and qemu is configured to pause */ + QSTATE_WATCHDOG, /* watchdog fired and qemu is configured to pause */ + QSTATE_MAX } QemuState; extern const char *bios_name; @@ -31,6 +37,8 @@ extern uint8_t qemu_uuid[]; int qemu_uuid_parse(const char *str, uint8_t *uuid); #define UUID_FMT "%02hhx%02hhx%02hhx%02hhx-%02hhx%02hhx-%02hhx%02hhx-%02hhx%02hhx-%02hhx%02hhx%02hhx%02hhx%02hhx%02hhx" +QemuState qemu_state_get(void); +void qemu_state_set(QemuState state); typedef struct vm_change_state_entry VMChangeStateEntry; typedef void VMChangeStateHandler(void *opaque, int running, QemuState state); diff --git a/vl.c b/vl.c index faa7c5f..2619c8e 100644 --- a/vl.c +++ b/vl.c @@ -320,6 +320,22 @@ static int default_driver_check(QemuOpts *opts, void *opaque) } /***********************************************************/ +/* QEMU state */ + +static QemuState qemu_current_state = QSTATE_NOSTATE; + +QemuState qemu_state_get(void) +{ + return qemu_current_state; +} + +void qemu_state_set(QemuState state) +{ + assert(state < QSTATE_MAX); + qemu_current_state = state; +} + +/***********************************************************/ /* real time host monotonic timer */ /***********************************************************/ @@ -1158,6 +1174,7 @@ void vm_start(void) if (!vm_running) { cpu_enable_ticks(); vm_running = 1; + qemu_state_set(QSTATE_RUNNING); vm_state_notify(1, QSTATE_RUNNING); resume_all_vcpus(); monitor_protocol_event(QEVENT_RESUME, NULL); @@ -3341,6 +3358,7 @@ int main(int argc, char **argv, char **envp) } if (incoming) { + qemu_state_set(QSTATE_INMIGRATE); int ret = qemu_start_incoming_migration(incoming); if (ret < 0) { fprintf(stderr, "Migration failed. Exit code %s(%d), exiting.\n", @@ -3349,6 +3367,8 @@ int main(int argc, char **argv, char **envp) } } else if (autostart) { vm_start(); + } else { + qemu_state_set(QSTATE_PRELAUNCH); } os_setup_post();