qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Paolo Bonzini <pbonzini@redhat.com>
To: "Andreas Färber" <afaerber@suse.de>, qemu-devel@nongnu.org
Cc: peter.maydell@linaro.org, armbru@redhat.com, aliguori@amazon.com,
	lcapitulino@redhat.com
Subject: Re: [Qemu-devel] [PATCH] scripts: Add qom-tree script as modern equivalent of info qtree
Date: Fri, 07 Feb 2014 14:06:02 +0100	[thread overview]
Message-ID: <52F4DA3A.7010700@redhat.com> (raw)
In-Reply-To: <52F4D52F.3020004@suse.de>

Il 07/02/2014 13:44, Andreas Färber ha scritto:
> Am 07.02.2014 12:21, schrieb Paolo Bonzini:
>> Let's stop talking about theory and let's look at the actual ccode, please.
>
> I have posted actual patches, you haven't.

I have reviewed those, and said that we can apply all three.  It's 
certainly better than reverting.  That doesn't mean that keeping broken 
code would have been better than reverting.  And let me repeat that 
*reverting a broken patch should always be one of the alternatives*.

But my request to "look at the actual code" was not related to 
contribution of patches.  It referred to _all_ of QEMU device hierarchy. 
Your assertion that "qdev is dead" seems quite an exaggeration; having 
contributed quite a few patches to "kill" qdev-specific mechanisms in 
favor of generic ones, it seems very much alive to me.

Let's look at qdev.  Ask ourselves what useful functionality of Device 
has nothing to do with devices.  Ask ourselves where it clashes with the 
design of Object, and which of the two is better.  Make a design that is 
consistent with what we need, not with a generic 2-year old vision that 
sometimes borders on dogma.  Then we can write code.

This is all totally unrelated from which interesting relationships are 
useful to extract and visualize from the QOM tree---and my point there 
is that both parent-child ("qom-tree") and controller-controlled ("info 
qtree") are useful relationships.

Paolo

  reply	other threads:[~2014-02-07 13:06 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-02-05 17:35 [Qemu-devel] [PATCH] scripts: Add qom-tree script as modern equivalent of info qtree Andreas Färber
2014-02-05 17:48 ` Paolo Bonzini
2014-02-05 17:51   ` Andreas Färber
2014-02-05 17:55     ` Paolo Bonzini
2014-02-05 18:01       ` Andreas Färber
2014-02-07  7:13         ` Paolo Bonzini
2014-02-07 11:09           ` Andreas Färber
2014-02-07 11:21             ` Paolo Bonzini
2014-02-07 12:44               ` Andreas Färber
2014-02-07 13:06                 ` Paolo Bonzini [this message]
2014-02-07 13:48                   ` Andreas Färber
2014-02-07 14:18                     ` Paolo Bonzini
2014-02-07 15:00                     ` Markus Armbruster
2014-02-07 12:49               ` Andreas Färber
2014-02-07 13:16                 ` Paolo Bonzini
2014-02-05 17:56     ` Andreas Färber
2014-02-05 17:58       ` Paolo Bonzini
2014-02-05 18:08         ` Andreas Färber
2014-02-07 10:41           ` Paolo Bonzini
2014-02-07 11:00             ` Andreas Färber
2014-02-07 11:10               ` Paolo Bonzini
2014-02-05 18:24 ` Eric Blake
2014-02-05 18:39   ` Andreas Färber

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=52F4DA3A.7010700@redhat.com \
    --to=pbonzini@redhat.com \
    --cc=afaerber@suse.de \
    --cc=aliguori@amazon.com \
    --cc=armbru@redhat.com \
    --cc=lcapitulino@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    /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).