From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:38665) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RXJCG-0000pZ-Pb for qemu-devel@nongnu.org; Sun, 04 Dec 2011 16:02:05 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1RXJCF-0005jw-C7 for qemu-devel@nongnu.org; Sun, 04 Dec 2011 16:02:04 -0500 Received: from mail-yx0-f173.google.com ([209.85.213.173]:53187) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RXJCF-0005js-9X for qemu-devel@nongnu.org; Sun, 04 Dec 2011 16:02:03 -0500 Received: by yenm6 with SMTP id m6so387560yen.4 for ; Sun, 04 Dec 2011 13:02:02 -0800 (PST) Message-ID: <4EDBDFC7.3090206@codemonkey.ws> Date: Sun, 04 Dec 2011 15:01:59 -0600 From: Anthony Liguori MIME-Version: 1.0 References: <4ED98C06.5040405@codemonkey.ws> <4EDA3104.8030304@redhat.com> <4EDA95ED.6030706@codemonkey.ws> In-Reply-To: <4EDA95ED.6030706@codemonkey.ws> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v2 00/18] qom: dynamic properties and composition tree (v2) List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini Cc: qemu-devel@nongnu.org On 12/03/2011 03:34 PM, Anthony Liguori wrote: > On 12/03/2011 08:24 AM, Paolo Bonzini wrote: >> On 12/03/2011 03:40 AM, Anthony Liguori wrote: >>> That is still true. The next step, inheritance, will pull the properties >>> into a base class. That base class can be used elsewhere outside of the >>> device model. >>> >>> But this is already a 20 patch series. If you want all of that in one >>> series, it's going to be 100 patches that are not terribly easy to >>> review at once. >> >> Without a design document and a roadmap, however, it's impossible to try to >> understand how the pieces will be together. 100 patches may require some time to >> digest, but 20 patches require a crystal ball to figure out what's ahead. > > You can see a bit further by looking at: > > https://github.com/aliguori/qemu/commits/qom-next > > That fills out the composition tree pretty well for the pc. The next step is > aggressive refactoring such that the qdev objects reflect the composition. IOW, > we should create the rtc from within the piix3 initialization function. I've begun the work of introducing proper inheritance. There's a lot going on but the basic idea is: 1) introduce QOM base type (Object), make qdev inherit from it 2) create a dynamic typeinfo based DeviceInfo, make device class point to deviceinfo 3) model qdev hierarchy in QOM 4) starting from the bottom of the hierarchy, remove DeviceInfo subclass and push that functionality into QOM classes 5) once (4) is complete, remove DeviceInfo 6) refactor any use of multiple child busses into separate devices with one bus 7) refactor busstate as an interface 8) refactor device model to make more aggressive use of composition 9) refactor life cycle events into virtual methods The tree I've posted is on step (4). Regards, Anthony Liguori