From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id 5BE9C2BF2 for ; Thu, 24 Mar 2016 16:21:56 +0100 (CET) Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga103.fm.intel.com with ESMTP; 24 Mar 2016 08:21:55 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.24,385,1455004800"; d="scan'208";a="930974898" Received: from sie-lab-214-036.ir.intel.com (HELO sie-lab-214-36.ir.intel.com) ([10.237.214.36]) by fmsmga001.fm.intel.com with ESMTP; 24 Mar 2016 08:21:54 -0700 From: Pablo de Lara To: dev@dpdk.org Cc: wenzhuo.lu@intel.com, bruce.richardson@intel.com, Pablo de Lara Date: Thu, 24 Mar 2016 15:22:01 +0000 Message-Id: <1458832924-1489-1-git-send-email-pablo.de.lara.guarch@intel.com> X-Mailer: git-send-email 2.5.5 In-Reply-To: <1458832675-666-1-git-send-email-pablo.de.lara.guarch@intel.com> References: <1458832675-666-1-git-send-email-pablo.de.lara.guarch@intel.com> Subject: [dpdk-dev] [PATCH v4 0/3] Fix incorrect max TX queue numbers for ixgbe X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 24 Mar 2016 15:21:56 -0000 IXGBE supports 128 TX queues. However, the full 128 queues are only available in VT and DCB mode. In normal default "none" mode (VT/DCB off) the maximum number of available queues is only 64. IXGBE doesn't check the mode when reporting the available number of queues. If a queue larger than 64 is used in default mode, the TX packets will be dropped silently. This patchset: - Modifies the device info to show the correct maximum number of available TX queues, depending on the mode. - Adds a check to forbid using a queue number larger than 64 during device configuration (in default mode), so that the problem is reported as early as possible. - It also changes the order of where the dev_conf parameters are copied into the dev structure so that the correct maximum number of queues is reported for the correct mode. Changes in v4: - Split the patch in three smaller patches - Updated release notes document Changes in v3: - Fixed wrong set of TX queues for 82596EB Changes in v2: - Reorder memcpy of device configuration in rte_eth_dev_configure(), so function gets the correct maximum number of queues (depending on the operation mode), before checking the requested number of queues. - Renamed new macro - Reworded/wrapped commit message Pablo de Lara (1): ethdev: copy device configuration earlier Wenzhuo Lu (2): ixgbe: fix incorrect tx queue number assignment ixgbe: fix incorrect max tx queue number doc/guides/rel_notes/release_16_04.rst | 6 ++++++ drivers/net/ixgbe/ixgbe_ethdev.c | 22 +++++++++++++++++++++- drivers/net/ixgbe/ixgbe_ethdev.h | 1 + lib/librte_ether/rte_ethdev.c | 6 +++--- 4 files changed, 31 insertions(+), 4 deletions(-) -- 2.5.5