From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 4D994A04DD; Thu, 5 Nov 2020 15:31:24 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id CC4AAC876; Thu, 5 Nov 2020 15:31:22 +0100 (CET) Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by dpdk.org (Postfix) with ESMTP id A3B1AC872 for ; Thu, 5 Nov 2020 15:31:19 +0100 (CET) IronPort-SDR: EJNHZUO4CimB0JCgMDjblXPpZ9Tn6xK7ODczz+mVnKHljzSfeFS2k5X1IpTqtICPgYbpvxe2VV /dF12MfXNCog== X-IronPort-AV: E=McAfee;i="6000,8403,9795"; a="156379979" X-IronPort-AV: E=Sophos;i="5.77,453,1596524400"; d="scan'208";a="156379979" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga005.jf.intel.com ([10.7.209.41]) by orsmga101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 05 Nov 2020 06:31:18 -0800 IronPort-SDR: mPsgI3p+6rk15Gzo1X+8+KqqY0K/o/JS0FO91064ZpJAAxTU3v0AGOcKSDLeK1NnYVWh1uw6Pc pD/hc+iQ1mWQ== X-IronPort-AV: E=Sophos;i="5.77,453,1596524400"; d="scan'208";a="539421535" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.213.252.54]) ([10.213.252.54]) by orsmga005-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 05 Nov 2020 06:31:13 -0800 To: Steve Yang , dev@dpdk.org Cc: beilei.xing@intel.com, wenzhuo.lu@intel.com, bernard.iremonger@intel.com, qiming.yang@intel.com, mdr@ashroe.eu, nhorman@tuxdriver.com, andrew.rybchenko@oktetlabs.ru References: <20201105095151.61052-1-stevex.yang@intel.com> From: Ferruh Yigit Message-ID: <97eae347-ab19-5ad7-6cee-b7f55777b775@intel.com> Date: Thu, 5 Nov 2020 14:31:06 +0000 MIME-Version: 1.0 In-Reply-To: <20201105095151.61052-1-stevex.yang@intel.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [PATCH v1] app/testpmd: fix the overhead overflow issue X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 11/5/2020 9:51 AM, Steve Yang wrote: > If "dev_info.max_mtu" isn't set by driver, the rte_ethdev will set > default value "UINT16_MAX", that will cause the overhead value overflow. > > Add more strict condition to ensure the overhead valid. > > Fixes: bed05536c8f8 ("app/testpmd: fix max Rx packet length for VLAN packets") > > Signed-off-by: Steve Yang Acked-by: Ferruh Yigit