|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