[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/7] monitor: Add host_net_add device argument c
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH 4/7] monitor: Add host_net_add device argument completion. |
Date: |
Fri, 23 May 2014 14:05:03 +0200 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
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.)
- [Qemu-devel] [PATCH 2/7] monitor: Add watchdog_action argument completion., (continued)
- [Qemu-devel] [PATCH 2/7] monitor: Add watchdog_action argument completion., Hani Benhabiles, 2014/05/19
- [Qemu-devel] [PATCH 3/7] monitor: Add migrate_set_capability completion., Hani Benhabiles, 2014/05/19
- Re: [Qemu-devel] monitor: Command completion for various commands, Hani Benhabiles, 2014/05/19
- [Qemu-devel] monitor: Command completion for various commands, Hani Benhabiles, 2014/05/19
- [Qemu-devel] [PATCH 4/7] monitor: Add host_net_add device argument completion., Hani Benhabiles, 2014/05/19
[Qemu-devel] [PATCH 5/7] readline: Make completion strings always unique., Hani Benhabiles, 2014/05/19
[Qemu-devel] [PATCH 6/7] monitor: Add host_net_remove arguments completion., Hani Benhabiles, 2014/05/19
[Qemu-devel] [PATCH 7/7] monitor: Add delvm and loadvm argument completion., Hani Benhabiles, 2014/05/19