From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:60585) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RcoBn-0007z9-Jm for qemu-devel@nongnu.org; Mon, 19 Dec 2011 20:08:20 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1RcoBl-0006ao-HY for qemu-devel@nongnu.org; Mon, 19 Dec 2011 20:08:19 -0500 Received: from mail-gy0-f173.google.com ([209.85.160.173]:49585) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RcoBl-0006ae-Dr for qemu-devel@nongnu.org; Mon, 19 Dec 2011 20:08:17 -0500 Received: by ghbg16 with SMTP id g16so221521ghb.4 for ; Mon, 19 Dec 2011 17:08:17 -0800 (PST) Message-ID: <4EEFDFFE.6000402@codemonkey.ws> Date: Mon, 19 Dec 2011 19:08:14 -0600 From: Anthony Liguori MIME-Version: 1.0 References: <20111219211737.GA17469@amt.cnet> <4EEFB9AE.7050309@codemonkey.ws> <4EEFCD71.5040603@web.de> <4EEFD7A9.3050007@codemonkey.ws> <4EEFD8D1.3060707@web.de> In-Reply-To: <4EEFD8D1.3060707@web.de> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit 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: Jan Kiszka Cc: Lai Jiangshan , kvm@vger.kernel.org, "Michael S. Tsirkin" , Marcelo Tosatti , qemu-devel , Blue Swirl , Avi Kivity On 12/19/2011 06:37 PM, Jan Kiszka wrote: > 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) on >>> creation. I'm not sure how to model two classes that will still only >>> mean a single qdev registration. >> >> See other reply in thread. >> >> We should model this as two separate qdev devices. We can avoid >> regressing migration in qemu-kvm by just having a common vmstate name. >> >> 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. Let me say that I know this is the last bit of qemu-kvm that needs merging and that this has been an epic effort. I wouldn't refuse to merge a pull request that came in with this in its current form. If we merged this now, I would be submitting patches in the not too distant future to remove all of this backend stuff in favor of proper modeling (including using two separate devices). There's lot of inconsistency in qdev already today so adding a little more isn't the end of the world. We're going to need to eventually have this debate soon so it's up to you whether you want to just get this merged now and worry about this another day or resolve this before merge. I don't see any compatibility issues here so I'm not really concerned about introducing a regression by breaking it into two devices. Regards, Anthony Liguori > > Jan >