mirror of
https://github.com/openvswitch/ovs
synced 2025-09-02 07:15:17 +00:00
netdev-offload-dpdk: Setting RSS hash types in RSS action.
When we send parallel flows such as VXLAN to a PF[1] port in OVS with multiple PMDs. OVS will create a RTE flow with Mark and RSS actions to send flows to the software data path. But the RSS action does not work well and all the flows are forwarded to a single PMD. This is because RSS hash types should be set in RSS action. [1]: In our testbed, a Mellanox ConnectX-6 is used as a PF port. [i.maximets] DPDK PMD drivers supposed to provide "best-effort" RSS configuration if the type is set to zero. However, they are very inconsistent in practice and barely put any effort to provide a good configuration. For example, mlx5 driver seems to use just RTE_ETH_RSS_IP, which is not enough for most deployments. Setting the types the same way we configure them for a normal RSS in netdev-dpdk to workaround the scalability issue. Signed-off-by: Harold Huang <baymaxhuang@gmail.com> Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
This commit is contained in:
committed by
Ilya Maximets
parent
fcdf8ae4a3
commit
48ae7b802b
@@ -18,6 +18,7 @@
|
|||||||
|
|
||||||
#include <sys/types.h>
|
#include <sys/types.h>
|
||||||
#include <netinet/ip6.h>
|
#include <netinet/ip6.h>
|
||||||
|
#include <rte_ethdev.h>
|
||||||
#include <rte_flow.h>
|
#include <rte_flow.h>
|
||||||
#include <rte_gre.h>
|
#include <rte_gre.h>
|
||||||
|
|
||||||
@@ -1714,7 +1715,7 @@ add_flow_mark_rss_actions(struct flow_actions *actions,
|
|||||||
.conf = (struct rte_flow_action_rss) {
|
.conf = (struct rte_flow_action_rss) {
|
||||||
.func = RTE_ETH_HASH_FUNCTION_DEFAULT,
|
.func = RTE_ETH_HASH_FUNCTION_DEFAULT,
|
||||||
.level = 0,
|
.level = 0,
|
||||||
.types = 0,
|
.types = RTE_ETH_RSS_IP | RTE_ETH_RSS_UDP | RTE_ETH_RSS_TCP,
|
||||||
.queue_num = netdev_n_rxq(netdev),
|
.queue_num = netdev_n_rxq(netdev),
|
||||||
.queue = rss_data->queue,
|
.queue = rss_data->queue,
|
||||||
.key_len = 0,
|
.key_len = 0,
|
||||||
|
Reference in New Issue
Block a user