From patchwork Wed May 6 11:29:43 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Dybkowski, AdamX" X-Patchwork-Id: 69824 X-Patchwork-Delegate: gakhil@marvell.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 BDC2CA034F; Wed, 6 May 2020 13:37:49 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id C7D3F1D8ED; Wed, 6 May 2020 13:37:48 +0200 (CEST) Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by dpdk.org (Postfix) with ESMTP id 4BB381D6E5 for ; Wed, 6 May 2020 13:37:47 +0200 (CEST) IronPort-SDR: Kxe255eUsdrszxxUtPsc/4tXdQUiOEDgZVQsTskZo4BAS2DNCsHHx3fDq3Mm9l2iITPM0ZUzck kt6j56sthe/g== X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga002.jf.intel.com ([10.7.209.21]) by orsmga103.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 06 May 2020 04:37:46 -0700 IronPort-SDR: ycR4E5HjsJee6bMM4dKQNDG+wztEJKdeGKRaRCsKJCdE2EEu4YFmipOBtO/GtV4AV7jMK9Q4qD hqlmaUo5kP8A== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.73,358,1583222400"; d="scan'208";a="278204369" Received: from adamdybx-mobl.ger.corp.intel.com (HELO localhost.localdomain) ([10.104.125.123]) by orsmga002.jf.intel.com with ESMTP; 06 May 2020 04:37:44 -0700 From: Adam Dybkowski To: dev@dpdk.org, fiona.trahe@intel.com, akhil.goyal@nxp.com Cc: Adam Dybkowski Date: Wed, 6 May 2020 13:29:43 +0200 Message-Id: <20200506112943.25385-1-adamx.dybkowski@intel.com> X-Mailer: git-send-email 2.17.1 Subject: [dpdk-dev] [PATCH] common/qat: fix stats update 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" This patch fixes enqueued and dequeued count statistics that should contain the numer of operations enqueued by the end user app instead of the total number of QAT requests - bigger in case of a multiple-request dynamic Huffman compression operation. Fixes: c13cecf60f12 ("compress/qat: support IM buffer too small operation") Signed-off-by: Adam Dybkowski Acked-by: Fiona Trahe --- Requires applying other patch first: http://patches.dpdk.org/patch/69765/ --- drivers/common/qat/qat_qp.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/common/qat/qat_qp.c b/drivers/common/qat/qat_qp.c index 098b99786..9d7091af3 100644 --- a/drivers/common/qat/qat_qp.c +++ b/drivers/common/qat/qat_qp.c @@ -818,7 +818,7 @@ qat_enqueue_comp_op_burst(void *qp, void **ops, uint16_t nb_ops) kick_tail: queue->tail = tail; tmp_qp->enqueued += total_descriptors_built; - tmp_qp->stats.enqueued_count += total_descriptors_built; + tmp_qp->stats.enqueued_count += nb_ops_sent; txq_write_tail(tmp_qp, queue); return nb_ops_sent; } @@ -880,7 +880,7 @@ qat_dequeue_op_burst(void *qp, void **ops, uint16_t nb_ops) } tmp_qp->dequeued += fw_resp_counter; - tmp_qp->stats.dequeued_count += fw_resp_counter; + tmp_qp->stats.dequeued_count += op_resp_counter; rx_queue->head = head; if (rx_queue->nb_processed_responses > QAT_CSR_HEAD_WRITE_THRESH)