From patchwork Fri May 8 01:36:50 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Simei Su X-Patchwork-Id: 69964 X-Patchwork-Delegate: xiaolong.ye@intel.com Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 1E0A4A00C5; Fri, 8 May 2020 03:41:16 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 431421DB9B; Fri, 8 May 2020 03:41:15 +0200 (CEST) Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by dpdk.org (Postfix) with ESMTP id 1C9A81DB8F for ; Fri, 8 May 2020 03:41:12 +0200 (CEST) IronPort-SDR: I/y7HVQXqzH7fKoxBE6KbezzFLw9TZshe3ojiI8zXHF4fob7+vOUB3Rj9aK47mFvgerzNpwiDh troI9swh2N3w== X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga005.jf.intel.com ([10.7.209.41]) by orsmga106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 07 May 2020 18:41:12 -0700 IronPort-SDR: giMLkrwn2xVDCj3aAwjlz/qtxv3vYnbdile9CmUyOiB2peekOdXEJXsCozZ2k8S3H2cNAE3ad4 vG1AgQm5yDlA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.73,366,1583222400"; d="scan'208";a="435512205" Received: from unknown (HELO npg-dpdk-cvl-simeisu-118d193.sh.intel.com) ([10.67.110.178]) by orsmga005.jf.intel.com with ESMTP; 07 May 2020 18:41:10 -0700 From: Simei Su To: qi.z.zhang@intel.com, xiaolong.ye@intel.com Cc: dev@dpdk.org, yahui.cao@intel.com, simei.su@intel.com Date: Fri, 8 May 2020 09:36:50 +0800 Message-Id: <1588901810-381405-1-git-send-email-simei.su@intel.com> X-Mailer: git-send-email 1.8.3.1 Subject: [dpdk-dev] [PATCH] net/ice/base: fix FDIR rule completion report value 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: , Errors-To: dev-bounces@dpdk.org Sender: "dev" IAVF FDIR needs to check program status from FD completion descriptors for both successful and failed case, rather than only ask for completion on error. This patch corrects completion report value so that the called can determine whether to report completion on error or other cases. Fixes: 55daca4e45fc ("net/ice/base: change function to static") Signed-off-by: Simei Su Acked-by: Qi Zhang Tested-by: Zeng,Xiaoxiao --- drivers/net/ice/base/ice_fdir.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ice/base/ice_fdir.c b/drivers/net/ice/base/ice_fdir.c index 33a1732..f899641 100644 --- a/drivers/net/ice/base/ice_fdir.c +++ b/drivers/net/ice/base/ice_fdir.c @@ -488,7 +488,7 @@ ICE_FXD_FLTR_QW1_PCMD_REMOVE; fdir_fltr_ctx.swap = ICE_FXD_FLTR_QW1_SWAP_NOT_SET; fdir_fltr_ctx.comp_q = ICE_FXD_FLTR_QW0_COMP_Q_ZERO; - fdir_fltr_ctx.comp_report = ICE_FXD_FLTR_QW0_COMP_REPORT_SW_FAIL; + fdir_fltr_ctx.comp_report = ICE_FXD_FLTR_QW0_COMP_REPORT_SW; fdir_fltr_ctx.fdid_prio = input->fdid_prio; fdir_fltr_ctx.desc_prof = 1; fdir_fltr_ctx.desc_prof_prio = 3;