From: Andrew Bailey <abailey@iol.unh.edu>
To: luca.vizzarro@arm.com, patrickrobb1997@gmail.com
Cc: dev@dpdk.org, lylavoie@iol.unh.edu, ahassick@iol.unh.edu,
knimoji@iol.unh.edu, Andrew Bailey <abailey@iol.unh.edu>
Subject: [PATCH v1 1/2] dts: update parsing for cryptodev latency
Date: Wed, 13 May 2026 11:27:14 -0400 [thread overview]
Message-ID: <20260513152715.133381-1-abailey@iol.unh.edu> (raw)
In-Reply-To: <20260428181501.72434-1-abailey@iol.unh.edu>
Previously, the parsing for cryptodev latency output would fail to parse
due to an incorrect regex. This commit updates this regex to properly
capture the burst size attribute for latency results.
Bugzilla ID: 1938
Fixes: 8ee2df9da125 ("dts: add cryptodev package")
Signed-off-by: Andrew Bailey <abailey@iol.unh.edu>
---
dts/api/cryptodev/__init__.py | 3 +--
dts/api/cryptodev/types.py | 4 ++--
2 files changed, 3 insertions(+), 4 deletions(-)
diff --git a/dts/api/cryptodev/__init__.py b/dts/api/cryptodev/__init__.py
index a4fafc3713..15cffcb409 100644
--- a/dts/api/cryptodev/__init__.py
+++ b/dts/api/cryptodev/__init__.py
@@ -117,7 +117,6 @@ def run_app(self, num_vfs: int = 1) -> list[CryptodevResults]:
f"dependencies missing for virtual device {self._app_params['vdevs'][0].name}"
)
raise e
-
regex = r"^\s+\d+.*$"
parser_options = re.MULTILINE
parser: type[CryptodevResults]
@@ -126,7 +125,7 @@ def run_app(self, num_vfs: int = 1) -> list[CryptodevResults]:
case TestType.throughput:
parser = ThroughputResults
case TestType.latency:
- regex = r"total operations:.*time[^\n]*"
+ regex = r"total operations:.*?time[^\n]*"
parser_options |= re.DOTALL
parser = LatencyResults
case TestType.pmd_cyclecount:
diff --git a/dts/api/cryptodev/types.py b/dts/api/cryptodev/types.py
index df73a86fa4..7b4fd17cb6 100644
--- a/dts/api/cryptodev/types.py
+++ b/dts/api/cryptodev/types.py
@@ -65,11 +65,11 @@ class LatencyResults(CryptodevResults):
#: buffer size ran with app
buffer_size: int = field(
- metadata=TextParser.find_int(r"Buf(?:.*\n\s+\d+\s+)?(?:fer size:\s+)?(\d+)"),
+ metadata=TextParser.find_int(r"Buffer size:\s+(\d+)"),
)
#: burst size ran with app
burst_size: int = field(
- metadata=TextParser.find_int(rf"Burst(?:.*\n\s+\d+\s+){2}?(?: size:\s+)?(\d+)"),
+ metadata=TextParser.find_int(r"Burst size:\s+(\d+)"),
)
#: total operations ran
total_ops: int = field(metadata=TextParser.find_int(r"total operations:\s+(\d+)"))
--
2.50.1
next prev parent reply other threads:[~2026-05-13 15:27 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-04-28 18:15 [RFC PATCH 1/2] dts: update parsing for cryptodev latency Andrew Bailey
2026-04-28 18:15 ` [RFC PATCH 2/2] dts: add latency coverage for cryptodev testing Andrew Bailey
2026-05-13 15:27 ` Andrew Bailey [this message]
2026-05-13 15:27 ` [PATCH v1 " Andrew Bailey
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260513152715.133381-1-abailey@iol.unh.edu \
--to=abailey@iol.unh.edu \
--cc=ahassick@iol.unh.edu \
--cc=dev@dpdk.org \
--cc=knimoji@iol.unh.edu \
--cc=luca.vizzarro@arm.com \
--cc=lylavoie@iol.unh.edu \
--cc=patrickrobb1997@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox