[dpdk-dev] [PATCH v1 2/2] net/ice: remove redundant default RSS field for IP fragment packets

[Ferruh Yigit]

On 8/2/2021 9:36 AM, Wenjun Wu wrote:
> This patch removes redundant default RSS field for IP fragment packets,
> only src MAC address and dst MAC address are needed.
> 

Same comments with first patch.
Impact is not clear, and this patch seems a little different from first path,
this clears one flag from 'cfg.hash_flds', why and if this is not causing any
impact (that is what I understand from description) can you explain in the
commit log why it doesn't change the behavior of the code?

Also again long patch title.

> Signed-off-by: Wenjun Wu <wenjun1.wu at intel.com>
> ---
>  drivers/net/ice/ice_ethdev.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/net/ice/ice_ethdev.c b/drivers/net/ice/ice_ethdev.c
> index a4cd39c954..64ee569525 100644
> --- a/drivers/net/ice/ice_ethdev.c
> +++ b/drivers/net/ice/ice_ethdev.c
> @@ -2975,7 +2975,7 @@ ice_rss_hash_set(struct ice_pf *pf, uint64_t rss_hf)
>  
>  	if (rss_hf & ETH_RSS_FRAG_IPV4) {
>  		cfg.addl_hdrs = ICE_FLOW_SEG_HDR_IPV4 | ICE_FLOW_SEG_HDR_IPV_FRAG;
> -		cfg.hash_flds = ICE_FLOW_HASH_IPV4 | BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_ID);
> +		cfg.hash_flds = ICE_FLOW_HASH_IPV4;
>  		ret = ice_add_rss_cfg_wrap(pf, vsi->idx, &cfg);
>  		if (ret)
>  			PMD_DRV_LOG(ERR, "%s IPV4_FRAG rss flow fail %d",
> @@ -2984,7 +2984,7 @@ ice_rss_hash_set(struct ice_pf *pf, uint64_t rss_hf)
>  
>  	if (rss_hf & ETH_RSS_FRAG_IPV6) {
>  		cfg.addl_hdrs = ICE_FLOW_SEG_HDR_IPV6 | ICE_FLOW_SEG_HDR_IPV_FRAG;
> -		cfg.hash_flds = ICE_FLOW_HASH_IPV6 | BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_ID);
> +		cfg.hash_flds = ICE_FLOW_HASH_IPV6;
>  		ret = ice_add_rss_cfg_wrap(pf, vsi->idx, &cfg);
>  		if (ret)
>  			PMD_DRV_LOG(ERR, "%s IPV6_FRAG rss flow fail %d",
>