Peter Xu <pet...@redhat.com> writes:
> [Peter collected Eduardo's patch comment and formatted into patch]
>
> Suggested-by: Eduardo Habkost <ehabk...@redhat.com>
> Signed-off-by: Peter Xu <pet...@redhat.com>
> ---
> 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.
> */
> .parent = TYPE_DEVICE,
> .class_init = migration_class_init,
