All of lore.kernel.org
 help / color / mirror / Atom feed
From: Markus Armbruster <armbru@redhat.com>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: 'Паша' <Pavel.Dovgaluk@ispras.ru>,
	"Andreas Färber" <afaerber@suse.de>,
	"Natalia Fursova" <Natalia.Fursova@ispras.ru>,
	qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] qgraph
Date: Tue, 11 Jun 2019 15:39:58 +0200	[thread overview]
Message-ID: <87ftogp7f5.fsf@dusky.pond.sub.org> (raw)
In-Reply-To: <8d391b41-bf6d-b83b-7b22-25fefa18c518@redhat.com> (Paolo Bonzini's message of "Tue, 11 Jun 2019 12:31:24 +0200")

Paolo Bonzini <pbonzini@redhat.com> writes:

> On 11/06/19 10:56, Markus Armbruster wrote:
>> Yes, this is how introspection (both QMP and QOM) is commonly used.
>> Just keep in mind one difference: QMP is static, QOM is dynamic.
>> 
>> QMP being static means it's defined at compile time.  So is the value of
>> query-qmp-schema.  Same QEMU build, same value.  This permits caching.
>> 
>> QOM being dynamic means to introspect an object's properties, you have
>> to create it.  Worse, an object's properties may (in theory) change at
>> any time.  *Properties*, not just property *values*.  In practice, I'd
>> expect properties to change only at realize time.
>
> Right, and we should move more towards class-based properties so that
> the dynamic nature of QOM is only used for the bare minimum needed (e.g.
> memory regions).

What are we doing to make new code conform to that?

What are we doing to update existing code?


  reply	other threads:[~2019-06-11 13:41 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <5cf62de9.1c69fb81.66fc.8f4fSMTPIN_ADDED_BROKEN@mx.google.com>
2019-06-04 15:06 ` [Qemu-devel] qgraph Paolo Bonzini
2019-06-05 12:34   ` Natalia Fursova
     [not found]   ` <5cf7b6e6.1c69fb81.1cdca.e260SMTPIN_ADDED_BROKEN@mx.google.com>
2019-06-05 13:07     ` Paolo Bonzini
2019-06-05 14:22       ` Natalia Fursova
2019-06-10  9:53       ` Natalia Fursova
2019-06-10 11:57         ` Andreas Färber
2019-06-10 12:03           ` Paolo Bonzini
2019-06-10 13:28             ` Andreas Färber
2019-06-10 13:52               ` Paolo Bonzini
2019-06-10 16:12                 ` Andreas Färber
2019-06-10 16:18                   ` Paolo Bonzini
2019-06-11  8:56                     ` Markus Armbruster
2019-06-11 10:31                       ` Paolo Bonzini
2019-06-11 13:39                         ` Markus Armbruster [this message]
2019-06-11 13:44                           ` Paolo Bonzini
2019-07-02 11:44                             ` Natalia Fursova
2019-07-02 15:26                               ` Markus Armbruster
2019-07-03  8:19                                 ` Natalia Fursova
     [not found]                             ` <5d1b4524.1c69fb81.ddba5.77bdSMTPIN_ADDED_BROKEN@mx.google.com>
2019-07-02 11:54                               ` Peter Maydell
2019-06-11 13:51                         ` Andreas Färber
2019-06-04  8:37 Natalia Fursova

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=87ftogp7f5.fsf@dusky.pond.sub.org \
    --to=armbru@redhat.com \
    --cc=Natalia.Fursova@ispras.ru \
    --cc=Pavel.Dovgaluk@ispras.ru \
    --cc=afaerber@suse.de \
    --cc=pbonzini@redhat.com \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.