|SUCCESS| maintainers: fix app/test attributions
dpdklab at iol.unh.edu
dpdklab at iol.unh.edu
Thu Apr 30 01:33:34 CEST 2026
Test-Label: ai-code-review
Test-Status: SUCCESS
https://dpdk.org/patch/163397
_AI Code Review_
=== Patch Review: series.patch (via Claude) ===
Review date: 2026-04-29
git sha 20260429162913.272259-1-stephen at networkplumber.org
Author: Stephen Hemminger <stephen at networkplumber.org>
Subject: [PATCH] maintainers: fix app/test attributions
This patch reclassifies ownership of two test files in MAINTAINERS:
test_event_ring.c moves from "Eventdev test application" to "Eventdev
API", and the hash test glob is narrowed to exclude crypto hash test
vectors.
> MAINTAINERS | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 0f5539f851..928e695abb 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -540,6 +540,7 @@ F: lib/eventdev/
> F: doc/guides/prog_guide/eventdev/eventdev.rst
> F: drivers/event/skeleton/
> F: app/test/test_eventdev.c
> +F: app/test/test_event_ring.c
> F: examples/l3fwd/l3fwd_event*
>
> Eventdev Ethdev Rx Adapter API
[ ... ]
> @@ -1754,7 +1755,8 @@ M: Vladimir Medvedkin <vladimir.medvedkin at intel.com>
> F: lib/hash/
> F: doc/guides/prog_guide/hash_lib.rst
> F: doc/guides/prog_guide/toeplitz_hash_lib.rst
> -F: app/test/test_*hash*
> +F: app/test/test_hash*
> +F: app/test/test_thash*
> F: app/test/test_func_reentrancy.c
The original "test_*hash*" glob could match both "test_hash" and
"test_thash" files in a single pattern. Does splitting this into two
separate globs change the coverage? Could there be test files following
a "test_Xhash" pattern (where X is neither 't' nor empty) that would no
longer match either of the new globs?
[ ... ]
> @@ -1985,7 +1987,6 @@ T: git://dpdk.org/next/dpdk-next-eventdev
> F: app/test-eventdev/
> F: doc/guides/tools/testeventdev.rst
> F: doc/guides/tools/img/eventdev_*
> -F: app/test/test_event_ring.c
More information about the test-report
mailing list