On 10/31, Qi Zhang wrote: >When a new hash profie is created, we need to reset all related >GLQF_HSYMM registers, otherwise unexpected hash behaviour may happens >on packet that hit that profile. > >The patch fix the issue that we only do reset when symmetric hash >is required, but actually for non symmmetric hash we also need this, >since GLQF_HSYMM might be polluted by previous configuration. > >Fixes: ddae0440353f ("net/ice/base: enable symmetric hash for RSS") > >Signed-off-by: Qi Zhang <qi.z.zh...@intel.com> >Tested-by: Simei Su <simei...@intel.com> >--- > drivers/net/ice/base/ice_flow.c | 2 -- > 1 file changed, 2 deletions(-) > >diff --git a/drivers/net/ice/base/ice_flow.c b/drivers/net/ice/base/ice_flow.c >index d2dbb1177..797d7e6e6 100644 >--- a/drivers/net/ice/base/ice_flow.c >+++ b/drivers/net/ice/base/ice_flow.c >@@ -2199,8 +2199,6 @@ ice_add_rss_cfg_sync(struct ice_hw *hw, u16 vsi_handle, >u64 hashed_flds, > status = ice_add_rss_list(hw, vsi_handle, prof); > > prof->cfg.symm = symm; >- if (!symm) >- goto exit; > > update_symm: > ice_rss_update_symm(hw, prof); >-- >2.13.6 >
Reviewed-by: Xiaolong Ye <xiaolong...@intel.com> Applied to dpdk-next-net-intel. Thanks.