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

[Gerhard Wiesinger <lists@wiesinger.com>]

[-- Attachment #1: Type: TEXT/PLAIN, Size: 1687 bytes --]

On Thu, 5 Apr 2012, Yan Vugenfirer wrote:

> On Thu, Apr 5, 2012 at 8:25 AM, Gerhard Wiesinger <lists@wiesinger.com> 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/