On Tue, Sep 27, 2022 at 9:24 AM Kevin Liu <kevinx....@intel.com> wrote: > > Check whether the data packet is illegal of ice and iavf driver. > > v2: > Change the scheme, check the data_len and update commit log. > > Kevin Liu (2): > net/iavf: check illegal packets > net/ice: check illegal packets > > drivers/net/iavf/iavf_rxtx.c | 9 +++++++++ > drivers/net/iavf/iavf_rxtx.h | 2 ++ > drivers/net/ice/ice_rxtx.c | 11 +++++++++++ > drivers/net/ice/ice_rxtx.h | 2 ++ > 4 files changed, 24 insertions(+)
This series looks wrong to me. - What happens if the application requests TSO to the nic? - Checking data_len of a mbuf means only checking the size of the first segment, so if any limitation applies to the packet size, it should by checking pkt_len. I will probably send reverts for the two patches. -- David Marchand
