Vladimir Sementsov-Ogievskiy <[email protected]> wrote: > On 02.03.23 19:33, Juan Quintela wrote: >> Once that we are there, we rename the function to migrate_compress() >> to be consistent with all other capabilities. >> Signed-off-by: Juan Quintela <[email protected]> > > Reviewed-by: Vladimir Sementsov-Ogievskiy <[email protected]> > >> --- >> migration/migration.h | 1 - >> migration/options.h | 1 + >> migration/migration.c | 11 +---------- >> migration/options.c | 9 +++++++++ > > [..] > >> --- a/migration/options.c >> +++ b/migration/options.c >> @@ -41,6 +41,15 @@ bool migrate_colo(void) >> return s->capabilities[MIGRATION_CAPABILITY_X_COLO]; >> } >> +bool migrate_compress(void) >> +{ >> + MigrationState *s; >> + >> + s = migrate_get_current(); >> + >> + return s->capabilities[MIGRATION_CAPABILITY_COMPRESS]; >> +} > > Probably, we could generate these functions automatically, or at least > have a helper, so that all such functions are one-line wrappers on > that helper.
Markus, or any other qapi guru, I would be happy if this could be managed automagically O:-) Later, Juan.
