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.

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
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to