[PATCH 8/8] net/sfc: use the even spread mode in flow action RSS

[Ivan Malov]

If the user provides contiguous ascending queue IDs,
use the even spread mode to avoid wasting resources
which are needed to serve indirection table entries.

Signed-off-by: Ivan Malov <ivan.ma...@oktetlabs.ru>
Reviewed-by: Andrew Rybchenko <andrew.rybche...@oktetlabs.ru>
Reviewed-by: Andy Moreton <amore...@xilinx.com>
---
 drivers/net/sfc/sfc_flow_rss.c | 19 +++++++++++++++++++
 drivers/net/sfc/sfc_flow_rss.h |  1 +
 2 files changed, 20 insertions(+)

diff --git a/drivers/net/sfc/sfc_flow_rss.c b/drivers/net/sfc/sfc_flow_rss.c
index 4bf3002164..e28c943335 100644
--- a/drivers/net/sfc/sfc_flow_rss.c
+++ b/drivers/net/sfc/sfc_flow_rss.c
@@ -140,6 +140,8 @@ sfc_flow_rss_parse_conf(struct sfc_adapter *sa,
                return EINVAL;
        }
 
+       out->rte_hash_function = in->func;
+
        if (in->queue_num == 0) {
                sfc_err(sa, "flow-rss: parse: 'queue_num' is 0; MIN=1");
                return EINVAL;
@@ -317,6 +319,9 @@ sfc_flow_rss_ctx_program_tbl(struct sfc_adapter *sa,
 
        SFC_ASSERT(sfc_adapter_is_locked(sa));
 
+       if (nb_tbl_entries == 0)
+               return 0;
+
        if (conf->nb_qid_offsets != 0) {
                SFC_ASSERT(ctx->qid_offsets != NULL);
 
@@ -336,6 +341,7 @@ sfc_flow_rss_ctx_program(struct sfc_adapter *sa, struct 
sfc_flow_rss_ctx *ctx)
 {
        efx_rx_scale_context_type_t ctx_type = EFX_RX_SCALE_EXCLUSIVE;
        struct sfc_adapter_shared * const sas = sfc_sa2shared(sa);
+       const efx_nic_cfg_t *encp = efx_nic_cfg_get(sa->nic);
        const struct sfc_flow_rss *flow_rss = &sa->flow_rss;
        struct sfc_rss *ethdev_rss = &sas->rss;
        struct sfc_flow_rss_conf *conf;
@@ -366,6 +372,19 @@ sfc_flow_rss_ctx_program(struct sfc_adapter *sa, struct 
sfc_flow_rss_ctx *ctx)
 
        nb_tbl_entries = RTE_MAX(flow_rss->nb_tbl_entries_min, nb_qid_offsets);
 
+       if (conf->rte_hash_function == RTE_ETH_HASH_FUNCTION_DEFAULT &&
+           conf->nb_qid_offsets == 0 &&
+           conf->qid_span <= encp->enc_rx_scale_even_spread_max_nqueues) {
+               /*
+                * Conformance to a specific hash algorithm is a don't care to
+                * the user. The queue array is contiguous and ascending. That
+                * means that the even spread context may be requested here in
+                * order to avoid wasting precious indirection table resources.
+                */
+               ctx_type = EFX_RX_SCALE_EVEN_SPREAD;
+               nb_tbl_entries = 0;
+       }
+
        if (ctx->nic_handle_refcnt == 0) {
                rc = efx_rx_scale_context_alloc_v2(sa->nic, ctx_type,
                                                   conf->qid_span,
diff --git a/drivers/net/sfc/sfc_flow_rss.h b/drivers/net/sfc/sfc_flow_rss.h
index 3341d06cf4..2ed81dc190 100644
--- a/drivers/net/sfc/sfc_flow_rss.h
+++ b/drivers/net/sfc/sfc_flow_rss.h
@@ -20,6 +20,7 @@ extern "C" {
 
 struct sfc_flow_rss_conf {
        uint8_t                         key[EFX_RSS_KEY_SIZE];
+       enum rte_eth_hash_function      rte_hash_function;
        efx_rx_hash_type_t              efx_hash_types;
        unsigned int                    nb_qid_offsets;
        unsigned int                    qid_span;
-- 
2.30.2