From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1Ne85e-0003qY-Mr for qemu-devel@nongnu.org; Sun, 07 Feb 2010 09:26:22 -0500 Received: from [199.232.76.173] (port=38516 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Ne85e-0003qF-4Z for qemu-devel@nongnu.org; Sun, 07 Feb 2010 09:26:22 -0500 Received: from Debian-exim by monty-python.gnu.org with spam-scanned (Exim 4.60) (envelope-from ) id 1Ne85c-00072B-Jw for qemu-devel@nongnu.org; Sun, 07 Feb 2010 09:26:21 -0500 Received: from fmmailgate03.web.de ([217.72.192.234]:44335) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1Ne85b-00071W-92 for qemu-devel@nongnu.org; Sun, 07 Feb 2010 09:26:19 -0500 Message-ID: <4B6ECD80.6090407@web.de> Date: Sun, 07 Feb 2010 15:26:08 +0100 From: Jan Kiszka MIME-Version: 1.0 References: <4822161334c3e10d7772dbd08dafdd3a78c86ce4.1265187223.git.jan.kiszka@siemens.com> <4B6EC180.7000203@redhat.com> <4B6EC557.9090804@web.de> <4B6EC6EC.4020509@redhat.com> In-Reply-To: <4B6EC6EC.4020509@redhat.com> Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="------------enig9AD18C6AA326E5C1D7EC8B88" Sender: jan.kiszka@web.de Subject: [Qemu-devel] Re: [PATCH v2 14/21] qemu-kvm: Rework VCPU state writeback API List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Avi Kivity Cc: Anthony Liguori , kvm@vger.kernel.org, Glauber Costa , Marcelo Tosatti , qemu-devel@nongnu.org, Alexander Graf This is an OpenPGP/MIME signed message (RFC 2440 and 3156) --------------enig9AD18C6AA326E5C1D7EC8B88 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Avi Kivity wrote: > On 02/07/2010 03:51 PM, Jan Kiszka wrote: >> Avi Kivity wrote: >> =20 >>> On 02/03/2010 10:53 AM, Jan Kiszka wrote: >>> =20 >>>> This grand cleanup drops all reset and vmsave/load related >>>> synchronization points in favor of four(!) generic hooks: >>>> >>>> - cpu_synchronize_all_states in qemu_savevm_state_complete >>>> (initial sync from kernel before vmsave) >>>> - cpu_synchronize_all_post_init in qemu_loadvm_state >>>> (writeback after vmload) >>>> - cpu_synchronize_all_post_init in main after machine init >>>> - cpu_synchronize_all_post_reset in qemu_system_reset >>>> (writeback after system reset) >>>> >>>> These writeback points + the existing one of VCPU exec after >>>> cpu_synchronize_state map on three levels of writeback: >>>> >>>> - KVM_PUT_ASYNC_STATE (during runtime, other VCPUs continue to run) >>>> >>>> =20 >>> Wouldn't that be SYNC_STATE (state that is modified by the current vc= pu >>> only)? >>> =20 >> It's async /wrt other VCPUs. They continue to run and may interact wit= h >> this VCPU while updating its state. >> =20 >=20 > Well, to me it makes more sense to name them from the point of view of > the vcpu that is doing the update. I'm open for a better name - except for "sync" as writebacks are always synchronous from the POV of the modified VCPU. Is KVM_PUT_RUNTIME_STATE clearer? Jan --------------enig9AD18C6AA326E5C1D7EC8B88 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.9 (GNU/Linux) Comment: Using GnuPG with SUSE - http://enigmail.mozdev.org iEYEARECAAYFAktuzYUACgkQitSsb3rl5xTkMgCbB53E+rQ/IagBs1W0UoJvZtva X5sAn1J+CY8vLFK32rk4QLLvh0xbOZ53 =joxY -----END PGP SIGNATURE----- --------------enig9AD18C6AA326E5C1D7EC8B88--