From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=35281 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Pn54q-0002wr-Pu for qemu-devel@nongnu.org; Wed, 09 Feb 2011 03:07:05 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Pn54p-0008Qo-Pk for qemu-devel@nongnu.org; Wed, 09 Feb 2011 03:07:04 -0500 Received: from fmmailgate02.web.de ([217.72.192.227]:48369) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Pn54p-0008Qj-CY for qemu-devel@nongnu.org; Wed, 09 Feb 2011 03:07:03 -0500 Message-ID: <4D524B25.80907@web.de> Date: Wed, 09 Feb 2011 09:07:01 +0100 From: Jan Kiszka MIME-Version: 1.0 References: <35fe72889a43ad4465c298a70069e9a6fc1eab66.1297077506.git.jan.kiszka@siemens.com> <20110208185932.GB13617@amt.cnet> In-Reply-To: <20110208185932.GB13617@amt.cnet> Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="------------enigF55D9DBCB4A86D1E11E08ECB" Sender: jan.kiszka@web.de Subject: [Qemu-devel] Re: [PATCH 04/15] Improve vm_stop reason declarations List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Marcelo Tosatti Cc: Avi Kivity , kvm@vger.kernel.org, qemu-devel@nongnu.org This is an OpenPGP/MIME signed message (RFC 2440 and 3156) --------------enigF55D9DBCB4A86D1E11E08ECB Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable 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; >> =20 >> - if (running || (reason !=3D EXCP_DEBUG && reason !=3D EXCP_INTERR= UPT) || >> - s->state =3D=3D RS_INACTIVE || s->state =3D=3D RS_SYSCALL) >> + if (running || (reason !=3D VMSTOP_DEBUG && reason !=3D VMSTOP_IN= TERRUPT) || >> + s->state =3D=3D RS_INACTIVE || s->state =3D=3D RS_SYSCALL) { >> return; >=20 > What about VMSTOP_USER ? >=20 > VMSTOP_INTERRUPT -> "the VM is stopped by an interrupt". >=20 > VMSTOP_USER -> "the VM is stopped by the user". Makes a lot of sense, will change. >=20 >> 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 =3D vm_running; >> =20 >> DPRINTF("done iterating\n"); >> - vm_stop(0); >> + vm_stop(VMSTOP_RWSTATE); >=20 > 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? Jan --------------enigF55D9DBCB4A86D1E11E08ECB Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.15 (GNU/Linux) Comment: Using GnuPG with SUSE - http://enigmail.mozdev.org/ iEYEARECAAYFAk1SSyUACgkQitSsb3rl5xSNswCfWtTHKOcXV70whJ6M0j8QY47E k1MAn121i3eXT44a/tozB6GcF1HPVvBF =nI68 -----END PGP SIGNATURE----- --------------enigF55D9DBCB4A86D1E11E08ECB--