Message ID | 20200519113701.19241-1-jcaamano@suse.com |
---|---|
State | Accepted |
Headers | show |
Series | [ovs-dev] netdev-dpdk: don't set rx mq mode for net_virtio | expand |
On Tue, May 19, 2020 at 01:37:01PM +0200, Jaime Caamaño Ruiz wrote: > Since DPDK 19.11 [1], it is not allowed to set any RX mq mode for virtio > driver. > > [1] https://github.com/DPDK/dpdk/commit/13b3137f3b7c8f866947a9b34e06a8aec0d084f7 > > Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com> > --- LGTM, thanks! Acked-by: Flavio Leitner <fbl@sysclose.org>
On 5/19/20 2:20 PM, Flavio Leitner wrote: > On Tue, May 19, 2020 at 01:37:01PM +0200, Jaime Caamaño Ruiz wrote: >> Since DPDK 19.11 [1], it is not allowed to set any RX mq mode for virtio >> driver. >> >> [1] https://github.com/DPDK/dpdk/commit/13b3137f3b7c8f866947a9b34e06a8aec0d084f7 >> >> Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com> >> --- > > LGTM, thanks! > Acked-by: Flavio Leitner <fbl@sysclose.org> Thanks! Applied to master and backported down to 2.13. Best regards, Ilya Maximets.
> On 5/19/20 2:20 PM, Flavio Leitner wrote: > > On Tue, May 19, 2020 at 01:37:01PM +0200, Jaime Caamaño Ruiz wrote: > >> Since DPDK 19.11 [1], it is not allowed to set any RX mq mode for virtio > >> driver. > >> > >> [1] > https://github.com/DPDK/dpdk/commit/13b3137f3b7c8f866947a9b34e06a8aec > 0d084f7 > >> > >> Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com> > >> --- > > > > LGTM, thanks! > > Acked-by: Flavio Leitner <fbl@sysclose.org> > > Thanks! > > Applied to master and backported down to 2.13. The patch[1] for DPDK was also applied to the DPDK 18.11 (18.11.6 it seems), unless there is an objection I was going to backport this to OVS 2.12 and 2.11 also? [1] http://patchwork.dpdk.org/patch/60774/ Regards Ian
On 10/8/20 1:48 PM, Stokes, Ian wrote: >> On 5/19/20 2:20 PM, Flavio Leitner wrote: >>> On Tue, May 19, 2020 at 01:37:01PM +0200, Jaime Caamaño Ruiz wrote: >>>> Since DPDK 19.11 [1], it is not allowed to set any RX mq mode for virtio >>>> driver. >>>> >>>> [1] >> https://github.com/DPDK/dpdk/commit/13b3137f3b7c8f866947a9b34e06a8aec >> 0d084f7 >>>> >>>> Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com> >>>> --- >>> >>> LGTM, thanks! >>> Acked-by: Flavio Leitner <fbl@sysclose.org> >> >> Thanks! >> >> Applied to master and backported down to 2.13. > > The patch[1] for DPDK was also applied to the DPDK 18.11 (18.11.6 it seems), unless there is an objection I was going to backport this to OVS 2.12 and 2.11 also? Oh. Yes, you're right. Please, go ahead. Thanks! > > [1] http://patchwork.dpdk.org/patch/60774/ > > Regards > Ian >
> On 10/8/20 1:48 PM, Stokes, Ian wrote: > >> On 5/19/20 2:20 PM, Flavio Leitner wrote: > >>> On Tue, May 19, 2020 at 01:37:01PM +0200, Jaime Caamaño Ruiz wrote: > >>>> Since DPDK 19.11 [1], it is not allowed to set any RX mq mode for virtio > >>>> driver. > >>>> > >>>> [1] > >> > https://github.com/DPDK/dpdk/commit/13b3137f3b7c8f866947a9b34e06a8aec > >> 0d084f7 > >>>> > >>>> Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com> > >>>> --- > >>> > >>> LGTM, thanks! > >>> Acked-by: Flavio Leitner <fbl@sysclose.org> > >> > >> Thanks! > >> > >> Applied to master and backported down to 2.13. > > > > The patch[1] for DPDK was also applied to the DPDK 18.11 (18.11.6 it seems), > unless there is an objection I was going to backport this to OVS 2.12 and 2.11 > also? > > Oh. Yes, you're right. Please, go ahead. > > Thanks! > No problem, just pushed to 2.11 and 2.12. Thanks Ian
diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c index 44ebf96da..1a3471ce9 100644 --- a/lib/netdev-dpdk.c +++ b/lib/netdev-dpdk.c @@ -163,7 +163,6 @@ typedef uint16_t dpdk_port_t; static const struct rte_eth_conf port_conf = { .rxmode = { - .mq_mode = ETH_MQ_RX_RSS, .split_hdr_size = 0, .offloads = 0, }, @@ -965,6 +964,14 @@ dpdk_eth_dev_port_config(struct netdev_dpdk *dev, int n_rxq, int n_txq) rte_eth_dev_info_get(dev->port_id, &info); + /* As of DPDK 19.11, it is not allowed to set a mq_mode for + * virtio PMD driver. */ + if (!strcmp(info.driver_name, "net_virtio")) { + conf.rxmode.mq_mode = ETH_MQ_RX_NONE; + } else { + conf.rxmode.mq_mode = ETH_MQ_RX_RSS; + } + /* As of DPDK 17.11.1 a few PMDs require to explicitly enable * scatter to support jumbo RX. * Setting scatter for the device is done after checking for
Since DPDK 19.11 [1], it is not allowed to set any RX mq mode for virtio driver. [1] https://github.com/DPDK/dpdk/commit/13b3137f3b7c8f866947a9b34e06a8aec0d084f7 Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com> --- lib/netdev-dpdk.c | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-)