[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/4] doc: update TYPE_MIGRATION documents
From: |
Peter Xu |
Subject: |
Re: [Qemu-devel] [PATCH 4/4] doc: update TYPE_MIGRATION documents |
Date: |
Tue, 4 Jul 2017 16:13:27 +0800 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Tue, Jul 04, 2017 at 10:04:01AM +0200, Markus Armbruster wrote:
> Peter Xu <address@hidden> writes:
>
> > [Peter collected Eduardo's patch comment and formatted into patch]
> >
> > Suggested-by: Eduardo Habkost <address@hidden>
> > Signed-off-by: Peter Xu <address@hidden>
> > ---
> > migration/migration.c | 15 +++++++++------
> > 1 file changed, 9 insertions(+), 6 deletions(-)
> >
> > diff --git a/migration/migration.c b/migration/migration.c
> > index dbdc121..6c64b99 100644
> > --- a/migration/migration.c
> > +++ b/migration/migration.c
> > @@ -2052,12 +2052,15 @@ static void migration_instance_init(Object *obj)
> > static const TypeInfo migration_type = {
> > .name = TYPE_MIGRATION,
> > /*
> > - * NOTE: "migration" itself is not really a device. We used
> > - * TYPE_DEVICE here only to leverage some existing QDev features
> > - * like "-global" properties, and HW_COMPAT_* fields (which are
> > - * finally applied as global properties as well). If one day the
> > - * global property feature can be migrated from QDev to QObject in
> > - * general, then we can switch to QObject as well.
> > + * NOTE:
> > + *
> > + * TYPE_MIGRATION is not really a device, as the object is not
> > + * created using qdev_create(), it is not attached to the qdev
> > + * device tree, and it is never realized.
> > + *
> > + * If one day we allow non-device QOM objects to use the global
> > + * property system, we can switch this from TYPE_DEVICE to
> > + * TYPE_OBJECT.
>
> Suggest to turn this paragraph into a TODO:
>
> * TODO Make this TYPE_OBJECT once QOM provides something like
> * TYPE_DEVICE's "-global" properties.
Sure. Will modify in next version. Thanks,
--
Peter Xu
- Re: [Qemu-devel] [PATCH 3/4] doc: add item for "-M enforce-config-section", (continued)
[Qemu-devel] [PATCH 4/4] doc: update TYPE_MIGRATION documents, Peter Xu, 2017/07/02
Re: [Qemu-devel] [PATCH 0/4] migration: fix iotest 055, only-migratable break, QingFeng Hao, 2017/07/03