From patchwork Tue Jun 22 09:25:28 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: =?utf-8?q?Havl=C3=ADk_Martin?= X-Patchwork-Id: 94665 X-Patchwork-Delegate: andrew.rybchenko@oktetlabs.ru Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 01867A0C41; Tue, 22 Jun 2021 11:26:11 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id DF67F40DF7; Tue, 22 Jun 2021 11:26:10 +0200 (CEST) Received: from eva.fit.vutbr.cz (eva.fit.vutbr.cz [147.229.176.14]) by mails.dpdk.org (Postfix) with ESMTP id 45EBE40DF5 for ; Tue, 22 Jun 2021 11:26:09 +0200 (CEST) Received: from dpdk-test7.liberouter.org ([IPv6:2001:718:800:ff00:d97f:a151:83a2:7c4f]) (authenticated bits=0) by eva.fit.vutbr.cz (8.16.1/8.16.1) with ESMTPSA id 15M9Ps4i054017 (version=TLSv1.3 cipher=TLS_AES_256_GCM_SHA384 bits=256 verify=NO); Tue, 22 Jun 2021 11:26:09 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=stud.fit.vutbr.cz; s=studfit; t=1624353969; bh=doMoMOHm4RZrpkcCH/bdW98Xylm//PBu/qY9WWyfCUg=; h=From:To:Cc:Subject:Date:In-Reply-To:References; b=N6WehzMcIdccC/GdxyliIMdL6EVClvg8KemQzq8SzYsV+wZP+9hxQ2ogCLYKwILc4 iEdAuZoM146liHLMM1d7FbC7FymLW0wzNOyiWyUwmqxUemvCwjTJBOkndhcDVj2xcM MAmq53gESE2fTtgH0FuN1nkh2UplJ8UFNjoYRACY= From: Martin Havlik To: xhavli56@stud.fit.vutbr.cz, Chas Williams , "Min Hu (Connor)" , Tomasz Kulasek , Declan Doherty Cc: dev@dpdk.org, Jan Viktorin Date: Tue, 22 Jun 2021 11:25:28 +0200 Message-Id: <20210622092531.73112-2-xhavli56@stud.fit.vutbr.cz> X-Mailer: git-send-email 2.27.0 In-Reply-To: <20210622092531.73112-1-xhavli56@stud.fit.vutbr.cz> References: <20210622092531.73112-1-xhavli56@stud.fit.vutbr.cz> MIME-Version: 1.0 Subject: [dpdk-dev] [PATCH 1/3] net/bonding: fix proper return value check and correct log message X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 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" Return value is now saved to errval and log message on error reports correct function name, doesn't use q_id which was out of context, and uses up-to-date errval. Fixes: 112891cd27e5 ("net/bonding: add dedicated HW queues for LACP control") Cc: tomaszx.kulasek@intel.com Signed-off-by: Martin Havlik Cc: Jan Viktorin Acked-by: Min Hu (Connor) Reviewed-by: Andrew Rybchenko --- drivers/net/bonding/rte_eth_bond_pmd.c | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/drivers/net/bonding/rte_eth_bond_pmd.c b/drivers/net/bonding/rte_eth_bond_pmd.c index b01ef003e..4c43bf916 100644 --- a/drivers/net/bonding/rte_eth_bond_pmd.c +++ b/drivers/net/bonding/rte_eth_bond_pmd.c @@ -1805,12 +1805,13 @@ slave_configure(struct rte_eth_dev *bonded_eth_dev, != 0) return errval; - if (bond_ethdev_8023ad_flow_verify(bonded_eth_dev, - slave_eth_dev->data->port_id) != 0) { + errval = bond_ethdev_8023ad_flow_verify(bonded_eth_dev, + slave_eth_dev->data->port_id); + if (errval != 0) { RTE_BOND_LOG(ERR, - "rte_eth_tx_queue_setup: port=%d queue_id %d, err (%d)", - slave_eth_dev->data->port_id, q_id, errval); - return -1; + "bond_ethdev_8023ad_flow_verify: port=%d, err (%d)", + slave_eth_dev->data->port_id, errval); + return errval; } if (internals->mode4.dedicated_queues.flow[slave_eth_dev->data->port_id] != NULL)