From patchwork Tue May 11 02:11:16 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Alvin Zhang X-Patchwork-Id: 93124 X-Patchwork-Delegate: qi.z.zhang@intel.com 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 EC59DA0C3F; Tue, 11 May 2021 04:11:27 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 7E8804003E; Tue, 11 May 2021 04:11:27 +0200 (CEST) Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by mails.dpdk.org (Postfix) with ESMTP id 55DA04003C for ; Tue, 11 May 2021 04:11:26 +0200 (CEST) IronPort-SDR: fw5vUAI7LGbARLz3bDuatwOmsoDLuynUsYphxJRhSYueZ8+t+2OGgL50Y1IDM/KIP4H9gis8VU 4Vpz1wPIz1NA== X-IronPort-AV: E=McAfee;i="6200,9189,9980"; a="199010812" X-IronPort-AV: E=Sophos;i="5.82,290,1613462400"; d="scan'208";a="199010812" Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by fmsmga103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 10 May 2021 19:11:24 -0700 IronPort-SDR: E+Vh0psWwFB4W6yFTCI+7BxrGegXGO3GZIS37K0PbCThtCKotYfbEyJTO9zSIITnDuAUSFWUdX xJOZVeVd2N6g== X-IronPort-AV: E=Sophos;i="5.82,290,1613462400"; d="scan'208";a="454749087" Received: from shwdenpg235.ccr.corp.intel.com ([10.240.182.60]) by fmsmga004-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 10 May 2021 19:11:21 -0700 From: Alvin Zhang To: beilei.xing@intel.com, Ting.Xu@intel.com Cc: dev@dpdk.org, Alvin Zhang Date: Tue, 11 May 2021 10:11:16 +0800 Message-Id: <20210511021116.30844-1-alvinx.zhang@intel.com> X-Mailer: git-send-email 2.21.0.windows.1 In-Reply-To: <20210510122050.32668-2-alvinx.zhang@intel.com> References: <20210510122050.32668-2-alvinx.zhang@intel.com> MIME-Version: 1.0 Subject: [dpdk-dev] [PATCH v2] net/iavf: fix error logs 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" Remove some useless logs which are duplicated of the lower-level function logs and may confuse users when running VF without some features that not supported by PF. Fixes: 5a038d19962d ("net/iavf: fix RSS configuration on i40e VF") Fixes: 95f2f0e9fc2a ("net/iavf: improve default RSS") Signed-off-by: Alvin Zhang --- drivers/net/iavf/iavf_vchnl.c | 13 ++----------- 1 file changed, 2 insertions(+), 11 deletions(-) diff --git a/drivers/net/iavf/iavf_vchnl.c b/drivers/net/iavf/iavf_vchnl.c index ca5c56e..dfa7ce3 100644 --- a/drivers/net/iavf/iavf_vchnl.c +++ b/drivers/net/iavf/iavf_vchnl.c @@ -1519,11 +1519,8 @@ args.out_size = IAVF_AQ_BUF_SZ; err = iavf_execute_vf_cmd(adapter, &args); - if (err) { - PMD_DRV_LOG(ERR, - "Failed to execute command of OP_GET_RSS_HENA_CAPS"); + if (err) return err; - } *caps = ((struct virtchnl_rss_hena *)args.out_buffer)->hena; return 0; @@ -1535,7 +1532,6 @@ struct iavf_info *vf = IAVF_DEV_PRIVATE_TO_VF(adapter); struct virtchnl_rss_hena vrh; struct iavf_cmd_info args; - int err; vrh.hena = hena; args.ops = VIRTCHNL_OP_SET_RSS_HENA; @@ -1544,12 +1540,7 @@ args.out_buffer = vf->aq_resp; args.out_size = IAVF_AQ_BUF_SZ; - err = iavf_execute_vf_cmd(adapter, &args); - if (err) - PMD_DRV_LOG(ERR, - "Failed to execute command of OP_SET_RSS_HENA"); - - return err; + return iavf_execute_vf_cmd(adapter, &args); } int