[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v6 10/10] migration: hmp: dump globals
From: |
Peter Xu |
Subject: |
Re: [Qemu-devel] [PATCH v6 10/10] migration: hmp: dump globals |
Date: |
Fri, 30 Jun 2017 14:22:16 +0800 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Wed, Jun 28, 2017 at 02:12:44PM -0300, Eduardo Habkost wrote:
> On Tue, Jun 27, 2017 at 12:10:19PM +0800, Peter Xu wrote:
> > Now we have some globals that can be configured for migration. Dump them
> > in HMP info migration for better debugging.
> >
> > (we can also use this to monitor whether COMPAT fields are applied
> > correctly on compatible machines)
> >
> > Reviewed-by: Juan Quintela <address@hidden>
> > Signed-off-by: Peter Xu <address@hidden>
> [...]
> > +void migration_global_dump(Monitor *mon)
> > +{
> > + MigrationState *ms = migrate_get_current();
> > +
> > + monitor_printf(mon, "globals: store-global-state=%d,
> > only_migratable=%d, "
> > + "send-configuration=%d, send-section-footer=%d\n",
> > + ms->store_global_state, ms->only_migratable,
> > + ms->send_configuration, ms->send_section_footer);
>
> I wouldn't call them "globals", I would just call them "properties" (or
> maybe just "settings"). They happen to be affected by global properties
> if the accel/machine/user sets them, but they are just plain old QOM
> properties.
Yes, "properties" seems better. Considering that the patch is already
merged, I am just afraid it's too trivial to send a single patch to
fix this, so I'll just temporarily skip this one. Anyway, please let
me know if you still want me to do this, I'd be glad to. Thanks,
--
Peter Xu
- [Qemu-devel] [PATCH v6 08/10] migration: move skip_section_footers, (continued)
[Qemu-devel] [PATCH v6 10/10] migration: hmp: dump globals, Peter Xu, 2017/06/27
[Qemu-devel] [PATCH v6 11/10] migration: add comment for TYPE_MIGRATE, Peter Xu, 2017/06/28