On Mon, Jul 17, 2017 at 06:58:44PM +0200, Juan Quintela wrote: > Peter Xu <pet...@redhat.com> wrote: > > Do the same thing to migration capabilities, just like what we did in > > previous patch for migration parameters. > > > > Signed-off-by: Peter Xu <pet...@redhat.com> > > Reviewed-by: Juan Quintela <quint...@redhat.com>
Thanks! > > > A pitty that the preprocessor is not able to pass to uppercase ... > > > +#define DEFINE_PROP_MIG_CAP(name, x) \ > > + DEFINE_PROP_BOOL(name, MigrationState, enabled_capabilities[x], false) > > #define DEFINE_PROP_MIG_CAP(name) \ > DEFINE_PROP_BOOL(#name, MigrationState, > enabled_capabilities[MIGRATION_CAPABILITY_##name], false) Yes a pity, this is more beautiful (though this may let the grepping of specific MIGRATION_CAPABILITY_* slightly harder since it breaks the macro). -- Peter Xu