From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:44585) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1abNYK-0004vw-Lt for qemu-devel@nongnu.org; Thu, 03 Mar 2016 02:20:05 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1abNYG-0003Mw-LJ for qemu-devel@nongnu.org; Thu, 03 Mar 2016 02:20:04 -0500 Received: from [59.151.112.132] (port=44198 helo=heian.cn.fujitsu.com) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1abNYF-0003Kd-Oz for qemu-devel@nongnu.org; Thu, 03 Mar 2016 02:20:00 -0500 References: <1456748635-1912-1-git-send-email-zhangchen.fnst@cn.fujitsu.com> <1456748635-1912-2-git-send-email-zhangchen.fnst@cn.fujitsu.com> <56D685FE.6030207@redhat.com> <56D6955E.5090102@cn.fujitsu.com> <56D7DE4A.5000101@redhat.com> From: Zhang Chen Message-ID: <56D7E5C5.2040707@cn.fujitsu.com> Date: Thu, 3 Mar 2016 15:20:37 +0800 MIME-Version: 1.0 In-Reply-To: <56D7DE4A.5000101@redhat.com> Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH V2 1/3] net/filter-traffic: add filter-traffic.h List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Jason Wang , qemu devel Cc: zhanghailiang , Li Zhijian , Gui jianfeng , "eddie.dong" , "Dr. David Alan Gilbert" , Yang Hongyang On 03/03/2016 02:48 PM, Jason Wang wrote: > > On 03/02/2016 03:25 PM, Zhang Chen wrote: >> >> On 03/02/2016 02:19 PM, Jason Wang wrote: >>> On 02/29/2016 08:23 PM, Zhang Chen wrote: >>>> We can reuse filter-traffic by filter-mirror, >>>> filter-redirector and so on. >>> I think we could share more than this. E.g just use filter-mirror.c to >>> implement both mirror and redirector. >> OK, should we change the name to filter-mirror? like filter-traffic? >> and add redirect=on/off >> >> -object >> filter-mirror,id=qtest-f0,netdev=qtest-bn0,queue=tx,indev=redirect0,redirect=on,outdev=mirror0 >> > I think we'd better use another type for this. > What is the "another type " ? please tell me the detail for this. Thanks zhangchen > > -- Thanks zhangchen