Luiz Capitulino <[email protected]> writes: > On Mon, 30 Jul 2012 14:45:04 -0500 > Anthony Liguori <[email protected]> wrote: > >> Orit Wasserman <[email protected]> writes: >> >> > The management can enable/disable a capability for the next migration by >> > using >> > migrate_set_parameter command. >> > The management can query the current migration capabilities using >> > query-migrate-parameters >> > >> > Signed-off-by: Orit Wasserman <[email protected]> >> > Signed-off-by: Juan Quintela <[email protected]> >> >> We have a way to add new commands. Let's not invent a new one. >> Otherwise every subsystem would have it's own approach to querying >> what's available. > > I think it does make sense for setting/getting migration's capabilities, which > are just booleans. And that's what the commands currently do, btw. > > I'd only recommend to rename them to migrate_set_capability > query-migrate-capabilities.
If that's the intent, it should take/return a list of capabilities (expressed as an enum). Regards, Anthony Liguori
