Re: [dpdk-dev] [PATCH v3 17/19] vhost-user: iommu: postpone device creation until ring are mapped

[Maxime Coquelin <maxime.coquelin@redhat.com>]

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
>