From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:59078) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZQ3p5-0007jE-Iq for qemu-devel@nongnu.org; Thu, 13 Aug 2015 21:30:20 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZQ3p2-0001gb-Cz for qemu-devel@nongnu.org; Thu, 13 Aug 2015 21:30:19 -0400 Received: from [59.151.112.132] (port=61150 helo=heian.cn.fujitsu.com) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZQ3p1-0001Ku-Q4 for qemu-devel@nongnu.org; Thu, 13 Aug 2015 21:30:16 -0400 Message-ID: <55CD43DE.8040301@cn.fujitsu.com> Date: Fri, 14 Aug 2015 09:26:54 +0800 From: Yang Hongyang MIME-Version: 1.0 References: <1438915585-30367-1-git-send-email-yanghy@cn.fujitsu.com> <55C86C30.8080604@redhat.com> In-Reply-To: <55C86C30.8080604@redhat.com> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v6 00/10] For QEMU 2.5: Add a netfilter object and netbuffer filter List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Jason Wang , qemu-devel@nongnu.org Cc: thuth@redhat.com, zhang.zhanghailiang@huawei.com, lizhijian@cn.fujitsu.com, mrhines@linux.vnet.ibm.com, stefanha@redhat.com Hi Jason, Sorry for the late reply, just come back from vacation, and I won't have much time until after kvm forum 2015, anyway, I will address the comments and send a v7 if I have time. On 08/10/2015 05:17 PM, Jason Wang wrote: > > > On 08/07/2015 10:46 AM, Yang Hongyang wrote: >> Add multiqueue support in v6. >> >> This patch add a new object netfilter, capture all network packets. >> Also implement a netbuffer based on this object. >> the "buffer" netfilter could be used by VM FT solutions like >> MicroCheckpointing, to buffer/release packets. Or to simulate >> packet delay. >> [...] >> > > Thanks for the patches. Overall looks good to me, just have some minor > comments, see individual patches for details. Thank you for the review! > > I think we need something like "info filter" to display filters > information, or just add them in "info network" Ok, thanks! > . > -- Thanks, Yang.