On Mon, Aug 16, 2021 at 10:15 AM <dapengx.yu@intel.com> wrote:
>
> From: Dapeng Yu <dapengx.yu@intel.com>
>
> [ upstream commit 03a91d7efb7098ba739ba458bcc118c3e348fae1 ]
Thanks, applied
> In original implementation, device reconfiguration will generate
> a new default RSS key if there is no one from user, it is unexpected
> when updating a completely unrelated configuration.
>
> This patch makes default RSS key unchanged, during the lifetime of the
> DPDK application even if there are multiple reconfigurations.
>
> Fixes: 50370662b727 ("net/ice: support device and queue ops")
>
> Signed-off-by: Dapeng Yu <dapengx.yu@intel.com>
> ---
> drivers/net/ice/ice_ethdev.c | 35 +++++++++++++++++++++++++++++------
> 1 file changed, 29 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/net/ice/ice_ethdev.c b/drivers/net/ice/ice_ethdev.c
> index 4e15ed54bf..f8046b50fb 100644
> --- a/drivers/net/ice/ice_ethdev.c
> +++ b/drivers/net/ice/ice_ethdev.c
> @@ -2458,6 +2458,31 @@ ice_dev_uninit(struct rte_eth_dev *dev)
> return 0;
> }
>
> +static void
> +ice_get_default_rss_key(uint8_t *rss_key, uint32_t rss_key_size)
> +{
> + static struct ice_aqc_get_set_rss_keys default_key;
> + static bool default_key_done;
> + uint8_t *key = (uint8_t *)&default_key;
> + size_t i;
> +
> + if (rss_key_size > sizeof(default_key)) {
> + PMD_DRV_LOG(WARNING,
> + "requested size %u is larger than default %zu, "
> + "only %zu bytes are gotten for key\n",
> + rss_key_size, sizeof(default_key),
> + sizeof(default_key));
> + }
> +
> + if (!default_key_done) {
> + /* Calculate the default hash key */
> + for (i = 0; i < sizeof(default_key); i++)
> + key[i] = (uint8_t)rte_rand();
> + default_key_done = true;
> + }
> + rte_memcpy(rss_key, key, RTE_MIN(rss_key_size, sizeof(default_key)));
> +}
> +
> static int ice_init_rss(struct ice_pf *pf)
> {
> struct ice_hw *hw = ICE_PF_TO_HW(pf);
> @@ -2505,15 +2530,13 @@ static int ice_init_rss(struct ice_pf *pf)
> }
> }
> /* configure RSS key */
> - if (!rss_conf->rss_key) {
> - /* Calculate the default hash key */
> - for (i = 0; i < vsi->rss_key_size; i++)
> - vsi->rss_key[i] = (uint8_t)rte_rand();
> - } else {
> + if (!rss_conf->rss_key)
> + ice_get_default_rss_key(vsi->rss_key, vsi->rss_key_size);
> + else
> rte_memcpy(vsi->rss_key, rss_conf->rss_key,
> RTE_MIN(rss_conf->rss_key_len,
> vsi->rss_key_size));
> - }
> +
> rte_memcpy(key.standard_rss_key, vsi->rss_key, vsi->rss_key_size);
> ret = ice_aq_set_rss_key(hw, vsi->idx, &key);
> if (ret)
> --
> 2.27.0
>
--
Christian Ehrhardt
Staff Engineer, Ubuntu Server
Canonical Ltd