Re: Bad MAINTAINERS pattern in section 'LOCKING PRIMITIVES'

2018-10-01 Thread Will Deacon 
On Fri, Sep 28, 2018 at 02:54:36PM -0700, Joe Perches wrote:
> Please fix this defect appropriately.

Patch below, which was probably less effort to write than your report ;)

Will

--->8

>From 867b47473ad490c21591378a7041fe7c06b71b59 Mon Sep 17 00:00:00 2001
From: Will Deacon 
Date: Mon, 1 Oct 2018 14:47:58 +0100
Subject: [PATCH] MAINTAINERS: Remove dead path from LOCKING PRIMITIVES entry

Since 890658b7ab48 ("locking/mutex: Kill arch specific code"), there
are no mutex header files under arch/, so we can remove the redundant
entry from MAINTAINERS.

Reported-by: Joe Perches 
Signed-off-by: Will Deacon 
---
 MAINTAINERS | 1 -
 1 file changed, 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index d870cb57c887..276c32df137a 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -8582,7 +8582,6 @@ F:include/linux/spinlock*.h
 F: arch/*/include/asm/spinlock*.h
 F: include/linux/rwlock*.h
 F: include/linux/mutex*.h
-F: arch/*/include/asm/mutex*.h
 F: include/linux/rwsem*.h
 F: arch/*/include/asm/rwsem.h
 F: include/linux/seqlock.h
-- 
2.1.4

Re: Bad MAINTAINERS pattern in section 'LOCKING PRIMITIVES'

2018-10-01 Thread Will Deacon 
On Fri, Sep 28, 2018 at 02:54:36PM -0700, Joe Perches wrote:
> Please fix this defect appropriately.

Patch below, which was probably less effort to write than your report ;)

Will

--->8

>From 867b47473ad490c21591378a7041fe7c06b71b59 Mon Sep 17 00:00:00 2001
From: Will Deacon 
Date: Mon, 1 Oct 2018 14:47:58 +0100
Subject: [PATCH] MAINTAINERS: Remove dead path from LOCKING PRIMITIVES entry

Since 890658b7ab48 ("locking/mutex: Kill arch specific code"), there
are no mutex header files under arch/, so we can remove the redundant
entry from MAINTAINERS.

Reported-by: Joe Perches 
Signed-off-by: Will Deacon 
---
 MAINTAINERS | 1 -
 1 file changed, 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index d870cb57c887..276c32df137a 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -8582,7 +8582,6 @@ F:include/linux/spinlock*.h
 F: arch/*/include/asm/spinlock*.h
 F: include/linux/rwlock*.h
 F: include/linux/mutex*.h
-F: arch/*/include/asm/mutex*.h
 F: include/linux/rwsem*.h
 F: arch/*/include/asm/rwsem.h
 F: include/linux/seqlock.h
-- 
2.1.4

Bad MAINTAINERS pattern in section 'LOCKING PRIMITIVES'

2018-09-28 Thread Joe Perches 
Please fix this defect appropriately.

linux-next MAINTAINERS section:

8566LOCKING PRIMITIVES
8567M:  Peter Zijlstra 
8568M:  Ingo Molnar 
8569M:  Will Deacon 
8570L:  linux-kernel@vger.kernel.org
8571T:  git 
git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git locking/core
8572S:  Maintained
8573F:  Documentation/locking/
8574F:  include/linux/lockdep.h
8575F:  include/linux/spinlock*.h
8576F:  arch/*/include/asm/spinlock*.h
8577F:  include/linux/rwlock*.h
8578F:  include/linux/mutex*.h
--> 8579F:  arch/*/include/asm/mutex*.h
8580F:  include/linux/rwsem*.h
8581F:  arch/*/include/asm/rwsem.h
8582F:  include/linux/seqlock.h
8583F:  lib/locking*.[ch]
8584F:  kernel/locking/
8585X:  kernel/locking/locktorture.c

Commit that introduced this:

commit d4c3be70ca0e7a1ae308bedd3462900c61e97b11
 Author: Ingo Molnar 
 Date:   Thu Jun 9 12:20:25 2016 +0200
 
 MAINTAINERS: Update locking tree description and file patterns
 
 Update the file patterns, the Git tree URI and also widen the
 scope from 'LOCKDEP and LOCKSTAT' to 'LOCKING PRIMITIVES'.
 
 Signed-off-by: Ingo Molnar 
 Cc: Peter Zijlstra 
 Cc: Linus Torvalds 
 Cc: Andrew Morton 
 Cc: Thomas Gleixner 
 Cc: Paul E. McKenney 
 Cc: linux-kernel@vger.kernel.org
 
  MAINTAINERS | 16 
  1 file changed, 12 insertions(+), 4 deletions(-)

Last commit with arch/*/include/asm/mutex*.h

commit 890658b7ab48d1362a0362df842cecc73c83146f
Author: Peter Zijlstra 
Date:   Tue Aug 23 13:36:04 2016 +0200

locking/mutex: Kill arch specific code

Its all generic atomic_long_t stuff now.

Tested-by: Jason Low 
Signed-off-by: Peter Zijlstra (Intel) 
Cc: Andrew Morton 
Cc: Linus Torvalds 
Cc: Paul E. McKenney 
Cc: Peter Zijlstra 
Cc: Thomas Gleixner 
Cc: linux-a...@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Ingo Molnar 

 arch/alpha/include/asm/mutex.h  |   9 ---
 arch/arc/include/asm/mutex.h|  18 -
 arch/arm/include/asm/mutex.h|  21 --
 arch/arm64/include/asm/Kbuild   |   1 -
 arch/avr32/include/asm/mutex.h  |   9 ---
 arch/blackfin/include/asm/Kbuild|   1 -
 arch/c6x/include/asm/mutex.h|   6 --
 arch/cris/include/asm/mutex.h   |   9 ---
 arch/frv/include/asm/mutex.h|   9 ---
 arch/h8300/include/asm/mutex.h  |   9 ---
 arch/hexagon/include/asm/mutex.h|   8 ---
 arch/ia64/include/asm/mutex.h   |  90 
 arch/m32r/include/asm/mutex.h   |   9 ---
 arch/m68k/include/asm/Kbuild|   1 -
 arch/metag/include/asm/Kbuild   |   1 -
 arch/microblaze/include/asm/mutex.h |   1 -
 arch/mips/include/asm/Kbuild|   1 -
 arch/mn10300/include/asm/mutex.h|  16 -
 arch/nios2/include/asm/mutex.h  |   1 -
 arch/openrisc/include/asm/mutex.h   |  27 
 arch/parisc/include/asm/Kbuild  |   1 -
 arch/powerpc/include/asm/mutex.h| 132 
 arch/s390/include/asm/mutex.h   |   9 ---
 arch/score/include/asm/mutex.h  |   6 --
 arch/sh/include/asm/mutex-llsc.h| 109 -
 arch/sh/include/asm/mutex.h |  12 
 arch/sparc/include/asm/Kbuild   |   1 -
 arch/tile/include/asm/Kbuild|   1 -
 arch/um/include/asm/Kbuild  |   1 -
 arch/unicore32/include/asm/mutex.h  |  20 --
 arch/x86/include/asm/mutex.h|   5 --
 arch/x86/include/asm/mutex_32.h | 110 --
 arch/x86/include/asm/mutex_64.h | 127 --
 arch/xtensa/include/asm/mutex.h |   9 ---
 include/asm-generic/mutex-dec.h |  88 
 include/asm-generic/mutex-null.h|  19 --
 include/asm-generic/mutex-xchg.h| 120 
 include/asm-generic/mutex.h |   9 ---
 38 files changed, 1026 deletions(-)

Bad MAINTAINERS pattern in section 'LOCKING PRIMITIVES'

2018-09-28 Thread Joe Perches 
Please fix this defect appropriately.

linux-next MAINTAINERS section:

8566LOCKING PRIMITIVES
8567M:  Peter Zijlstra 
8568M:  Ingo Molnar 
8569M:  Will Deacon 
8570L:  linux-kernel@vger.kernel.org
8571T:  git 
git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git locking/core
8572S:  Maintained
8573F:  Documentation/locking/
8574F:  include/linux/lockdep.h
8575F:  include/linux/spinlock*.h
8576F:  arch/*/include/asm/spinlock*.h
8577F:  include/linux/rwlock*.h
8578F:  include/linux/mutex*.h
--> 8579F:  arch/*/include/asm/mutex*.h
8580F:  include/linux/rwsem*.h
8581F:  arch/*/include/asm/rwsem.h
8582F:  include/linux/seqlock.h
8583F:  lib/locking*.[ch]
8584F:  kernel/locking/
8585X:  kernel/locking/locktorture.c

Commit that introduced this:

commit d4c3be70ca0e7a1ae308bedd3462900c61e97b11
 Author: Ingo Molnar 
 Date:   Thu Jun 9 12:20:25 2016 +0200
 
 MAINTAINERS: Update locking tree description and file patterns
 
 Update the file patterns, the Git tree URI and also widen the
 scope from 'LOCKDEP and LOCKSTAT' to 'LOCKING PRIMITIVES'.
 
 Signed-off-by: Ingo Molnar 
 Cc: Peter Zijlstra 
 Cc: Linus Torvalds 
 Cc: Andrew Morton 
 Cc: Thomas Gleixner 
 Cc: Paul E. McKenney 
 Cc: linux-kernel@vger.kernel.org
 
  MAINTAINERS | 16 
  1 file changed, 12 insertions(+), 4 deletions(-)

Last commit with arch/*/include/asm/mutex*.h

commit 890658b7ab48d1362a0362df842cecc73c83146f
Author: Peter Zijlstra 
Date:   Tue Aug 23 13:36:04 2016 +0200

locking/mutex: Kill arch specific code

Its all generic atomic_long_t stuff now.

Tested-by: Jason Low 
Signed-off-by: Peter Zijlstra (Intel) 
Cc: Andrew Morton 
Cc: Linus Torvalds 
Cc: Paul E. McKenney 
Cc: Peter Zijlstra 
Cc: Thomas Gleixner 
Cc: linux-a...@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Ingo Molnar 

 arch/alpha/include/asm/mutex.h  |   9 ---
 arch/arc/include/asm/mutex.h|  18 -
 arch/arm/include/asm/mutex.h|  21 --
 arch/arm64/include/asm/Kbuild   |   1 -
 arch/avr32/include/asm/mutex.h  |   9 ---
 arch/blackfin/include/asm/Kbuild|   1 -
 arch/c6x/include/asm/mutex.h|   6 --
 arch/cris/include/asm/mutex.h   |   9 ---
 arch/frv/include/asm/mutex.h|   9 ---
 arch/h8300/include/asm/mutex.h  |   9 ---
 arch/hexagon/include/asm/mutex.h|   8 ---
 arch/ia64/include/asm/mutex.h   |  90 
 arch/m32r/include/asm/mutex.h   |   9 ---
 arch/m68k/include/asm/Kbuild|   1 -
 arch/metag/include/asm/Kbuild   |   1 -
 arch/microblaze/include/asm/mutex.h |   1 -
 arch/mips/include/asm/Kbuild|   1 -
 arch/mn10300/include/asm/mutex.h|  16 -
 arch/nios2/include/asm/mutex.h  |   1 -
 arch/openrisc/include/asm/mutex.h   |  27 
 arch/parisc/include/asm/Kbuild  |   1 -
 arch/powerpc/include/asm/mutex.h| 132 
 arch/s390/include/asm/mutex.h   |   9 ---
 arch/score/include/asm/mutex.h  |   6 --
 arch/sh/include/asm/mutex-llsc.h| 109 -
 arch/sh/include/asm/mutex.h |  12 
 arch/sparc/include/asm/Kbuild   |   1 -
 arch/tile/include/asm/Kbuild|   1 -
 arch/um/include/asm/Kbuild  |   1 -
 arch/unicore32/include/asm/mutex.h  |  20 --
 arch/x86/include/asm/mutex.h|   5 --
 arch/x86/include/asm/mutex_32.h | 110 --
 arch/x86/include/asm/mutex_64.h | 127 --
 arch/xtensa/include/asm/mutex.h |   9 ---
 include/asm-generic/mutex-dec.h |  88 
 include/asm-generic/mutex-null.h|  19 --
 include/asm-generic/mutex-xchg.h| 120 
 include/asm-generic/mutex.h |   9 ---
 38 files changed, 1026 deletions(-)