All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/3] MAINTAINERS: Remove asterisk from EFI directory names
  2016-05-03 19:29 [GIT PULL 0/3] EFI urgent fixes Matt Fleming
@ 2016-05-03 19:29     ` Matt Fleming
  0 siblings, 0 replies; 6+ messages in thread
From: Matt Fleming @ 2016-05-03 19:29 UTC (permalink / raw)
  To: Ingo Molnar, Thomas Gleixner, H . Peter Anvin
  Cc: Matt Fleming, Ard Biesheuvel, linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	linux-efi-u79uwXL29TY76Z2rM5mHXA, Catalin Marinas, Mark Rutland,
	stable-u79uwXL29TY76Z2rM5mHXA

Mark reported that having asterisks on the end of directory names
confuses get_maintainer.pl when it encounters subdirectories, and that
my name does not appear when run on drivers/firmware/efi/libstub.

Reported-by: Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>
Cc: Ard Biesheuvel <ard.biesheuvel-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
Cc: Catalin Marinas <catalin.marinas-5wv7dgnIgG8@public.gmane.org>
Cc: <stable-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>
Signed-off-by: Matt Fleming <matt-mF/unelCI9GS6iBeEJttW/XRex20P6io@public.gmane.org>
---
 MAINTAINERS | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 42e65d128d01..4dca3b3895ba 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -4223,8 +4223,8 @@ F:	Documentation/efi-stub.txt
 F:	arch/ia64/kernel/efi.c
 F:	arch/x86/boot/compressed/eboot.[ch]
 F:	arch/x86/include/asm/efi.h
-F:	arch/x86/platform/efi/*
-F:	drivers/firmware/efi/*
+F:	arch/x86/platform/efi/
+F:	drivers/firmware/efi/
 F:	include/linux/efi*.h
 
 EFI VARIABLE FILESYSTEM
-- 
2.7.3

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH 1/3] MAINTAINERS: Remove asterisk from EFI directory names
@ 2016-05-03 19:29     ` Matt Fleming
  0 siblings, 0 replies; 6+ messages in thread
From: Matt Fleming @ 2016-05-03 19:29 UTC (permalink / raw)
  To: Ingo Molnar, Thomas Gleixner, H . Peter Anvin
  Cc: Matt Fleming, Ard Biesheuvel, linux-kernel, linux-efi,
	Catalin Marinas, Mark Rutland, stable

Mark reported that having asterisks on the end of directory names
confuses get_maintainer.pl when it encounters subdirectories, and that
my name does not appear when run on drivers/firmware/efi/libstub.

Reported-by: Mark Rutland <mark.rutland@arm.com>
Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Catalin Marinas <catalin.marinas@arm.com>
Cc: <stable@vger.kernel.org>
Signed-off-by: Matt Fleming <matt@codeblueprint.co.uk>
---
 MAINTAINERS | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 42e65d128d01..4dca3b3895ba 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -4223,8 +4223,8 @@ F:	Documentation/efi-stub.txt
 F:	arch/ia64/kernel/efi.c
 F:	arch/x86/boot/compressed/eboot.[ch]
 F:	arch/x86/include/asm/efi.h
-F:	arch/x86/platform/efi/*
-F:	drivers/firmware/efi/*
+F:	arch/x86/platform/efi/
+F:	drivers/firmware/efi/
 F:	include/linux/efi*.h
 
 EFI VARIABLE FILESYSTEM
-- 
2.7.3

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/3] MAINTAINERS: Remove asterisk from EFI directory names
@ 2016-05-13 15:11 Xose Vazquez Perez
  2016-05-13 15:23 ` Joe Perches
  0 siblings, 1 reply; 6+ messages in thread
From: Xose Vazquez Perez @ 2016-05-13 15:11 UTC (permalink / raw)
  To: Matt Fleming, Mark Rutland, Greg KH, Joe Perches, LKML

Matt Fleming wrote:

> Mark reported that having asterisks on the end of directory names
> confuses get_maintainer.pl when it encounters subdirectories, and that

If this's a get_maintainer.pl bug, it should also happen in:

F:      arch/arm64/boot/dts/qcom/*
F:      arch/mips/bcm47xx/*
F:      arch/mips/bmips/*
F:      arch/mips/include/asm/mach-bcm47xx/*
F:      arch/mips/include/asm/mach-bmips/*
F:      arch/x86/kernel/cpu/mcheck/*
F:      arch/x86/kernel/cpu/microcode/*
F:      arch/x86/net/*
F:      Documentation/devicetree/bindings/arc/*
F:      Documentation/misc-devices/mei/*
F:      Documentation/ptp/*
F:      drivers/block/xen-blkback/*
F:      drivers/char/*
F:      drivers/cpuidle/*
F:      drivers/firmware/broadcom/*
F:      drivers/hwtracing/coresight/*
F:      drivers/media/i2c/s5c73m3/*
F:      drivers/media/pci/netup_unidvb/*
F:      drivers/media/platform/vivid/*
F:      drivers/media/usb/pwc/*
F:      drivers/misc/*
F:      drivers/misc/mei/*
F:      drivers/mtd/nand/gpmi-nand/*
F:      drivers/net/ethernet/pasemi/*
F:      drivers/net/ethernet/tehuti/*
F:      drivers/net/wireless/ath/*
F:      drivers/net/xen-netback/*
F:      drivers/nvdimm/*
F:      drivers/ptp/*
F:      drivers/soc/samsung/*
F:      drivers/soc/ti/*
F:      fs/*
F:      kernel/events/*
F:      net/wireless/*
F:      sound/soc/blackfin/*
F:      sound/soc/samsung/*
F:      tools/testing/selftests/seccomp/*

> my name does not appear when run on drivers/firmware/efi/libstub.
> Reported-by: Mark Rutland <mark.rutland@arm.com>
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> Cc: Catalin Marinas <catalin.marinas@arm.com>
> Cc: <stable@vger.kernel.org>
> Signed-off-by: Matt Fleming <matt@codeblueprint.co.uk>
> ---
>  MAINTAINERS | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 42e65d128d01..4dca3b3895ba 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -4223,8 +4223,8 @@ F:	Documentation/efi-stub.txt
>  F:	arch/ia64/kernel/efi.c
>  F:	arch/x86/boot/compressed/eboot.[ch]
>  F:	arch/x86/include/asm/efi.h
> -F:	arch/x86/platform/efi/*
> -F:	drivers/firmware/efi/*
> +F:	arch/x86/platform/efi/
> +F:	drivers/firmware/efi/
>  F:	include/linux/efi*.h
>  
>  EFI VARIABLE FILESYSTEM
> -- 
> 2.7.3

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/3] MAINTAINERS: Remove asterisk from EFI directory names
  2016-05-13 15:11 [PATCH 1/3] MAINTAINERS: Remove asterisk from EFI directory names Xose Vazquez Perez
@ 2016-05-13 15:23 ` Joe Perches
  2016-05-13 15:34   ` Matt Fleming
  2016-05-13 16:15   ` Xose Vazquez Perez
  0 siblings, 2 replies; 6+ messages in thread
From: Joe Perches @ 2016-05-13 15:23 UTC (permalink / raw)
  To: Xose Vazquez Perez, Matt Fleming, Mark Rutland, Greg KH, LKML

On Fri, 2016-05-13 at 17:11 +0200, Xose Vazquez Perez wrote:
> Matt Fleming wrote:
> 
> > 
> > Mark reported that having asterisks on the end of directory names
> > confuses get_maintainer.pl when it encounters subdirectories, and
> > that
> If this's a get_maintainer.pl bug, it should also happen in:

Mark is confused.

The meaning of * is all the files in a particular
directory but not any subdirectories.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/3] MAINTAINERS: Remove asterisk from EFI directory names
  2016-05-13 15:23 ` Joe Perches
@ 2016-05-13 15:34   ` Matt Fleming
  2016-05-13 16:15   ` Xose Vazquez Perez
  1 sibling, 0 replies; 6+ messages in thread
From: Matt Fleming @ 2016-05-13 15:34 UTC (permalink / raw)
  To: Joe Perches; +Cc: Xose Vazquez Perez, Mark Rutland, Greg KH, LKML

On Fri, 13 May, at 08:23:47AM, Joe Perches wrote:
> On Fri, 2016-05-13 at 17:11 +0200, Xose Vazquez Perez wrote:
> > Matt Fleming wrote:
> > 
> > > 
> > > Mark reported that having asterisks on the end of directory names
> > > confuses get_maintainer.pl when it encounters subdirectories, and
> > > that
> > If this's a get_maintainer.pl bug, it should also happen in:
> 
> Mark is confused.
 
Sorry, "confuses" was my word, not Mark's.

> The meaning of * is all the files in a particular
> directory but not any subdirectories.
 
Exactly. This is not what EFI maintainer entries should be using. I am
responsible for everything under drivers/firmware/efi, including
subdirectories.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/3] MAINTAINERS: Remove asterisk from EFI directory names
  2016-05-13 15:23 ` Joe Perches
  2016-05-13 15:34   ` Matt Fleming
@ 2016-05-13 16:15   ` Xose Vazquez Perez
  1 sibling, 0 replies; 6+ messages in thread
From: Xose Vazquez Perez @ 2016-05-13 16:15 UTC (permalink / raw)
  To: Joe Perches, Matt Fleming, Mark Rutland, Greg KH, LKML

On 05/13/2016 05:23 PM, Joe Perches wrote:

> The meaning of * is all the files in a particular
> directory but not any subdirectories.

Then "*" is only relevant for these entries:
drivers/char/*
drivers/misc/*
drivers/net/wireless/ath/*
fs/*

drivers/cpuidle/* <---- This is a bug, it should be drivers/cpuidle

All the others only have files inside, and "*"
is redundant.

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2016-05-13 16:15 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-05-13 15:11 [PATCH 1/3] MAINTAINERS: Remove asterisk from EFI directory names Xose Vazquez Perez
2016-05-13 15:23 ` Joe Perches
2016-05-13 15:34   ` Matt Fleming
2016-05-13 16:15   ` Xose Vazquez Perez
  -- strict thread matches above, loose matches on Subject: below --
2016-05-03 19:29 [GIT PULL 0/3] EFI urgent fixes Matt Fleming
     [not found] ` <1462303781-8686-1-git-send-email-matt-mF/unelCI9GS6iBeEJttW/XRex20P6io@public.gmane.org>
2016-05-03 19:29   ` [PATCH 1/3] MAINTAINERS: Remove asterisk from EFI directory names Matt Fleming
2016-05-03 19:29     ` Matt Fleming

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.