[dpdk-dev] [PATCH v1 00/12] fix rx packets dropped issue

[Ferruh Yigit]

On 12/9/2020 3:16 AM, Steve Yang wrote:
> The jumbo frame used the 'RTE_ETHER_MAX_LEN' as boundary condition, this
> fix will change the boundary condition with 'RTE_ETHER_MTU'.
> 
> When the MTU(1500) set, the frame type of rx packet will be different
> if used different overhead, it will cause the consistency issue, and the
> normal packet will be dropped. Hence, using fixed value 'RTE_ETHER_MTU'
> can avoid this issue.
> 
> Following scopes will be changed:
> - 'rte_ethdev'
> - 'app', e.g.: 'test-pmd';
> - net PMDs which support VLAN tag(s) within overhead, e.g.: i40e;
> 
> Steve Yang (12):
>    net/dpaa2: fix the jumbo frame flag condition for mtu set
>    net/e1000: fix the jumbo frame flag condition for mtu set
>    net/hns3: fix the jumbo frame flag condition for mtu set
>    net/i40e: fix the jumbo frame flag condition
>    net/iavf: fix the jumbo frame flag condition
>    net/ice: fix the jumbo frame flag condition
>    net/ipn3ke: fix the jumbo frame flag condition for mtu set
>    net/octeontx: fix the jumbo frame flag condition for mtu set
>    net/octeontx2: fix the jumbo frame flag condition for mtu
>    net/qede: fix the jumbo frame flag condition for mtu set
>    net/sfc: fix the jumbo frame flag condition for mtu set
>    net/thunderx: fix the jumbo frame flag condition for mtu set
> 

Overall looks good to me, there is no review from the individual drivers, but if 
there is no objection I am for merging it soon. A few things below:

1) I see following usage not updated, can you please include them in next version:
ixgbe_set_vf_lpe
ixgbe_dev_mtu_set
cxgbe_dev_mtu_set
cxgbe_dev_rx_queue_setup
axgb_mtu_set
enetc_mtu_set
hinic_dev_set_mtu
dpaa_mtu_set
nfp_net_dev_mtu_set
lio_dev_mtu_set


2) I guess 'eth_dev' update will come as separate patch, can you please send it 
without waiting this set to be merged, since both are related having them both 
can help discussions.

Thanks,
ferruh