> @@ -975,27 +1003,30 @@ i40e_flow_fdir_fill_eth_ip_head(const struct 
> i40e_fdir_input
> *fdir_input,
>       raw_pkt += sizeof(uint16_t);
>       len += sizeof(uint16_t);
> 
> -     switch (fdir_input->pctype) {
> -     case I40E_FILTER_PCTYPE_L2_PAYLOAD:
> +     if (is_customized_pctype) {
> +             cus_pctype = i40e_flow_fdir_find_customized_pctype(pf, pctype);
> +             if (!cus_pctype)
> +                     PMD_DRV_LOG(ERR, "unknown pctype %u.",
> +                                 fdir_input->pctype);
Doesn't return here?
If it is impossible cus_pctype is NULL, the check is unnecessary.
Because you used the cus_ptype below.

Thanks
Jingjing

Reply via email to