All of lore.kernel.org
 help / color / mirror / Atom feed
From: Gerhard Wiesinger <lists@wiesinger.com>
To: Dmitry Fleytman <dmitry.fleytman@ravellosystems.com>
Cc: Anthony Liguori <aliguori@us.ibm.com>,
	Alex Fishman <alex.fishman@ravellosystems.com>,
	"Michael S. Tsirkin" <mst@redhat.com>,
	yvugenfi@redhat.com, Izik Eidus <izik.eidus@ravellosystems.com>,
	qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH v4 0/9] VMXNET3 paravirtual NIC device implementation
Date: Sun, 25 Mar 2012 08:39:41 +0200	[thread overview]
Message-ID: <4F6EBDAD.8050408@wiesinger.com> (raw)
In-Reply-To: <4F697C56.4060702@wiesinger.com>

Hello Dmitry,

Tried it also on qemu, without success. Same behavior.

I sniffed also with tcpdump:
ICMP traffic is dumped immediately but TCP traffic not. Looks like a TCP 
problem.

For config and other details see below.

Ciao,
Gerhard

On 21.03.2012 07:59, Gerhard Wiesinger wrote:
> On 20.03.2012 09:00, Dmitry Fleytman wrote:
>> Hello, Gerhard
>>
>> I've tested telnet connections on Knoppix running on QEMU-KVM with 
>> patch V5.
>> Everything works fine on my setup.
>> What is your network setup? How do you connect tap1 interface to the
>> outer world?
>
> Hello Dmitry ,
>
> Did you also test with tap?
>
> As patch V2 (i think it was that version) worked well but was not 
> stable I'm guessing that there might be some other problems:
> 1.) tap interface?
> 2.) TCP offload checksumming (as far as I saw I think there were some 
> changes).
>
> Same command line with pcnet is also ok (changed only interface card). 
> My network setup for testing several adapters is stable since about 
> one year.
> /root/download/qemu/git/qemu-kvm/x86_64-softmmu/qemu-system-x86_64
> -drive 
> if=ide,index=3,media=cdrom,file=ISO/KNOPPIX_V6.7.1CD-2011-09-14-DE.iso
> -boot order=cad,menu=on -m 2048 -k de -vga vmware -vnc :0
> -bios /root/download/seabios/git/seabios/out/bios.bin
> -chardev stdio,id=seabios -device 
> isa-debugcon,iobase=0x402,chardev=seabios
> -device pcnet,mac=1a:46:0b:ca:bc:7e,vlan=1,romfile=
> -net tap,ifname=tap1,script=no,downscript=no,vlan=1
>
>
> I use a bridge on eth0 and connect the tap interfaces:
> brctl  show
> bridge name     bridge id               STP enabled     interfaces
> br0             8000.001fc689da45       no              eth0
>                                                         tap0
>                                                         tap1
>
>
>>
>> Also, since you have ping failure to init MSI-X is not related to the
>> problem - device just falls back to MSI interrupts,
>> but anyway, why does it fail? Could it be some QEMU/KVM versions
>> incompartibility?
>
> Don't know why it fails. I'm using latest git QEMU/KVM version.
>
> Will try it on qemu only today in the evening.
>
> Thnx.
>
> Ciao,
> Gerhard
>>
>> Best regards,
>> Dmitry Fleytman.
>>
>> On Mon, Mar 19, 2012 at 9:24 PM, Gerhard 
>> Wiesinger<lists@wiesinger.com>  wrote:
>>> Hello Dmitry,
>>>
>>> Tried also v5 patch without success:
>>> /root/download/qemu/git/qemu-kvm/x86_64-softmmu/qemu-system-x86_64
>>> -drive
>>> if=ide,index=3,media=cdrom,file=ISO/KNOPPIX_V6.7.1CD-2011-09-14-DE.iso
>>> -boot order=cad,menu=on
>>> -m 2048 -k de -vga vmware -vnc :0
>>> -bios /root/download/seabios/git/seabios/out/bios.bin
>>> -chardev stdio,id=seabios -device 
>>> isa-debugcon,iobase=0x402,chardev=seabios
>>> -device vmxnet3,mac=1a:46:0b:ca:bc:7e,vlan=1,romfile=
>>> -net tap,ifname=tap1,script=no,downscript=no,vlan=1
>>>
>>> ping ok, but outside tcp communication fails:
>>> # timeout Knoppix =>  outside
>>> telnet 192.168.0.2 22
>>> # timeout outside =>  Knoppix  failes
>>> telnet 192.168.0.30 22
>>>
>>> RTL8139 with same command line is ok.
>>>
>>> Maybe that helps directly at startup:
>>> kvm_msix_vector_add: kvm_add_msix failed: No space left on device
>>> [vmxnet3][WR][vmxnet3_use_msix_vectors]: Failed to use MSI-X vector 
>>> 9, error
>>> -28
>>> [vmxnet3][WR][vmxnet3_init_msix]: Failed to use MSI-X vectors, error 0
>>> [vmxnet3][WR][vmxnet3_pci_init]: Failed to initialize MSI-X, 
>>> configuration
>>> is inconsistent.
>>> [vmxnet3][WR][vmxnet3_peer_has_vnet_hdr]: Peer has no virtio 
>>> extension. Task
>>> offloads will be emulated.
>>>
>>> I'm using git qemu-kvm and not git qemu.
>>>
>>> Thnx.
>>>
>>> Ciao,
>>> Gerhard
>>>
>>>
>>> On 18.03.2012 16:30, Dmitry Fleytman wrote:
>>>> Hello, Gerhard
>>>>
>>>> I've rechecked SSH connection both incoming and outgoing with patch 
>>>> v5.
>>>> Everything works fine.
>>>> If you still see problems, please, provide your exact configuration.
>>>>
>>>> Thanking you for your support,
>>>> Dmitry Fleytman.
>>>>
>>>>
>>>> On Sun, Mar 18, 2012 at 10:29 AM, Gerhard Wiesinger
>>>> <gerhard@wiesinger.com>    wrote:
>>>>> Hello,
>>>>>
>>>>> I'm still having problems with v4 patch: ping works well, even 
>>>>> with large
>>>>> packet sizes but ssh doesn't work at all.
>>>>> Tested with Knoppix 6.7 and Fedora 16.
>>>>>
>>>>> Thnx.
>>>>>
>>>>> Ciao,
>>>>> Gerhard
>>>>>
>>>>>
>>>>> On 15.03.2012 22:08, Dmitry Fleytman wrote:
>>>>>> This set of patches implements VMWare VMXNET3 paravirtual NIC 
>>>>>> device.
>>>>>> The device supports of all the device features including offload
>>>>>> capabilties,
>>>>>> VLANs and etc.
>>>>>> The device is tested on different OSes:
>>>>>>      Fedora 15
>>>>>>      Ubuntu 10.4
>>>>>>      Centos 6.2
>>>>>>      Windows 2008R2
>>>>>>      Windows 2008 64bit
>>>>>>      Windows 2008 32bit
>>>>>>      Windows 2003 64bit
>>>>>>      Windows 2003 32bit
>>>>>>
>>>>>> Changes in V4:
>>>>>>     Fixed a few problems uncovered by NETIO test suit
>>>>>>     Assertion on failure to initialize MSI/MSI-X replaced with 
>>>>>> warning
>>>>>>     message and fallback to Legacy/MSI respectively
>>>>>>
>>>>>>       Reported-by: Gerhard Wiesinger<lists@wiesinger.com>
>>>>>>
>>>>>>     Various coding style adjustments and patch split-up as 
>>>>>> suggested by
>>>>>> Anthony Liguori
>>>>>>
>>>>>>       Reported-by: Anthony Liguori<aliguori@us.ibm.com>
>>>>>>
>>>>>>     Live migration support added
>>>>>>
>>>>>> Changes in V3:
>>>>>>     Fixed crash when net device that is used as network fronted 
>>>>>> has no
>>>>>>     virtio HDR support.
>>>>>>     Task offloads emulation for cases when net device that is 
>>>>>> used as
>>>>>>     network fronted has no virtio HDR support.
>>>>>>
>>>>>>       Reported-by: Gerhard Wiesinger<lists@wiesinger.com>
>>>>>>
>>>>>> Changes in V2:
>>>>>>     License text changed accoring to community suggestions
>>>>>>     Standard license header from GPLv2+ - licensed QEMU files used
>>>>>>
>>>>>> Dmitry Fleytman (9):
>>>>>>    Adding missing flag VIRTIO_NET_HDR_F_DATA_VALID from Linux kernel
>>>>>>      source tre     Reformatting comments according to checkpatch.pl
>>>>>>      requirements
>>>>>>    Adding utility function net_checksum_add_cont() that allows 
>>>>>> checksum
>>>>>>         calculation of scattered data with odd chunk sizes
>>>>>>    Adding utility function iov_net_csum_add() for iovec checksum
>>>>>>      calculation
>>>>>>    MSI-X state save/load invocations moved to PCI Device save/load
>>>>>>      callbacks     to avoid code duplication in MSI-X-enabled 
>>>>>> devices
>>>>>>      that support live migration
>>>>>>    Header with various utility functions shared by VMWARE SCSI and
>>>>>>      network devi
>>>>>>    Various utility functions used by VMWARE network devices
>>>>>>    Packet abstraction used by VMWARE network devices
>>>>>>    VMXNET3 paravirtual device implementation
>>>>>>    VMXNET3 paravirtualized device integration.     Interface type
>>>>>>      "vmxnet3" added.
>>>>>>
>>>>>>   Makefile.objs           |    1 +
>>>>>>   default-configs/pci.mak |    1 +
>>>>>>   hw/pci.c                |    7 +
>>>>>>   hw/pci.h                |    1 +
>>>>>>   hw/virtio-net.h         |   13 +-
>>>>>>   hw/virtio-pci.c         |    2 -
>>>>>>   hw/vmware_utils.h       |  122 +++
>>>>>>   hw/vmxnet3.c            | 2435
>>>>>> +++++++++++++++++++++++++++++++++++++++++++++++
>>>>>>   hw/vmxnet3.h            |  757 +++++++++++++++
>>>>>>   hw/vmxnet_debug.h       |  121 +++
>>>>>>   hw/vmxnet_pkt.c         | 1243 ++++++++++++++++++++++++
>>>>>>   hw/vmxnet_pkt.h         |  479 ++++++++++
>>>>>>   hw/vmxnet_utils.c       |  165 ++++
>>>>>>   hw/vmxnet_utils.h       |  320 +++++++
>>>>>>   iov.c                   |   29 +
>>>>>>   iov.h                   |    3 +
>>>>>>   net.c                   |    2 +-
>>>>>>   net/checksum.c          |   13 +-
>>>>>>   net/checksum.h          |   14 +-
>>>>>>   19 files changed, 5712 insertions(+), 16 deletions(-)
>>>>>>   create mode 100644 hw/vmware_utils.h
>>>>>>   create mode 100644 hw/vmxnet3.c
>>>>>>   create mode 100644 hw/vmxnet3.h
>>>>>>   create mode 100644 hw/vmxnet_debug.h
>>>>>>   create mode 100644 hw/vmxnet_pkt.c
>>>>>>   create mode 100644 hw/vmxnet_pkt.h
>>>>>>   create mode 100644 hw/vmxnet_utils.c
>>>>>>   create mode 100644 hw/vmxnet_utils.h
>>>>>>
>
>

  reply	other threads:[~2012-03-25  6:42 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-15 21:08 [Qemu-devel] [PATCH v4 0/9] VMXNET3 paravirtual NIC device implementation Dmitry Fleytman
2012-03-15 21:09 ` [Qemu-devel] [PATCH v4 1/9] Adding missing flag VIRTIO_NET_HDR_F_DATA_VALID from Linux kernel source tre Reformatting comments according to checkpatch.pl requirements Dmitry Fleytman
2012-03-15 21:09 ` [Qemu-devel] [PATCH v4 2/9] Adding utility function net_checksum_add_cont() that allows checksum calculation of scattered data with odd chunk sizes Dmitry Fleytman
2012-03-15 21:09 ` [Qemu-devel] [PATCH v4 3/9] Adding utility function iov_net_csum_add() for iovec checksum calculation Dmitry Fleytman
2012-03-15 21:09 ` [Qemu-devel] [PATCH v4 4/9] MSI-X state save/load invocations moved to PCI Device save/load callbacks to avoid code duplication in MSI-X-enabled devices that support live migration Dmitry Fleytman
2012-03-15 23:00   ` Michael S. Tsirkin
2012-03-16  9:18     ` Dmitry Fleytman
2012-03-15 21:09 ` [Qemu-devel] [PATCH v4 5/9] Header with various utility functions shared by VMWARE SCSI and network devi Dmitry Fleytman
2012-03-15 21:09 ` [Qemu-devel] [PATCH v4 6/9] Various utility functions used by VMWARE network devices Dmitry Fleytman
2012-03-15 21:09 ` [Qemu-devel] [PATCH v4 7/9] Packet abstraction " Dmitry Fleytman
2012-03-15 21:09 ` [Qemu-devel] [PATCH v4 8/9] VMXNET3 paravirtual device implementation Dmitry Fleytman
2012-03-15 21:09 ` [Qemu-devel] [PATCH v4 9/9] VMXNET3 paravirtualized device integration. Interface type "vmxnet3" added Dmitry Fleytman
2012-03-16 11:35   ` Paolo Bonzini
2012-03-18  9:26     ` Dmitry Fleytman
2012-03-18  8:29 ` [Qemu-devel] [PATCH v4 0/9] VMXNET3 paravirtual NIC device implementation Gerhard Wiesinger
2012-03-18 15:30   ` Dmitry Fleytman
2012-03-19 19:24     ` Gerhard Wiesinger
2012-03-20  8:00       ` Dmitry Fleytman
2012-03-21  6:59         ` Gerhard Wiesinger
2012-03-25  6:39           ` Gerhard Wiesinger [this message]
2012-03-18  8:32 ` Gerhard Wiesinger

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4F6EBDAD.8050408@wiesinger.com \
    --to=lists@wiesinger.com \
    --cc=alex.fishman@ravellosystems.com \
    --cc=aliguori@us.ibm.com \
    --cc=dmitry.fleytman@ravellosystems.com \
    --cc=izik.eidus@ravellosystems.com \
    --cc=mst@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=yvugenfi@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.