[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] RFC/net: Add a net filter
From: |
Thomas Huth |
Subject: |
Re: [Qemu-devel] [PATCH] RFC/net: Add a net filter |
Date: |
Wed, 22 Jul 2015 15:05:25 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 |
On 22/07/15 12:55, Yang Hongyang wrote:
> This patch add a net filter between network backend and NIC devices.
> All packets will pass by this filter.
> TODO:
> multiqueue support.
> plugin support.
>
> +--------------+ +-------------+
> +----------+ | filter | |frontend(NIC)|
> | real | | | | |
> | network <--+backend <-------+ |
> | backend | | peer +-------> peer |
> +----------+ +--------------+ +-------------+
>
> Usage:
> -netdev tap,id=bn0 # you can use whatever backend as needed
> -netdev filter,id=f0,backend=bn0,plugin=dump
> -device e1000,netdev=f0
That's basically a neat idea... however, one question remains: Will
there be other filters beside the "dump" filter that you mentioned in
your example? Do you already have something in mind?
If not, I think this is likely not worth the effort - and we likely
should keep it simple and implement the dump option as suggested with my
patches instead.
> diff --git a/net/filter.c b/net/filter.c
> new file mode 100644
> index 0000000..006c64a
> --- /dev/null
> +++ b/net/filter.c
> @@ -0,0 +1,200 @@
> +/*
> + * COarse-grain LOck-stepping Virtual Machines for Non-stop Service (COLO)
> + * (a.k.a. Fault Tolerance or Continuous Replication)
This looks like a bad copy-n-paste comment... ?
> + * Copyright (c) 2015 HUAWEI TECHNOLOGIES CO., LTD.
> + * Copyright (c) 2015 FUJITSU LIMITED
> + * Copyright (c) 2015 Intel Corporation
> + *
> + * This work is licensed under the terms of the GNU GPL, version 2 or
> + * later. See the COPYING file in the top-level directory.
> + */
...
Thomas
- [Qemu-devel] [PATCH v2 2/5] net/dump: Move DumpState into NetClientState, (continued)
- [Qemu-devel] [PATCH v2 2/5] net/dump: Move DumpState into NetClientState, Thomas Huth, 2015/07/13
- [Qemu-devel] [PATCH v2 1/5] net/dump: Add support for receive_iov function, Thomas Huth, 2015/07/13
- [Qemu-devel] [PATCH v2 3/5] net/dump: Rework net-dump init functions, Thomas Huth, 2015/07/13
- [Qemu-devel] [PATCH v2 4/5] net/dump: Add dump option for netdev devices, Thomas Huth, 2015/07/13
- [Qemu-devel] [PATCH v2 5/5] qemu options: Add information about dumpfile to help text, Thomas Huth, 2015/07/13
- Re: [Qemu-devel] [PATCH v2 0/5] For QEMU 2.5: Network traffic dumping for -netdev devices, Jason Wang, 2015/07/22
- Re: [Qemu-devel] [PATCH v2 0/5] For QEMU 2.5: Network traffic dumping for -netdev devices, Yang Hongyang, 2015/07/22
- Re: [Qemu-devel] [PATCH] RFC/net: Add a net filter, Stefan Hajnoczi, 2015/07/22
- Re: [Qemu-devel] [PATCH] RFC/net: Add a net filter, Yang Hongyang, 2015/07/22
- Re: [Qemu-devel] [PATCH] RFC/net: Add a net filter, Stefan Hajnoczi, 2015/07/23
- Re: [Qemu-devel] [PATCH] RFC/net: Add a net filter, Jason Wang, 2015/07/23
- Re: [Qemu-devel] [PATCH] RFC/net: Add a net filter, Yang Hongyang, 2015/07/27
- Re: [Qemu-devel] [PATCH] RFC/net: Add a net filter, Yang Hongyang, 2015/07/27
- Re: [Qemu-devel] [PATCH] RFC/net: Add a net filter, Jason Wang, 2015/07/27
- Re: [Qemu-devel] [PATCH] RFC/net: Add a net filter, Yang Hongyang, 2015/07/27
- Re: [Qemu-devel] [PATCH] RFC/net: Add a net filter, Jason Wang, 2015/07/27