On 05/02/2018 11:16 AM, Maxime Coquelin wrote:
> Hi Ian,
> 
> On 05/02/2018 11:42 AM, Stokes, Ian wrote:
>>> On 04/25/2018 10:17 AM, Stokes, Ian wrote:
>>>>> Modify travis linux build script to use the latest DPDK stable
>>>>> release 17.11.2. Update docs for latest DPDK stable releases.
>>>>>
>>>>
>>>> Thanks for this Kevin, I'm lining this up for the next pull request.
>>>>
>>>
>>> Thanks - note, this is relevant for master and branch-2.9.
>>>
>>
>> Hi Kevin,
>>
>> I was running validation for the this week's pull request and I
>> spotted vhost reconnect was failing.
>>
>> I traced it back to the move to 17.11.2.
>>
>> The test case is a vm running testpmd forwarding traffic in p2p.
>>
>> The vswitch itself is reset so traffic stops arriving to the vm, after
>> restarting the vswitch reconnects to the VM. At this point testpmd
>> within the VM segfaults.
>>

Thanks for finding it. Which version of guest DPDK and QEMU do you use?
Also what are settings for rx mergable buffers, vIOMMU, MTU, num queues?

>> Doing a git bisect on DPDK 17.11.2 the issue occurs with commit
>>
>> vhost: fix indirect descriptors table translation size (a0d990b19)
> 
> I would be surprised the problem comes from this patch.
> I guess your are testing without vIOMMU enabled> If this is the case, then 
> the size parameter isn't used at the time of
> this patch:
> 
> static __rte_always_inline uint64_t
> vhost_iova_to_vva(struct virtio_net *dev, struct vhost_virtqueue *vq,
>             uint64_t iova, uint64_t size, uint8_t perm)
> {
>     if (!(dev->features & (1ULL << VIRTIO_F_IOMMU_PLATFORM)))
>         return rte_vhost_gpa_to_vva(dev->mem, iova);
> 
>     return __vhost_iova_to_vva(dev, vq, iova, size, perm);
> }
> 
> So I fail to see how it could cause any regression.
> 
> Regards,
> Maxime
> 
>> Just wondering if you see similar behavior in testing?
>>
>> Regards
>> Ian
>>
>>

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to