From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id DBCFDA0C4B; Wed, 3 Nov 2021 01:54:41 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 58F23411CB; Wed, 3 Nov 2021 01:53:16 +0100 (CET) Received: from mail-pl1-f171.google.com (mail-pl1-f171.google.com [209.85.214.171]) by mails.dpdk.org (Postfix) with ESMTP id 9C6F64116E for ; Wed, 3 Nov 2021 01:53:11 +0100 (CET) Received: by mail-pl1-f171.google.com with SMTP id t11so1309688plq.11 for ; Tue, 02 Nov 2021 17:53:11 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=broadcom.com; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version; bh=veTrEIgClXCdx0JQKqO4qzNh3baEZwXHM2cCsSxFuUs=; b=VkEG8VFOzT8STxcC+rPoDHudvrlQ53jXTt5BnLUcYwBRMLnczOy6Akhkf9FHtMpiz0 EtLbkzKmR8K/0xOAgksYh70k9Fk+Ev9M1N1QJ/gIWGh2V4kAoLAEhb+OSb5QCJzkpARU u7ieRKr1atyq1h8YuYVr9w/RlbUc3/0sBh3nU= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version; bh=veTrEIgClXCdx0JQKqO4qzNh3baEZwXHM2cCsSxFuUs=; b=s2JSnzlnj6Rg3k+zv4X8nyQhoEc/Sj4QzqZkr/Cp1dH3HTpKnE61I5UW9HjVJBpz79 S7ryJY/3RqBTt5bzZJXewO+xy9/wcVfE7uYL6kgofqMwiLVqUrpWMaZpXHsz/2stThP7 0kwXrKFfsxKtxzY36nPs+cu0AmyKZey8hK9cnJM2hTcNF10plXiom8GRdlNs8uuqzE0k orqzXnc6oUBxQC2rB6CLgbHEp6ZQr0NNbkxR050LYGPqN4K5/MjMSZ0WQ7RS+v00SxPV 9V/gzpBh0MryBThKdm+l28n0d3cmwbKARuBIY7FW1NhvDIGUz3BR6GXgMTKK9Ldv8LG5 vdxQ== X-Gm-Message-State: AOAM532tAGrFWjQSvUrcOCELNTvl9nrMpW95CmcXPFXCGNtowPJslJhA s16g0AberKdAhaRwlTbFrQB9w82oN+Q25EAoVhmqtmj3qZcZ9086eE7p+Gugnt/BX83KulFXnm+ ivu1f8xeUYV7SVMMvvP1CHRXDAs7RnAuSRGdhJvJW3gm1ZmzWufetGZkRqQSvYaE= X-Google-Smtp-Source: ABdhPJzWnRrWZnAhxzSkQxdHIwonMf+4UgsDfG3lBnoVnKJ/3KHLCUBHzXnv7aXf9/5sDVdPPRs9QQ== X-Received: by 2002:a17:902:ec8e:b0:141:da55:6158 with SMTP id x14-20020a170902ec8e00b00141da556158mr19037033plg.7.1635900790508; Tue, 02 Nov 2021 17:53:10 -0700 (PDT) Received: from localhost.localdomain ([192.19.224.250]) by smtp.gmail.com with ESMTPSA id k2sm323450pfu.112.2021.11.02.17.53.09 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Tue, 02 Nov 2021 17:53:10 -0700 (PDT) From: Ajit Khaparde To: dev@dpdk.org Cc: Kishore Padmanabha , Venkat Duvvuru , Shahaji Bhosle , Mike Baucom Date: Tue, 2 Nov 2021 17:52:43 -0700 Message-Id: <20211103005251.25524-13-ajit.khaparde@broadcom.com> X-Mailer: git-send-email 2.30.1 (Apple Git-130) In-Reply-To: <20211103005251.25524-1-ajit.khaparde@broadcom.com> References: <20211102040556.7840-1-venkatkumar.duvvuru@broadcom.com> <20211103005251.25524-1-ajit.khaparde@broadcom.com> MIME-Version: 1.0 Content-Type: multipart/signed; protocol="application/pkcs7-signature"; micalg=sha-256; boundary="00000000000087d5b005cfd7d587" X-Content-Filtered-By: Mailman/MimeDel 2.1.29 Subject: [dpdk-dev] [PATCH v4 12/20] net/bnxt: update log messages in TruFlow path 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" --00000000000087d5b005cfd7d587 Content-Transfer-Encoding: 8bit From: Kishore Padmanabha Some of the error level log messages are made debug level messages. When Truflow is not enabled then Truflow init error messages are moved to debug level instead. Signed-off-by: Kishore Padmanabha Signed-off-by: Venkat Duvvuru Reviewed-by: Shahaji Bhosle Reviewed-by: Mike Baucom Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/tf_ulp/bnxt_ulp.c | 24 ++++++++++++------------ drivers/net/bnxt/tf_ulp/ulp_def_rules.c | 2 +- drivers/net/bnxt/tf_ulp/ulp_flow_db.c | 2 +- drivers/net/bnxt/tf_ulp/ulp_mark_mgr.c | 2 +- 4 files changed, 15 insertions(+), 15 deletions(-) diff --git a/drivers/net/bnxt/tf_ulp/bnxt_ulp.c b/drivers/net/bnxt/tf_ulp/bnxt_ulp.c index 7deacd1f3e..21f71d6445 100644 --- a/drivers/net/bnxt/tf_ulp/bnxt_ulp.c +++ b/drivers/net/bnxt/tf_ulp/bnxt_ulp.c @@ -1396,16 +1396,16 @@ bnxt_ulp_port_init(struct bnxt *bp) uint32_t ulp_flags; int32_t rc = 0; - if (!BNXT_PF(bp) && !BNXT_VF_IS_TRUSTED(bp)) { - BNXT_TF_DBG(ERR, - "Skip ulp init for port: %d, not a TVF or PF\n", + if (!BNXT_TRUFLOW_EN(bp)) { + BNXT_TF_DBG(DEBUG, + "Skip ulp init for port: %d, TF is not enabled\n", bp->eth_dev->data->port_id); return rc; } - if (!BNXT_TRUFLOW_EN(bp)) { - BNXT_TF_DBG(ERR, - "Skip ulp init for port: %d, truflow is not enabled\n", + if (!BNXT_PF(bp) && !BNXT_VF_IS_TRUSTED(bp)) { + BNXT_TF_DBG(DEBUG, + "Skip ulp init for port: %d, not a TVF or PF\n", bp->eth_dev->data->port_id); return rc; } @@ -1520,16 +1520,16 @@ bnxt_ulp_port_deinit(struct bnxt *bp) struct rte_pci_device *pci_dev; struct rte_pci_addr *pci_addr; - if (!BNXT_PF(bp) && !BNXT_VF_IS_TRUSTED(bp)) { - BNXT_TF_DBG(ERR, - "Skip ULP deinit port:%d, not a TVF or PF\n", + if (!BNXT_TRUFLOW_EN(bp)) { + BNXT_TF_DBG(DEBUG, + "Skip ULP deinit for port:%d, TF is not enabled\n", bp->eth_dev->data->port_id); return; } - if (!BNXT_TRUFLOW_EN(bp)) { - BNXT_TF_DBG(ERR, - "Skip ULP deinit for port:%d, truflow is not enabled\n", + if (!BNXT_PF(bp) && !BNXT_VF_IS_TRUSTED(bp)) { + BNXT_TF_DBG(DEBUG, + "Skip ULP deinit port:%d, not a TVF or PF\n", bp->eth_dev->data->port_id); return; } diff --git a/drivers/net/bnxt/tf_ulp/ulp_def_rules.c b/drivers/net/bnxt/tf_ulp/ulp_def_rules.c index 01233c0f5e..8790d7ac0d 100644 --- a/drivers/net/bnxt/tf_ulp/ulp_def_rules.c +++ b/drivers/net/bnxt/tf_ulp/ulp_def_rules.c @@ -323,7 +323,7 @@ ulp_default_flow_create(struct rte_eth_dev *eth_dev, ulp_ctx = bnxt_ulp_eth_dev_ptr2_cntxt_get(eth_dev); if (!ulp_ctx) { BNXT_TF_DBG(ERR, - "ULP context is not initialized. Failed to create dflt flow.\n"); + "ULP is not init'ed. Fail to create dflt flow.\n"); return -EINVAL; } diff --git a/drivers/net/bnxt/tf_ulp/ulp_flow_db.c b/drivers/net/bnxt/tf_ulp/ulp_flow_db.c index 79dc869e64..9968311c44 100644 --- a/drivers/net/bnxt/tf_ulp/ulp_flow_db.c +++ b/drivers/net/bnxt/tf_ulp/ulp_flow_db.c @@ -470,7 +470,7 @@ ulp_flow_db_init(struct bnxt_ulp_context *ulp_ctxt) } /* All good so return. */ - BNXT_TF_DBG(INFO, "FlowDB initialized with %d flows.\n", + BNXT_TF_DBG(DEBUG, "FlowDB initialized with %d flows.\n", flow_tbl->num_flows); return 0; error_free: diff --git a/drivers/net/bnxt/tf_ulp/ulp_mark_mgr.c b/drivers/net/bnxt/tf_ulp/ulp_mark_mgr.c index 271520e1d3..9dffaef73b 100644 --- a/drivers/net/bnxt/tf_ulp/ulp_mark_mgr.c +++ b/drivers/net/bnxt/tf_ulp/ulp_mark_mgr.c @@ -116,7 +116,7 @@ ulp_mark_db_init(struct bnxt_ulp_context *ctxt) mark_tbl->gfid_mask = (mark_tbl->gfid_num_entries / 2) - 1; mark_tbl->gfid_type_bit = (mark_tbl->gfid_num_entries / 2); - BNXT_TF_DBG(DEBUG, "GFID Max = 0x%08x\nGFID MASK = 0x%08x\n", + BNXT_TF_DBG(DEBUG, "GFID Max = 0x%08x GFID MASK = 0x%08x\n", mark_tbl->gfid_num_entries - 1, mark_tbl->gfid_mask); -- 2.30.1 (Apple Git-130) --00000000000087d5b005cfd7d587--