All of lore.kernel.org
 help / color / mirror / Atom feed
From: Peter Xu <peterx@redhat.com>
To: Fabiano Rosas <farosas@suse.de>
Cc: qemu-devel@nongnu.org,
	"Marc-André Lureau" <marcandre.lureau@redhat.com>,
	"Juraj Marcin" <jmarcin@redhat.com>,
	"Vladimir Sementsov-Ogievskiy" <vsementsov@yandex-team.ru>,
	"Daniel P. Berrangé" <berrange@redhat.com>,
	"Sana Sharma" <sansshar@redhat.com>,
	"Eric Blake" <eblake@redhat.com>,
	"Markus Armbruster" <armbru@redhat.com>,
	"Kevin Wolf" <kwolf@redhat.com>,
	"Paolo Bonzini" <pbonzini@redhat.com>,
	"Philippe Mathieu-Daudé" <philmd@mailo.com>,
	"Mark Cave-Ayland" <mark.caveayland@nutanix.com>,
	"Akihiko Odaki" <odaki@rsg.ci.i.u-tokyo.ac.jp>,
	"Cédric Le Goater" <clg@redhat.com>,
	"Dr . David Alan Gilbert" <dave@treblig.org>
Subject: Re: [PATCH 1/4] qdev: Pave way for exporting Property to be used in non-qdev
Date: Fri, 5 Jun 2026 11:40:17 -0400	[thread overview]
Message-ID: <aiLt4TZ3s8GiTwJP@x1.local> (raw)
In-Reply-To: <87fr316ogx.fsf@suse.de>

On Fri, Jun 05, 2026 at 11:39:26AM -0300, Fabiano Rosas wrote:
> Peter Xu <peterx@redhat.com> writes:
> 
> > Property itself is a well defined interface to either support smooth
> > conversions to Object's properties, or supports global properties.  However
> > currently it's tied to DeviceClass, aka, qdev. So non-qdev cannot use
> > Property list.
> >
> > My current observation shows Property is almost ready to be used as a
> > separated exported interface, except two small things that may need touch
> > up internally:
> >
> >         qdev_prop_allow_set
> 
> Just a thought, we could use something like this to block update of
> migration parameters during migration runtime.

Currently it may not achieve it (with qdev property or object property that
I'm trying), it's because qmp set parameters currently bypasses qom
properties.

We need to switch QMP set parameters to use qom set to achieve it, but yes,
that sounds like something good to consider in the future to merge the two
paths somehow.

If we use helpers like object_property_add_uint64_ptr() we save lines but
lose control over setter() blocking updates during live migration.  If we
use raw object_property_add() we get control but adds boilerplate lines.

Anyway, sounds like something for the future..

-- 
Peter Xu



  reply	other threads:[~2026-06-05 15:40 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-06-04 23:11 [PATCH 0/4] migration/qdev: Remove TYPE_DEVICE dependency on migration object Peter Xu
2026-06-04 23:11 ` [PATCH 1/4] qdev: Pave way for exporting Property to be used in non-qdev Peter Xu
2026-06-05 10:53   ` Mark Cave-Ayland
2026-06-05 15:18     ` Fabiano Rosas
2026-06-05 14:39   ` Fabiano Rosas
2026-06-05 15:40     ` Peter Xu [this message]
2026-06-04 23:11 ` [PATCH 2/4] qdev: Introduce helper object_apply_globals() Peter Xu
2026-06-05  8:52   ` Daniel P. Berrangé
2026-06-05 14:44     ` Fabiano Rosas
2026-06-05 14:47       ` Daniel P. Berrangé
2026-06-05 15:06         ` Fabiano Rosas
2026-06-04 23:11 ` [PATCH 3/4] qdev: Refactor and rename of qdev_class_add_property() Peter Xu
2026-06-05  8:43   ` Daniel P. Berrangé
2026-06-05 15:21     ` Peter Xu
2026-06-04 23:11 ` [PATCH 4/4] migration: Remove dependency to TYPE_DEVICE Peter Xu
2026-06-05  8:30   ` Marc-André Lureau
2026-06-05  8:49     ` Daniel P. Berrangé
2026-06-05 15:36       ` Peter Xu

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=aiLt4TZ3s8GiTwJP@x1.local \
    --to=peterx@redhat.com \
    --cc=armbru@redhat.com \
    --cc=berrange@redhat.com \
    --cc=clg@redhat.com \
    --cc=dave@treblig.org \
    --cc=eblake@redhat.com \
    --cc=farosas@suse.de \
    --cc=jmarcin@redhat.com \
    --cc=kwolf@redhat.com \
    --cc=marcandre.lureau@redhat.com \
    --cc=mark.caveayland@nutanix.com \
    --cc=odaki@rsg.ci.i.u-tokyo.ac.jp \
    --cc=pbonzini@redhat.com \
    --cc=philmd@mailo.com \
    --cc=qemu-devel@nongnu.org \
    --cc=sansshar@redhat.com \
    --cc=vsementsov@yandex-team.ru \
    /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.