[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 03/12] netfilter: add netfilter_{add|del} comman
From: |
Thomas Huth |
Subject: |
Re: [Qemu-devel] [PATCH 03/12] netfilter: add netfilter_{add|del} commands |
Date: |
Wed, 29 Jul 2015 10:15:48 -0400 (EDT) |
On Wednesday, July 29, 2015 12:51:47 PM,
"Yang Hongyang" <address@hidden> wrote:
>
> add netfilter_{add|del} commands
> This is mostly the same with netdev_{add|del} commands.
>
> Signed-off-by: Yang Hongyang <address@hidden>
> ---
...
> diff --git a/hmp-commands.hx b/hmp-commands.hx
> index d3b7932..5326a82 100644
> --- a/hmp-commands.hx
> +++ b/hmp-commands.hx
> @@ -1253,6 +1253,36 @@ Remove host network device.
> ETEXI
>
> {
> + .name = "netfilter_add",
> + .args_type = "netfilter:O",
> + .params = "[type],id=str,netdev=str[,prop=value][,...]",
I'm not fully sure about the syntax, but writing "[type]" in square
brackets sounds like this parameter is optional to me. But if
I've got this right, it is not optional, is it? So maybe use
"<type>" instead?
Thomas
- Re: [Qemu-devel] [PATCH 02/12] init/cleanup of netfilter object, (continued)
[Qemu-devel] [PATCH 01/12] net: add a new object netfilter, Yang Hongyang, 2015/07/29
[Qemu-devel] [PATCH 03/12] netfilter: add netfilter_{add|del} commands, Yang Hongyang, 2015/07/29
- Re: [Qemu-devel] [PATCH 03/12] netfilter: add netfilter_{add|del} commands,
Thomas Huth <=
Re: [Qemu-devel] [PATCH 00/12] For QEMU 2.5: Add a netfilter object and netbuffer filter, Thomas Huth, 2015/07/29