From mboxrd@z Thu Jan 1 00:00:00 1970 From: Maxime Coquelin Subject: Re: [PATCH v3 17/19] vhost-user: iommu: postpone device creation until ring are mapped Date: Thu, 2 Nov 2017 09:21:50 +0100 Message-ID: <8d27d7d9-9567-1a57-5a5f-760f2f117b73@redhat.com> References: <20171005083627.27828-1-maxime.coquelin@redhat.com> <20171005083627.27828-18-maxime.coquelin@redhat.com> <2DBBFF226F7CF64BAFCA79B681719D953A2CD10A@shsmsx102.ccr.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Cc: "mst@redhat.com" , "jfreiman@redhat.com" , "vkaplans@redhat.com" , "jasowang@redhat.com" To: "Yao, Lei A" , "dev@dpdk.org" , "Horton, Remy" , "Bie, Tiwei" , "yliu@fridaylinux.org" Return-path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id 0AB8A1B319 for ; Thu, 2 Nov 2017 09:22:00 +0100 (CET) In-Reply-To: <2DBBFF226F7CF64BAFCA79B681719D953A2CD10A@shsmsx102.ccr.corp.intel.com> Content-Language: en-US List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Hi Lei, On 11/02/2017 08:21 AM, Yao, Lei A wrote: > ... > Hi, Maxime > I met one issue with your patch set during the v17.11 test. Is it with v17.11-rc2 or -rc1? > The test scenario is following, > 1. Bind one NIC, use test-pmd set vhost-user with 2 queue > usertools/dpdk-devbind.py --bind=igb_uio 0000:05:00.0 > ./x86_64-native-linuxapp-gcc/app/testpmd -c 0xe -n 4 --socket-mem 1024,1024 \ > --vdev 'net_vhost0,iface=vhost-net,queues=2' - -i --rxq=2 --txq=2 --nb-cores=2 --rss-ip > 2. Launch qemu with virtio device which has 2 queue > 3. In VM, launch testpmd with virtio-pmd using only 1 queue. > x86_64-native-linuxapp-gcc/app/testpmd -c 0x07 -n 3 - -i --txqflags=0xf01 \ > --rxq=1 --txq=1 --rss-ip --nb-cores=1 > > First, > commit 09927b5249694bad1c094d3068124673722e6b8f > vhost: translate ring addresses when IOMMU enabled > The patch causes no traffic in PVP test. but link status is still up in vhost-user. > > Second, > eefac9536a901a1f0bb52aa3b6fec8f375f09190 > vhost: postpone device creation until rings are mapped > The patch causes link status "down" in vhost-user. > > Could you have a check at your side? Thanks. Sure, could you please on your side provide more info? 1. Host testpmd logs 2. Qemu version and cmdline Thanks, Maxime > BRs > Lei >