From patchwork Wed Dec 13 11:48:22 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Bruce Richardson X-Patchwork-Id: 135134 X-Patchwork-Delegate: jerinj@marvell.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 49656436DE; Wed, 13 Dec 2023 12:48:42 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id D335B42E8D; Wed, 13 Dec 2023 12:48:41 +0100 (CET) Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.12]) by mails.dpdk.org (Postfix) with ESMTP id 5CC514021D for ; Wed, 13 Dec 2023 12:48:40 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1702468121; x=1734004121; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=Op6Epc593MRIF8jB2CQ0jRpGSQpGeUVYgO4fJMciar4=; b=giCCAFSxw8qym0/ANL+jNF5ikj4HnQUjvTw400Zb8oAd4qQNh2T7hmR5 wlO9jFWV55DTjYVxkH2KuhpMDQ3SfaRbah5amF1MhO0hyLu41PU7OTzJ3 3vKQ4t6TZXwkwkG0KnTLjLHwg4/zV/ohx44qQU7EIJS4PENJif9zlPKpc Vqt2cwRY/vjsexwh5afiSFsor0HApS/K3uq5wGnKuT8gU+ed0GoHtOuS4 /mhfgx7xy/SXGJjfwsGRKIpxoaF2PTr2kKpn9I/41/mgOaU4EFluJk00h 9SqtuwOaLGGud7SXA5+8TDysUBFptnfKCyEyjKEug0En13f4lWYI5+xeJ g==; X-IronPort-AV: E=McAfee;i="6600,9927,10922"; a="2092569" X-IronPort-AV: E=Sophos;i="6.04,272,1695711600"; d="scan'208";a="2092569" Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by orvoesa104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 13 Dec 2023 03:48:31 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10922"; a="891989128" X-IronPort-AV: E=Sophos;i="6.04,272,1695711600"; d="scan'208";a="891989128" Received: from unknown (HELO silpixa00401385.ir.intel.com) ([10.237.214.152]) by fmsmga002.fm.intel.com with ESMTP; 13 Dec 2023 03:48:30 -0800 From: Bruce Richardson To: dev@dpdk.org Cc: Bruce Richardson , Abdullah Sevincer Subject: [PATCH] event/dlb2: remove unnecessary timer call Date: Wed, 13 Dec 2023 11:48:22 +0000 Message-Id: <20231213114822.12440-1-bruce.richardson@intel.com> X-Mailer: git-send-email 2.40.1 MIME-Version: 1.0 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 We only need to consult the timer value when the driver waits for a timeout, so skip making the call to get the current timer/TSC value if the timeout is zero, as no wait will ever be done. Signed-off-by: Bruce Richardson Acked-by: Abdullah Sevincer --- drivers/event/dlb2/dlb2.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/event/dlb2/dlb2.c b/drivers/event/dlb2/dlb2.c index 050ace0904..f395ef0cb7 100644 --- a/drivers/event/dlb2/dlb2.c +++ b/drivers/event/dlb2/dlb2.c @@ -3966,7 +3966,8 @@ dlb2_hw_dequeue_sparse(struct dlb2_eventdev *dlb2, else timeout = dlb2->global_dequeue_wait_ticks; - start_ticks = rte_get_timer_cycles(); + if (timeout != 0) + start_ticks = rte_get_timer_cycles(); use_scalar = use_scalar || (max_num & 0x3); @@ -4114,7 +4115,8 @@ dlb2_hw_dequeue(struct dlb2_eventdev *dlb2, else timeout = dlb2->global_dequeue_wait_ticks; - start_ticks = rte_get_timer_cycles(); + if (timeout != 0) + start_ticks = rte_get_timer_cycles(); while (num < max_num) { struct dlb2_dequeue_qe qes[DLB2_NUM_QES_PER_CACHE_LINE];