From patchwork Mon Apr 22 15:21:30 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Hemminger X-Patchwork-Id: 883 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 AADBC43EDB; Mon, 22 Apr 2024 17:23:48 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 79DE340268; Mon, 22 Apr 2024 17:23:48 +0200 (CEST) Received: from mail-pf1-f177.google.com (mail-pf1-f177.google.com [209.85.210.177]) by mails.dpdk.org (Postfix) with ESMTP id 1108B40265 for ; Mon, 22 Apr 2024 17:23:47 +0200 (CEST) Received: by mail-pf1-f177.google.com with SMTP id d2e1a72fcca58-6f0b9f943cbso2842690b3a.0 for ; Mon, 22 Apr 2024 08:23:46 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1713799426; x=1714404226; darn=dpdk.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=DrGEW9aXVYR7h59aVQQ8SNdeMVmDvz48u+/XaE+CxUU=; b=dNZjrUcuil0KlQdTE3Zsm3dZTn8H8u3AGE2W9sU3ElbT80tD/myYqGNCOuR6wJiI81 r8D+9c+ppoF6JFJ9CkG+JRZ+b1RkYsjPo8/7gQaeDKl3doSguzUBOMb3SujdoUSPPfq9 U1TfnKA9xpmwHw+7aiRLZAK42DKtuKDg+9ipPCKvBJ5rguCPu3SnghxyYZbezmepBUAp JtpnoEOTOYzktlbnkTcz3DT8AqWP4CFK2b7eXPMx9hzyceL3FLtIEZaxQPwuuIjN35m1 rN2SnbmUmnQdB7VQyhBCuTe1mh57mmPB6Y4D6B6WoFjqg/yxe2S44uQzIt2nW0aKyCjr WwWg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1713799426; x=1714404226; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=DrGEW9aXVYR7h59aVQQ8SNdeMVmDvz48u+/XaE+CxUU=; b=bQUOOG9N05lVtC1qqUeR0qs94rlVHrXUbNeVVMqgs4yr815rLOmLuunzRuMe4ni58J lklk16Qe8kxY06GDEXLhnFkapmugCZCnL5+Bf3JrWgtUNwW2j6i6FkC990kdWqqI6DrK QFoN04pP8zVIe0ADUigiWxQ+jc64RDvLXfALsQjXySv+TbzhOhHSY4y1f9upSDpgnMc9 8bgaxX375vPPqjRsbHtT3ymihVvOPMTAObNWYEcuoGNcg/n41VRrXHZ7TUBW46WNsIgj vtXxStZjo6nCjdiMFcahQlhk9dPhysrV+KIKUtudiYC7ZrdDM1JmOmDCW9Sw47NnuJjj ZDPw== X-Gm-Message-State: AOJu0YzvZcJ1ow8whKq3MxORaXLwJ589ya7DRB5vLylAde7dJWhiYI2v ATnGwWeMlTIYJKUhOcAfBhGVHNWm6BGQr9THhJX6OU40aWib2v6ypoSoycUMxj2C2/N0joaoxFh A X-Google-Smtp-Source: AGHT+IGk24Uq9Tx1YiurS6n0PX59qbgrK7y5cImH6bG+q5glxmLvfJ/VNGQSFR+CKlNaQRCWaufRQg== X-Received: by 2002:a05:6a20:914d:b0:1aa:2917:4b79 with SMTP id x13-20020a056a20914d00b001aa29174b79mr11720582pzc.15.1713799426125; Mon, 22 Apr 2024 08:23:46 -0700 (PDT) Received: from hermes.local (204-195-96-226.wavecable.com. [204.195.96.226]) by smtp.gmail.com with ESMTPSA id cb17-20020a056a02071100b0060063c4be3bsm1402783pgb.14.2024.04.22.08.23.45 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 22 Apr 2024 08:23:45 -0700 (PDT) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger Subject: [PATCH v5 0/9] latencystats: improve algorithms and tests Date: Mon, 22 Apr 2024 08:21:30 -0700 Message-ID: <20240422152336.147553-1-stephen@networkplumber.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240408195036.182545-1-stephen@networkplumber.org> References: <20240408195036.182545-1-stephen@networkplumber.org> 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 Latencystats uses variable length array and floating point when they are not necessary to acheive the same result. While testing also noticed that the code was computing wrong values on my test system, and that include files were missing. v5 - use scaled values for latency and jitter - add more test cases - review feedback Stephen Hemminger (9): latencystats: replace use of VLA latencystats: handle fractional cycles per ns latencystats: do not use floating point latencystats: fix log messages latencystats: update include files latencystats: enforce that unused callback function is NULL latencystats: add metric for number of samples test: use initialization in latencystats test test: add more latencystats tests app/test/test_latencystats.c | 84 ++++++---- lib/latencystats/rte_latencystats.c | 232 ++++++++++++++++------------ 2 files changed, 188 insertions(+), 128 deletions(-)