On Tue, May 20, 2014 at 12:03:17AM +0100, Hani Benhabiles wrote: > diff --git a/hmp-commands.hx b/hmp-commands.hx > index 919af6e..6aaec1b 100644 > --- a/hmp-commands.hx > +++ b/hmp-commands.hx > @@ -1209,9 +1209,10 @@ ETEXI > { > .name = "host_net_add", > .args_type = "device:s,opts:s?", > - .params = "tap|user|socket|vde|netmap|dump [options]", > + .params = "tap|user|socket|vde|bridge|dump [options]",
Why did you delete "netmap"? I guess "bridge" should have been appended. > diff --git a/monitor.c b/monitor.c > index 6a3a5c9..365c66a 100644 > --- a/monitor.c > +++ b/monitor.c > @@ -4593,6 +4593,20 @@ void migrate_set_capability_completion(ReadLineState > *rs, int nb_args, > } > } > > +void host_net_add_completion(ReadLineState *rs, int nb_args, const char *str) > +{ > + if (nb_args != 2) { > + return; > + } > + readline_set_completion_index(rs, strlen(str)); > + add_completion_option(rs, str, "tap"); > + add_completion_option(rs, str, "user"); > + add_completion_option(rs, str, "socket"); > + add_completion_option(rs, str, "vde"); > + add_completion_option(rs, str, "dump"); > + add_completion_option(rs, str, "bridge"); Please take a look at net_host_check_device() and share the list from there. (Some of the netdevs depend on build-time options.)