From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id E4A7D1B1A3; Wed, 10 Jan 2018 05:30:31 +0100 (CET) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga006.jf.intel.com ([10.7.209.51]) by fmsmga101.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 09 Jan 2018 20:30:31 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.46,338,1511856000"; d="scan'208";a="10053097" Received: from dpdk6.bj.intel.com ([172.16.182.87]) by orsmga006.jf.intel.com with ESMTP; 09 Jan 2018 20:30:29 -0800 From: Wei Dai To: wenzhuo.lu@intel.com, jingjing.wu@intel.com, yuan.peng@intel.com, konstantin.ananyev@intel.com Cc: dev@dpdk.org, stable@dpdk.org, Wei Dai Date: Wed, 10 Jan 2018 12:14:20 +0800 Message-Id: <1515557660-36763-3-git-send-email-wei.dai@intel.com> X-Mailer: git-send-email 2.7.5 In-Reply-To: <1515557660-36763-1-git-send-email-wei.dai@intel.com> References: <1515416551-28636-1-git-send-email-wei.dai@intel.com> <1515557660-36763-1-git-send-email-wei.dai@intel.com> Subject: [dpdk-dev] [PATCH v2 2/2] app/testpmd: fix invalid txq number setting 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: , X-List-Received-Date: Wed, 10 Jan 2018 04:30:32 -0000 If an invalid TX queue is configured from testpmd command like "port config all txq number", the global variable txq is updated by this invalid value. It may cause testpmd crash. This patch restores its last correct value when an invalid txq number configured is detected. Fixes: ce8d561418d4 ("app/testpmd: add port configuration settings") Cc: stable@dpdk.org Signed-off-by: Wei Dai --- app/test-pmd/cmdline.c | 2 ++ app/test-pmd/testpmd.c | 12 +++++++++--- app/test-pmd/testpmd.h | 1 + 3 files changed, 12 insertions(+), 3 deletions(-) diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c index a5a1d57..26dd81a 100644 --- a/app/test-pmd/cmdline.c +++ b/app/test-pmd/cmdline.c @@ -1527,6 +1527,8 @@ cmd_config_rx_tx_parsed(void *parsed_result, printf("Warning: Either rx or tx queues should be non zero\n"); return; } + /* bakcup last correct nb_txq */ + nb_txq_bak = nb_txq; nb_txq = res->value; } else if (!strcmp(res->name, "rxd")) { diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c index efafc24..8b49d96 100644 --- a/app/test-pmd/testpmd.c +++ b/app/test-pmd/testpmd.c @@ -190,6 +190,7 @@ queueid_t nb_rxq = 1; /**< Number of RX queues per port. */ queueid_t nb_txq = 1; /**< Number of TX queues per port. */ queueid_t nb_rxq_bak = 1; /**< Backup of last correct number of RX queues */ +queueid_t nb_txq_bak = 1; /**< Backup of last correct number of TX queues */ /* * Configurable number of RX/TX ring descriptors. @@ -721,8 +722,12 @@ init_fwd_streams(void) } if (nb_txq > port->dev_info.max_tx_queues) { printf("Fail: nb_txq(%d) is greater than " - "max_tx_queues(%d)\n", nb_txq, - port->dev_info.max_tx_queues); + "max_tx_queues(%d), restored to backup " + "txq number(%d)\n", nb_txq, + port->dev_info.max_tx_queues, + nb_txq_bak); + /* restored to last correct nb_txq */ + nb_txq = nb_txq_bak; return -1; } if (numa_support) { @@ -744,8 +749,9 @@ init_fwd_streams(void) } } - /* backup the correct nb_rxq */ + /* backup the correct nb_rxq and nb_txq */ nb_rxq_bak = nb_rxq; + nb_txq_bak = nb_txq; q = RTE_MAX(nb_rxq, nb_txq); if (q == 0) { diff --git a/app/test-pmd/testpmd.h b/app/test-pmd/testpmd.h index 6f7932d..bca93c1 100644 --- a/app/test-pmd/testpmd.h +++ b/app/test-pmd/testpmd.h @@ -393,6 +393,7 @@ extern queueid_t nb_rxq; extern queueid_t nb_txq; extern queueid_t nb_rxq_bak; +extern queueid_t nb_txq_bak; extern uint16_t nb_rxd; extern uint16_t nb_txd; -- 2.7.5