On Fri, 2020-07-10 at 21:04 +0200, Alexander A. Klimov wrote:
> Rationale:
> Documentation/arm/ixp4xx.rst contains "xdp" as part of "ixdp465"
> which has nothing to do with XDP.
> 
> Signed-off-by: Alexander A. Klimov <grandmas...@al2klimov.de>
> ---
>  Better?
> 
>  MAINTAINERS | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 1d4aa7f942de..735e2475e926 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -18708,8 +18708,8 @@ F:    include/trace/events/xdp.h
>  F:   kernel/bpf/cpumap.c
>  F:   kernel/bpf/devmap.c
>  F:   net/core/xdp.c
> -N:   xdp
> -K:   xdp
> +N:   (?:\b|_)xdp
> +K:   (?:\b|_)xdp

Generally, it's better to have comprehensive files lists
rather than adding name matching regexes.

Perhaps:
---
 MAINTAINERS | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 16854e47e8cb..2e96cbf15b31 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -18763,13 +18763,19 @@ M:    John Fastabend <john.fastab...@gmail.com>
 L:     net...@vger.kernel.org
 L:     b...@vger.kernel.org
 S:     Supported
-F:     include/net/xdp.h
+F:     Documentation/networking/af_xdp.rst
+F:     include/net/xdp*
 F:     include/trace/events/xdp.h
+F:     include/uapi/linux/if_xdp.h
+F:     include/uapi/linux/xdp_diag.h
 F:     kernel/bpf/cpumap.c
 F:     kernel/bpf/devmap.c
 F:     net/core/xdp.c
-N:     xdp
-K:     xdp
+F:     net/xdp/
+F:     samples/bpf/xdp*
+F:     tools/testing/selftests/bfp/*xdp*
+F:     tools/testing/selftests/bfp/*/*xdp*
+K:     (?:\b|_)xdp(?:\b|_)
 
 XDP SOCKETS (AF_XDP)
 M:     Björn Töpel <bjorn.to...@intel.com>


Reply via email to