Re: [ovs-dev] [PATCH] netdev-dpdk: defer MTU set after interface start

2017-12-13 Thread Aaron Conole
"Stokes, Ian" writes: >> > The issue only arises with the qede PMD and 67fe6d635193 >> > ("netdev-dpdk: use rte_eth_dev_set_mtu.") >> >> I had some more time to look at this today but this patch will break >> OVS DPDK for existing supported DPDK ports during testing. >> >> I tested with an XL710

Re: [ovs-dev] [PATCH] netdev-dpdk: defer MTU set after interface start

2017-12-13 Thread Stokes, Ian
> > The issue only arises with the qede PMD and 67fe6d635193 > > ("netdev-dpdk: use rte_eth_dev_set_mtu.") > > I had some more time to look at this today but this patch will break OVS DPDK > for existing supported DPDK ports during testing. > > I tested with an XL710 and the MTU will fail to appl

Re: [ovs-dev] [PATCH] netdev-dpdk: defer MTU set after interface start

2017-12-13 Thread Jan Scheurich
> > The issue only arises with the qede PMD and 67fe6d635193 > > ("netdev-dpdk: use rte_eth_dev_set_mtu.") > > I had some more time to look at this today but this patch will break OVS DPDK > for existing supported DPDK ports during testing. > > I tested with an XL710 and the MTU will fail to apply

Re: [ovs-dev] [PATCH] netdev-dpdk: defer MTU set after interface start

2017-11-28 Thread Stokes, Ian
> On Fri, Nov 24, 2017 at 9:26 AM, Stokes, Ian wrote: > >> Some PMD assumes that the RX buffers are already allocated when > >> setting the device MTU, because the RX buffer size depends on the MTU. > >> This worked until 67fe6d635193 added a call to rte_eth_dev_set_mtu() > >> in the init code, wh

Re: [ovs-dev] [PATCH] netdev-dpdk: defer MTU set after interface start

2017-11-24 Thread Matteo Croce
On Fri, Nov 24, 2017 at 9:26 AM, Stokes, Ian wrote: >> Some PMD assumes that the RX buffers are already allocated when setting >> the device MTU, because the RX buffer size depends on the MTU. >> This worked until 67fe6d635193 added a call to rte_eth_dev_set_mtu() in >> the init code, which would

Re: [ovs-dev] [PATCH] netdev-dpdk: defer MTU set after interface start

2017-11-24 Thread Nitin Katiyar
Croce ; d...@openvswitch.org; Kavanagh, Mark B Subject: Re: [ovs-dev] [PATCH] netdev-dpdk: defer MTU set after interface start > Some PMD assumes that the RX buffers are already allocated when > setting the device MTU, because the RX buffer size depends on the MTU. > This wor

Re: [ovs-dev] [PATCH] netdev-dpdk: defer MTU set after interface start

2017-11-24 Thread Stokes, Ian
> Some PMD assumes that the RX buffers are already allocated when setting > the device MTU, because the RX buffer size depends on the MTU. > This worked until 67fe6d635193 added a call to rte_eth_dev_set_mtu() in > the init code, which would set the MTU before the RX buffer allocation, > triggering

[ovs-dev] [PATCH] netdev-dpdk: defer MTU set after interface start

2017-11-16 Thread Matteo Croce
Some PMD assumes that the RX buffers are already allocated when setting the device MTU, because the RX buffer size depends on the MTU. This worked until 67fe6d635193 added a call to rte_eth_dev_set_mtu() in the init code, which would set the MTU before the RX buffer allocation, triggering a segment