From: Ivan Malov <ivan.ma...@oktetlabs.ru> efx_rx_scale_tbl_set() takes the number of entries in the scale table to be set, not the size of the table in bytes; currently this bug does not make any damage since the size argument is used to wrap the loop on the input table when filling in an MCDI request in case if the table size in the MCDI request is larger then one provided by the user, and MCDI scale table size is the same as the size of the table provided by the driver; this patch brings a fix for the bug
Fixes: 4ec1fc3ba881 ("net/sfc: add basic stubs for RSS support on driver attach") Cc: sta...@dpdk.org Signed-off-by: Ivan Malov <ivan.ma...@oktetlabs.ru> Signed-off-by: Andrew Rybchenko <arybche...@solarflare.com> --- drivers/net/sfc/sfc_rx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/sfc/sfc_rx.c b/drivers/net/sfc/sfc_rx.c index 1bf8644..364f718 100644 --- a/drivers/net/sfc/sfc_rx.c +++ b/drivers/net/sfc/sfc_rx.c @@ -1068,7 +1068,7 @@ struct sfc_dp_rx sfc_efx_rx = { goto finish; rc = efx_rx_scale_tbl_set(sa->nic, sa->rss_tbl, - sizeof(sa->rss_tbl)); + RTE_DIM(sa->rss_tbl)); } finish: -- 1.8.2.3