net/ice: change RTE log level to DEBUG

Message ID 20211026084938.1684619-1-dapengx.yu@intel.com (mailing list archive)
State Superseded, archived
Delegated to: Qi Zhang
Headers
Series net/ice: change RTE log level to DEBUG |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/iol-broadcom-Functional success Functional Testing PASS
ci/iol-broadcom-Performance success Performance Testing PASS
ci/github-robot: build success github build: passed
ci/iol-x86_64-unit-testing success Testing PASS
ci/iol-x86_64-compile-testing success Testing PASS
ci/iol-mellanox-Performance success Performance Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-aarch64-compile-testing success Testing PASS
ci/iol-aarch64-unit-testing success Testing PASS

Commit Message

Yu, DapengX Oct. 26, 2021, 8:49 a.m. UTC
  From: Dapeng Yu <dapengx.yu@intel.com>

When VF is reset in high frequency, the AdminQ command may fail, but
the failure can be recovered actually. So the error messages for getting
VF resource, getting VSI map, and re-directing flows should be
suppressed, in order to avoid causing concern for DCF users.

Fixes: 7564d5509611 ("net/ice: add DCF hardware initialization")
Cc: stable@dpdk.org

Signed-off-by: Dapeng Yu <dapengx.yu@intel.com>
---
 drivers/net/ice/ice_dcf.c           | 8 ++++----
 drivers/net/ice/ice_generic_flow.c  | 2 +-
 drivers/net/ice/ice_switch_filter.c | 4 ++--
 3 files changed, 7 insertions(+), 7 deletions(-)
  

Patch

diff --git a/drivers/net/ice/ice_dcf.c b/drivers/net/ice/ice_dcf.c
index 084f7a53db..9a53ba3538 100644
--- a/drivers/net/ice/ice_dcf.c
+++ b/drivers/net/ice/ice_dcf.c
@@ -241,7 +241,7 @@  ice_dcf_get_vf_resource(struct ice_dcf_hw *hw)
 	err = ice_dcf_send_cmd_req_no_irq(hw, VIRTCHNL_OP_GET_VF_RESOURCES,
 					  (uint8_t *)&caps, sizeof(caps));
 	if (err) {
-		PMD_DRV_LOG(ERR, "Failed to send msg OP_GET_VF_RESOURCE");
+		PMD_DRV_LOG(DEBUG, "Failed to send msg OP_GET_VF_RESOURCE");
 		return err;
 	}
 
@@ -249,7 +249,7 @@  ice_dcf_get_vf_resource(struct ice_dcf_hw *hw)
 					  (uint8_t *)hw->vf_res,
 					  ICE_DCF_VF_RES_BUF_SZ, NULL);
 	if (err) {
-		PMD_DRV_LOG(ERR, "Failed to get response of OP_GET_VF_RESOURCE");
+		PMD_DRV_LOG(DEBUG, "Failed to get response of OP_GET_VF_RESOURCE");
 		return -1;
 	}
 
@@ -283,7 +283,7 @@  ice_dcf_get_vf_vsi_map(struct ice_dcf_hw *hw)
 	err = ice_dcf_send_cmd_req_no_irq(hw, VIRTCHNL_OP_DCF_GET_VSI_MAP,
 					  NULL, 0);
 	if (err) {
-		PMD_DRV_LOG(ERR, "Failed to send msg OP_DCF_GET_VSI_MAP");
+		PMD_DRV_LOG(DEBUG, "Failed to send msg OP_DCF_GET_VSI_MAP");
 		return err;
 	}
 
@@ -291,7 +291,7 @@  ice_dcf_get_vf_vsi_map(struct ice_dcf_hw *hw)
 					  hw->arq_buf, ICE_DCF_AQ_BUF_SZ,
 					  &len);
 	if (err) {
-		PMD_DRV_LOG(ERR, "Failed to get response of OP_DCF_GET_VSI_MAP");
+		PMD_DRV_LOG(DEBUG, "Failed to get response of OP_DCF_GET_VSI_MAP");
 		return err;
 	}
 
diff --git a/drivers/net/ice/ice_generic_flow.c b/drivers/net/ice/ice_generic_flow.c
index 02f854666a..4aca24de89 100644
--- a/drivers/net/ice/ice_generic_flow.c
+++ b/drivers/net/ice/ice_generic_flow.c
@@ -2551,7 +2551,7 @@  ice_flow_redirect(struct ice_adapter *ad,
 			continue;
 		ret = p_flow->engine->redirect(ad, p_flow, rd);
 		if (ret) {
-			PMD_DRV_LOG(ERR, "Failed to redirect flows");
+			PMD_DRV_LOG(DEBUG, "Failed to redirect flows");
 			break;
 		}
 	}
diff --git a/drivers/net/ice/ice_switch_filter.c b/drivers/net/ice/ice_switch_filter.c
index 6b0c1bff1e..b4f3bbafff 100644
--- a/drivers/net/ice/ice_switch_filter.c
+++ b/drivers/net/ice/ice_switch_filter.c
@@ -1943,7 +1943,7 @@  ice_switch_redirect(struct ice_adapter *ad,
 	ret = ice_rem_adv_rule(hw, list_itr->lkups,
 			       lkups_cnt, &rinfo);
 	if (ret) {
-		PMD_DRV_LOG(ERR, "Failed to delete the old rule %d",
+		PMD_DRV_LOG(DEBUG, "Failed to delete the old rule %d",
 			    rdata->rule_id);
 		ret = -EINVAL;
 		goto out;
@@ -1956,7 +1956,7 @@  ice_switch_redirect(struct ice_adapter *ad,
 	ret = ice_add_adv_rule(hw, lkups_dp, lkups_cnt,
 			       &rinfo, rdata);
 	if (ret) {
-		PMD_DRV_LOG(ERR, "Failed to replay the rule");
+		PMD_DRV_LOG(DEBUG, "Failed to replay the rule");
 		ret = -EINVAL;
 	}