qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Anthony Liguori <aliguori@us.ibm.com>
To: Kevin Wolf <kwolf@redhat.com>
Cc: Peter Maydell <peter.maydell@linaro.org>,
	Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>,
	Jan Kiszka <jan.kiszka@siemens.com>,
	qemu-devel@nongnu.org, Luiz Capitulino <lcapitulino@redhat.com>,
	Markus Armbruster <armbru@redhat.com>
Subject: Re: [Qemu-devel] [PATCH v3 00/20] qom: dynamic properties and	composition tree
Date: Fri, 16 Dec 2011 07:48:19 -0600	[thread overview]
Message-ID: <4EEB4C23.6030904@us.ibm.com> (raw)
In-Reply-To: <4EEB1277.4070803@redhat.com>

On 12/16/2011 03:42 AM, Kevin Wolf wrote:
> Am 15.12.2011 19:10, schrieb Anthony Liguori:
>> On 12/12/2011 02:29 PM, Anthony Liguori wrote:
>>> This is a follow up to my previous series to get us started in the QOM
>>> direction.  A few things are different this time around.  Most notably:
>>>
>>>    1) Devices no longer have names.  Instead, path names are always used to
>>>       identify devices.
>>>
>>>    2) In order to support (1), dynamic properties are now supported.
>>>
>>>    3) The concept of a "root device" has been introduced.  The root device is
>>>       roughly modelling the motherboard of a machine.  This type is a container
>>>       type and it's best to think of it as something like a PCB board I guess.
>>>
>>> See my other mail for a description of my merge plan for QOM.
>>
>> Applied.
>>
>> Regards,
>>
>> Anthony Liguori
>
> So you pushed this with qdev_property_add_child() calls spread all over
> the place instead of being treated like other properties? :-(

I would really like to be able to do something like:

struct ISASerial {
   Device parent;

   UART uart;
};

static Properties isa_serial_properties[] = {
     DEFINE_PROP_CHILD(ISASerial, uart),
     ...
};

Actually, I'd really like to do:

struct ISASerial {
    Device parent;

    UART _child uart;
};

And use an IDL compiler to generate the property table.

But at any rate, to get there, we need to first refactor the device model so 
that child devices are being created and registered in the device init 
functions.  We also need to a bunch more of the qom series so that we can do 
in-place initialization of objects.

Right now, a lot of things that are "children" are created in deep corners of 
the machine init often times long before the parent is ever created.

Regards,

Anthony Liguori

>
> Kevin
>

      parent reply	other threads:[~2011-12-16 13:48 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-12-12 20:29 [Qemu-devel] [PATCH v3 00/20] qom: dynamic properties and composition tree Anthony Liguori
2011-12-12 20:29 ` [Qemu-devel] [PATCH v3 01/20] qom: add a reference count to qdev objects Anthony Liguori
2011-12-12 20:29 ` [Qemu-devel] [PATCH v3 02/20] qom: add new dynamic property infrastructure based on Visitors (v2) Anthony Liguori
2011-12-12 20:29 ` [Qemu-devel] [PATCH v3 03/20] qom: register legacy properties as new style properties (v2) Anthony Liguori
2011-12-12 20:29 ` [Qemu-devel] [PATCH v3 04/20] qom: introduce root device Anthony Liguori
2011-12-12 20:29 ` [Qemu-devel] [PATCH v3 05/20] qdev: provide an interface to return canonical path from root (v2) Anthony Liguori
2011-12-12 20:29 ` [Qemu-devel] [PATCH v3 06/20] qdev: provide a path resolution (v2) Anthony Liguori
2011-12-12 20:29 ` [Qemu-devel] [PATCH v3 07/20] qom: add child properties (composition) (v3) Anthony Liguori
2011-12-12 20:29 ` [Qemu-devel] [PATCH v3 08/20] qom: add link properties (v2) Anthony Liguori
2011-12-12 20:29 ` [Qemu-devel] [PATCH v3 09/20] qapi: allow a 'gen' key to suppress code generation Anthony Liguori
2011-12-12 20:29 ` [Qemu-devel] [PATCH v3 10/20] qmp: add qom-list command Anthony Liguori
2011-12-12 20:29 ` [Qemu-devel] [PATCH v3 11/20] qom: qom_{get, set} monitor commands (v2) Anthony Liguori
2011-12-12 20:29 ` [Qemu-devel] [PATCH v3 12/20] qdev: add explicitly named devices to the root complex Anthony Liguori
2011-12-12 20:29 ` [Qemu-devel] [PATCH v3 13/20] dev: add an anonymous peripheral container Anthony Liguori
2011-12-12 20:29 ` [Qemu-devel] [PATCH v3 14/20] rtc: make piix3 set the rtc as a child (v2) Anthony Liguori
2011-12-12 20:29 ` [Qemu-devel] [PATCH v3 15/20] rtc: add a dynamic property for retrieving the date Anthony Liguori
2011-12-12 20:29 ` [Qemu-devel] [PATCH v3 16/20] qom: optimize qdev_get_canonical_path using a parent link Anthony Liguori
2011-12-12 20:29 ` [Qemu-devel] [PATCH v3 17/20] qom: add vga node to the pc composition tree Anthony Liguori
2011-12-12 20:29 ` [Qemu-devel] [PATCH v3 18/20] qom: add string property type Anthony Liguori
2011-12-12 20:29 ` [Qemu-devel] [PATCH v3 19/20] qdev: add a qdev_get_type() function and expose as a 'type' property Anthony Liguori
2011-12-15 18:10 ` [Qemu-devel] [PATCH v3 00/20] qom: dynamic properties and composition tree Anthony Liguori
2011-12-16  9:42   ` Kevin Wolf
2011-12-16 10:17     ` Paolo Bonzini
2011-12-16 10:36       ` Kevin Wolf
2011-12-16 12:24         ` Paolo Bonzini
2011-12-16 12:55           ` Kevin Wolf
2011-12-16 13:28             ` Paolo Bonzini
2011-12-16 13:52               ` Anthony Liguori
2011-12-16 13:51           ` Anthony Liguori
2011-12-16 13:52             ` Paolo Bonzini
2011-12-16 14:11               ` Anthony Liguori
2011-12-16 14:18             ` Kevin Wolf
2011-12-16 14:54               ` Anthony Liguori
2011-12-16 14:56                 ` Paolo Bonzini
2011-12-16 15:10                 ` Kevin Wolf
2011-12-16 13:49       ` Anthony Liguori
2011-12-16 13:48     ` Anthony Liguori [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4EEB4C23.6030904@us.ibm.com \
    --to=aliguori@us.ibm.com \
    --cc=armbru@redhat.com \
    --cc=jan.kiszka@siemens.com \
    --cc=kwolf@redhat.com \
    --cc=lcapitulino@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@linux.vnet.ibm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).