[PATCH] tests/TestSuite_crypto_perf_cryptodev_perf: fix output parsing

[PATCH] tests/TestSuite_crypto_perf_cryptodev_perf: fix output parsing

[David Marchand]

The parsing of the test results was considering too many lines.

It was not an issue as the extra inspected line was an empty line.
Following fixes on the qat driver logs, this triggers a parsing error.

Fixes: a3c915c1a3b9 ("tests/crypto_perf_cryptodev_perf: optimize parse output")
Signed-off-by: David Marchand <david.marchand@redhat.com>
---
 tests/TestSuite_crypto_perf_cryptodev_perf.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/TestSuite_crypto_perf_cryptodev_perf.py b/tests/TestSuite_crypto_perf_cryptodev_perf.py
index 0531b9c6..633dd637 100644
--- a/tests/TestSuite_crypto_perf_cryptodev_perf.py
+++ b/tests/TestSuite_crypto_perf_cryptodev_perf.py
@@ -252,7 +252,7 @@ class TestCryptoPerfCryptodevPerf(TestCase):
                 if lines[line_index].startswith("    lcore id"):
                     self.logger.debug("data output line from: " + str(line_index))
                     break
-            data_line = line_index - 2
+            data_line = line_index - 1
 
             results = []
             pattern = re.compile(r"\s+")
-- 
2.46.0

Re: [PATCH] tests/TestSuite_crypto_perf_cryptodev_perf: fix output parsing

[Patrick Robb]

Reviewed-by: Patrick Robb <probb@iol.unh.edu>
Tested-by: Patrick Robb <probb@iol.unh.edu>

Did a non-regression test against origin/main, and tested with
https://patchwork.dpdk.org/project/dpdk/list/?series=33005, which is
the series which prompted this DTS patch.

Re: [PATCH] tests/TestSuite_crypto_perf_cryptodev_perf: fix output parsing

[David Marchand]

On Mon, Sep 16, 2024 at 11:25 AM David Marchand
<david.marchand@redhat.com> wrote:
>
> The parsing of the test results was considering too many lines.
>
> It was not an issue as the extra inspected line was an empty line.
> Following fixes on the qat driver logs, this triggers a parsing error.
>
> Fixes: a3c915c1a3b9 ("tests/crypto_perf_cryptodev_perf: optimize parse output")
> Signed-off-by: David Marchand <david.marchand@redhat.com>
Reviewed-by: Patrick Robb <probb@iol.unh.edu>
Tested-by: Patrick Robb <probb@iol.unh.edu>

Applied, thanks.


-- 
David Marchand