From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:53690) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RcniB-0001X8-FZ for qemu-devel@nongnu.org; Mon, 19 Dec 2011 19:37:44 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1RcniA-0002IX-DF for qemu-devel@nongnu.org; Mon, 19 Dec 2011 19:37:43 -0500 Received: from fmmailgate04.web.de ([217.72.192.242]:58027) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RcniA-0002IQ-0D for qemu-devel@nongnu.org; Mon, 19 Dec 2011 19:37:42 -0500 Received: from moweb002.kundenserver.de (moweb002.kundenserver.de [172.19.20.108]) by fmmailgate04.web.de (Postfix) with ESMTP id 1E4636FDD944 for ; Tue, 20 Dec 2011 01:37:41 +0100 (CET) Message-ID: <4EEFD8D1.3060707@web.de> Date: Tue, 20 Dec 2011 01:37:37 +0100 From: Jan Kiszka MIME-Version: 1.0 References: <20111219211737.GA17469@amt.cnet> <4EEFB9AE.7050309@codemonkey.ws> <4EEFCD71.5040603@web.de> <4EEFD7A9.3050007@codemonkey.ws> In-Reply-To: <4EEFD7A9.3050007@codemonkey.ws> Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="------------enig2C23A3E644571501918D702F" Subject: Re: [Qemu-devel] [PATCH v5 00/16] uq/master: Introduce basic irqchip support List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Anthony Liguori Cc: Lai Jiangshan , kvm@vger.kernel.org, "Michael S. Tsirkin" , Marcelo Tosatti , qemu-devel , Blue Swirl , Avi Kivity This is an OpenPGP/MIME signed message (RFC 2440 and 3156) --------------enig2C23A3E644571501918D702F Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable On 2011-12-20 01:32, Anthony Liguori wrote: > On 12/19/2011 05:49 PM, Jan Kiszka wrote: >> On 2011-12-19 23:24, Anthony Liguori wrote: >>> On 12/19/2011 03:17 PM, Marcelo Tosatti wrote: >>>> >>>> Anthony, >>>> >>>> Can you please review& ACK? >>>> >>>> You could even apply directly but well do a kvm-autotest run through= >>>> uq/master. Still, your review is needed. >>> >>> Overall, it looks good except for the backend/frontend split. This >>> should be done in terms of qdev inheritance. >> >> I cannot follow your idea here yet. There is no inheritance as we end = up >> with only a single class that permutes (selects a different backend) o= n >> creation. I'm not sure how to model two classes that will still only >> mean a single qdev registration. >=20 > See other reply in thread. >=20 > We should model this as two separate qdev devices. We can avoid > regressing migration in qemu-kvm by just having a common vmstate name. >=20 > apic is a no-user device so there's no way that changing the name of it= > in qemu-kvm can affect users. Look down http://thread.gmane.org/gmane.comp.emulators.kvm.devel/82598 for the discussion of that model. Jan --------------enig2C23A3E644571501918D702F 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.16 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/ iEYEARECAAYFAk7v2NEACgkQitSsb3rl5xTS6gCeOIB6S9OFcpssdGLchZFdjggi e3AAn301nFOyDmbp/Ak6NuSM7JVokRZd =sHSo -----END PGP SIGNATURE----- --------------enig2C23A3E644571501918D702F--