From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:46416) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aQ9Fl-00082K-Ik for qemu-devel@nongnu.org; Mon, 01 Feb 2016 02:50:30 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aQ9Fk-0003j8-85 for qemu-devel@nongnu.org; Mon, 01 Feb 2016 02:50:29 -0500 Received: from [59.151.112.132] (port=39361 helo=heian.cn.fujitsu.com) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aQ9Fj-0003iK-It for qemu-devel@nongnu.org; Mon, 01 Feb 2016 02:50:28 -0500 References: <1453862428-25570-1-git-send-email-zhangchen.fnst@cn.fujitsu.com> <56A9AAC8.4010803@redhat.com> <56AAC2A2.9020204@cn.fujitsu.com> <56AEC986.2000500@redhat.com> From: Li Zhijian Message-ID: <56AF0E35.8000905@cn.fujitsu.com> Date: Mon, 1 Feb 2016 15:50:13 +0800 MIME-Version: 1.0 In-Reply-To: <56AEC986.2000500@redhat.com> Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 8bit Subject: Re: [Qemu-devel] [PATCH V2] net/traffic-mirror:Add traffic-mirror List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Jason Wang , Zhang Chen , qemu devel Cc: Gui jianfeng , Yang Hongyang , "eddie.dong" , zhanghailiang , "Dr. David Alan Gilbert" On 02/01/2016 10:57 AM, Jason Wang wrote: > > > On 01/29/2016 09:38 AM, Li Zhijian wrote: >> >> >> On 01/28/2016 01:44 PM, Jason Wang wrote: >>> >>> >>> On 01/27/2016 10:40 AM, Zhang Chen wrote: >>>> From: ZhangChen >>>> >>>> Traffic-mirror is a netfilter plugin. >>>> It gives qemu the ability to copy and mirror guest's >>>> net packet. we output packet to chardev. >>>> >>>> usage: >>>> >>>> -netdev tap,id=hn0 >>>> -chardev socket,id=mirror0,host=ip_primary,port=X,server,nowait >>>> -traffic-mirror,id=m0,netdev=hn0,queue=tx/rx/all,outdev=mirror0 >>>> >>>> Signed-off-by: ZhangChen >>>> Signed-off-by: Wen Congyang >>>> Reviewed-by: Yang Hongyang >>> >>> Thanks for the patch. Several questions: >>> >>> - I'm curious about how the patch was tested? Simple setup e.g: >>> >>> -netdev tap,id=hn0 -device virtio-net-pci,netdev=hn0 -chardev >>> socket,id=c0,host=localhost,port=4444,server,nowait -object >>> traffic-mirror,netdev=hn0,outdev=c0,id=f0 -netdev >>> socket,id=s0,connect=127.0.0.1:4444 -device e1000,netdev=s0 >>> a strange thing is about "host=localhost", connection is refused at SUSE 11.3 but connection is connected successfully at Ubuntu 15.10 if i launch qemu with the command line above. I try to launch qemu at three physical machines installed with SUSE 11.3, they all connect failed. But when I specified "host=127.0.0.1", the connection is OK. I have comfirmed that: - "localhost have pointed to 127.0.0.1 if I "ping localhost" at SUSE - "telnet localhost 4444" works at SUSE >>> does not works for me. >> Hi， Jason >> >> I just test the mirror using the command line above, it don't work too. >> I am looking to it, and find that seems because the -net socket >> problem that >> I have ever post a patch try to fix（refer to ↓） >> [Qemu-devel] [PATCH] report a error message if -net socket can not >> connect to server >> https://lists.gnu.org/archive/html/qemu-devel/2015-12/msg00758.html > > Will have a look at this. > >> >> after applying this patch, the qemu monitor tell me following message: >> (qemu) qemu-system-x86_64: net socket is not connected Connection refused > > Maybe two issues. Have you tired to start the mirror on one VM and then > using socket backend to connect it from another VM? Yes, if i connect the mirror on VM1 using socket backend from another VM2, the connection is established successfully. But on VM2 guest, I can't dump any packet using 'tcpdump' That's because in current version code, mirror is not compatible with socket backend and we will fix it in next version. Best regards. Li Zhijian > >> >> >> Thanks >> Li Zhijian >> >> >> > > > > . >