From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:35212) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SFrwa-0003tI-QB for qemu-devel@nongnu.org; Thu, 05 Apr 2012 15:02:10 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1SFrwW-0005HA-GD for qemu-devel@nongnu.org; Thu, 05 Apr 2012 15:02:03 -0400 Received: from chello084112167138.7.11.vie.surfer.at ([84.112.167.138]:54848 helo=wiesinger.com) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SFrwW-0005GY-5Z for qemu-devel@nongnu.org; Thu, 05 Apr 2012 15:02:00 -0400 Date: Thu, 5 Apr 2012 21:00:18 +0200 (CEST) From: Gerhard Wiesinger In-Reply-To: Message-ID: References: <1332062822-14342-1-git-send-email-dmitry.fleytman@ravellosystems.com> <1332062822-14342-8-git-send-email-dmitry.fleytman@ravellosystems.com> <4F7C3401.1040104@ravellosystems.com> MIME-Version: 1.0 Content-Type: MULTIPART/MIXED; BOUNDARY="-1463811584-327197923-1333650492=:31949" Content-ID: Subject: Re: [Qemu-devel] [PATCH 7/7 v5] VMXNET3 paravirtualized device implementation Interface type "vmxnet3" added. List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Yan Vugenfirer Cc: Anthony Liguori , Alex Fishman , "Michael S. Tsirkin" , Tamir Shomer , yvugenfi@redhat.com, Izik Eidus , qemu-devel@nongnu.org, Dmitry Fleytman This message is in MIME format. The first part should be readable text, while the remaining parts are likely unreadable without MIME-aware tools. ---1463811584-327197923-1333650492=:31949 Content-Type: TEXT/PLAIN; CHARSET=ISO-8859-15; format=flowed Content-ID: Content-Transfer-Encoding: quoted-printable X-MIME-Autoconverted: from 8bit to quoted-printable by wiesinger.com id q35J0ljo025863 On Thu, 5 Apr 2012, Yan Vugenfirer wrote: > On Thu, Apr 5, 2012 at 8:25 AM, Gerhard Wiesinger = wrote: >> >> On Wed, 4 Apr 2012, Izik Eidus wrote: >>> >>> What about this patch?, everything that was asked from Dmitry was acc= omplished... >>> What prevent us from progressing with merging this patch? >> >> >> As already discussed on the list patch v5 doesn't work at least for me= . Previous patches worked better but were not stable under load. >> >> I'm also appreciating a working version and integrating the patch fast. >> >> Ciao, >> Gerhard >> >> -- >> http://www.wiesinger.com/ > > Hello Gerhard, > > 1. I understand that you send your exact command line to Dmitry and > your test scripts, but we cannot reproduce the issues. Can you send us > host network configuration? > > 2. Is it possible=A0for you to run tcpdump on the guest and on the host > and send us the files for review? Since there was no difference for v5 patch, I already sent this for v4=20 patch: See: https://lists.gnu.org/archive/html/qemu-devel/2012-03/msg04062.html https://lists.gnu.org/archive/html/qemu-devel/2012-03/msg04634.html I also sniffed between working NIC (rtl8139) and non working NIC (vmxnet3) and as already thought in //lists.gnu.org/archive/html/qemu-devel/2012-03/msg04062.html TCP offloading checksum is NOT correct. >>From wireshark sniff: Checksum SYN1: 0xe937 [incorrect, should be 0xe936 (maybe caused by "TCP = checksum offload"?)] Checksum SYN2: [incorrect, should be 0xe5af (maybe caused by "TCP checksu= m offload"?)] =3D> checksum is always too high by 1. Please fix it. Ciao, Gerhard -- http://www.wiesinger.com/ ---1463811584-327197923-1333650492=:31949--