All of lore.kernel.org
 help / color / mirror / Atom feed
From: Yang Hongyang <hongyang.yang@easystack.cn>
To: zhanghailiang <zhang.zhanghailiang@huawei.com>, qemu-devel@nongnu.org
Cc: jasowang@redhat.com, dgilbert@redhat.com, zhangchen.fnst@cn.fujitsu.com
Subject: Re: [Qemu-devel] [PATCH v3 0/4] Netfilter: Add each netdev a default filter
Date: Tue, 2 Feb 2016 11:11:15 +0800	[thread overview]
Message-ID: <56B01E53.6060306@easystack.cn> (raw)
In-Reply-To: <1454328077-18820-1-git-send-email-zhang.zhanghailiang@huawei.com>

If we have to add a default filter, then I have a suggestion only
for this series:
1. Add a nop filter. filter-nop.c
2. Add a "default-filter=xxx" property to -netdev, if not specified,
    default to nop.

On 02/01/2016 08:01 PM, zhanghailiang wrote:
> This series is a prerequisite for COLO, here we add each netdev
> a default buffer filter, it is disabled by default, and has
> no side effect for delivering packets in net layer.
>
> Note: this series is based on patch
>      '[PATCH v2] net/filter: Fix the output information for command 'info network'
>
> v3:
>   - Drop patch '[PATCH RFC v2 2/5] vl: Make object_create() public'
>   - Use object_new_with_props() instead of object_create() (Daniel)
> v2:
>   - Drop the patch net/filter: prevent the default filter to be deleted' (Jason)
>   - Re-implement netdev_add_filter() by re-using object_object() (Jason)
>   - Send patch 'net/filter: Fix the output information for command 'info
>     network' as an independent one. (Jason)
>
> zhanghailiang (4):
>    net/filter: Add a 'status' property for filter object
>    net/filter: Introduce a helper to add a filter to the netdev
>    filter-buffer: Accept zero interval
>    net/filter: Add a default filter to each netdev
>
>   include/net/filter.h | 12 ++++++++
>   net/filter-buffer.c  | 10 -------
>   net/filter.c         | 79 ++++++++++++++++++++++++++++++++++++++++++++++++++++
>   net/net.c            | 23 +++++++++++++++
>   4 files changed, 114 insertions(+), 10 deletions(-)
>

-- 
Thanks,
Yang

  parent reply	other threads:[~2016-02-02  3:11 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-01 12:01 [Qemu-devel] [PATCH v3 0/4] Netfilter: Add each netdev a default filter zhanghailiang
2016-02-01 12:01 ` [Qemu-devel] [PATCH v3 1/4] net/filter: Add a 'status' property for filter object zhanghailiang
2016-02-01 12:01 ` [Qemu-devel] [PATCH v3 2/4] net/filter: Introduce a helper to add a filter to the netdev zhanghailiang
2016-02-02  3:14   ` Yang Hongyang
2016-02-05  2:24     ` Hailiang Zhang
2016-02-01 12:01 ` [Qemu-devel] [PATCH v3 3/4] filter-buffer: Accept zero interval zhanghailiang
2016-02-01 12:01 ` [Qemu-devel] [PATCH v3 4/4] net/filter: Add a default filter to each netdev zhanghailiang
2016-02-02  3:16   ` Yang Hongyang
2016-02-05  2:35     ` Hailiang Zhang
2016-02-02  3:11 ` Yang Hongyang [this message]
2016-02-02 12:34   ` [Qemu-devel] [PATCH v3 0/4] Netfilter: Add each netdev a default filter Hailiang Zhang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=56B01E53.6060306@easystack.cn \
    --to=hongyang.yang@easystack.cn \
    --cc=dgilbert@redhat.com \
    --cc=jasowang@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=zhang.zhanghailiang@huawei.com \
    --cc=zhangchen.fnst@cn.fujitsu.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.