Re: [Qemu-devel] [PATCH 7/7 v5] VMXNET3 paravirtualized device implementation Interface type "vmxnet3" added.

[Gerhard Wiesinger]

On Thu, 5 Apr 2012, Yan Vugenfirer wrote:

> On Thu, Apr 5, 2012 at 8:25 AM, Gerhard Wiesinger <address@hidden> wrote:
> > On Wed, 4 Apr 2012, Izik Eidus wrote:
> > > What about this patch?, everything that was asked from Dmitry was 
> > > accomplished...
> > > 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 for 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 
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 checksum 
offload"?)]
=> checksum is always too high by 1.

Please fix it.

Ciao,
Gerhard

--
http://www.wiesinger.com/