2020-03-17 07:16:58

by Daniel Vetter

Subject: [PATCH] MAINTAINERS: Better regex for dma_buf|fence|resv

We're getting some random other stuff that we're not relly interested
in, so match only word boundaries. Also avoid the capture group while
at it.

Suggested by Joe Perches.

Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
Cc: Joe Perches <[email protected]>
Cc: Sumit Semwal <[email protected]>
Signed-off-by: Daniel Vetter <[email protected]>
---
MAINTAINERS | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 3005be638c2c..fc5d5aa53147 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -5025,7 +5025,7 @@ F: include/linux/dma-buf*
F: include/linux/reservation.h
F: include/linux/*fence.h
F: Documentation/driver-api/dma-buf.rst
-K: dma_(buf|fence|resv)
+K: '\bdma_(?:buf|fence|resv)\b'
T: git git://anongit.freedesktop.org/drm/drm-misc

DMA-BUF HEAPS FRAMEWORK
--
2.25.1

2020-03-17 08:19:43

by Sam Ravnborg

Subject: Re: [PATCH] MAINTAINERS: Better regex for dma_buf|fence|resv

Hi Daniel.

On Tue, Mar 17, 2020 at 08:15:47AM +0100, Daniel Vetter wrote:
> We're getting some random other stuff that we're not relly interested
really

> in, so match only word boundaries. Also avoid the capture group while
> at it.
>
> Suggested by Joe Perches.

You want a:
Suggested-by: Joe Perches <[email protected]>

The patch looks correct, but I am not fluent in perlish.

Sam
>
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: Joe Perches <[email protected]>
> Cc: Sumit Semwal <[email protected]>
> Signed-off-by: Daniel Vetter <[email protected]>
> ---
> MAINTAINERS | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 3005be638c2c..fc5d5aa53147 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -5025,7 +5025,7 @@ F: include/linux/dma-buf*
> F: include/linux/reservation.h
> F: include/linux/*fence.h
> F: Documentation/driver-api/dma-buf.rst
> -K: dma_(buf|fence|resv)
> +K: '\bdma_(?:buf|fence|resv)\b'
> T: git git://anongit.freedesktop.org/drm/drm-misc
>
> DMA-BUF HEAPS FRAMEWORK
> --
> 2.25.1
>
> _______________________________________________
> dri-devel mailing list
> [email protected]
> https://lists.freedesktop.org/mailman/listinfo/dri-devel

2020-03-17 12:51:53

by Joe Perches

Subject: Re: [PATCH] MAINTAINERS: Better regex for dma_buf|fence|resv

On Tue, 2020-03-17 at 08:15 +0100, Daniel Vetter wrote:
> We're getting some random other stuff that we're not relly interested
> in, so match only word boundaries. Also avoid the capture group while
> at it.
[]
> diff --git a/MAINTAINERS b/MAINTAINERS
[]
> @@ -5025,7 +5025,7 @@ F: include/linux/dma-buf*
> F: include/linux/reservation.h
> F: include/linux/*fence.h
> F: Documentation/driver-api/dma-buf.rst
> -K: dma_(buf|fence|resv)
> +K: '\bdma_(?:buf|fence|resv)\b'

You don't want the single quotes in the K: entry

K: \bdma_(?:buf|fence|resv)\b

My mistake for adding them in the initial suggestion.