From mboxrd@z Thu Jan 1 00:00:00 1970 From: wei zhao Subject: [PATCH] net/ixgbe: check jumbo frame enable parameter Date: Thu, 1 Nov 2018 15:55:26 +0800 Message-ID: <1541058926-3576-1-git-send-email-wei.zhao1@intel.com> Cc: stable@dpdk.org, qi.z.zhang@intel.com To: wei.zhao1@intel.com, dev@dpdk.org Return-path: List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" There is necessary to do some check of max packet size boundary for code safe when enable jumbo frame. Signed-off-by: Wei Zhao --- drivers/net/ixgbe/ixgbe_rxtx.c | 31 +++++++++++++++++++++++++++++++ 1 file changed, 31 insertions(+) diff --git a/drivers/net/ixgbe/ixgbe_rxtx.c b/drivers/net/ixgbe/ixgbe_rxtx.c index 2f0262a..d304dee 100644 --- a/drivers/net/ixgbe/ixgbe_rxtx.c +++ b/drivers/net/ixgbe/ixgbe_rxtx.c @@ -4814,6 +4814,34 @@ void __attribute__((cold)) return 0; } +int __attribute__((cold)) +ixgbe_dev_jumboenable_check(struct rte_eth_dev *dev, + uint16_t max_rx_pkt_len) +{ + struct rte_eth_dev_info dev_info; + + ixgbe_dev_info_get(dev, &dev_info); + + /* check that max packet size is within the allowed range */ + if (max_rx_pkt_len < ETHER_MIN_MTU) { + PMD_INIT_LOG(ERR, "max packet size is too small."); + return -EINVAL; + } + + if (max_rx_pkt_len > dev_info.max_rx_pktlen) { + PMD_INIT_LOG(ERR, "max packet size is too big."); + return -EINVAL; + } + + /* check jumbo mode if needed */ + if (max_rx_pkt_len < ETHER_MAX_LEN) { + PMD_INIT_LOG(ERR, "No need to enable jumbo."); + return -EINVAL; + } + + return 0; +} + /* * Initializes Receive Unit. */ @@ -4865,6 +4893,9 @@ int __attribute__((cold)) * Configure jumbo frame support, if any. */ if (rx_conf->offloads & DEV_RX_OFFLOAD_JUMBO_FRAME) { + if (ixgbe_dev_jumboenable_check(dev, rx_conf->max_rx_pkt_len)) + return -EINVAL; + hlreg0 |= IXGBE_HLREG0_JUMBOEN; maxfrs = IXGBE_READ_REG(hw, IXGBE_MAXFRS); maxfrs &= 0x0000FFFF; -- 1.8.3.1