mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* Bad MAINTAINERS pattern in section 'LOCKING PRIMITIVES'
@ 2018-09-28 21:54 Joe Perches
  2018-10-01 14:28 ` Will Deacon
  0 siblings, 1 reply; 3+ messages in thread
From: Joe Perches @ 2018-09-28 21:54 UTC (permalink / raw)
  To: linux-kernel
  Cc: Peter Zijlstra, Ingo Molnar, Will Deacon, Jason Low, Ingo Molnar

Please fix this defect appropriately.

linux-next MAINTAINERS section:

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

Commit that introduced this:

commit d4c3be70ca0e7a1ae308bedd3462900c61e97b11
 Author: Ingo Molnar <mingo@kernel.org>
 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 <mingo@kernel.org>
     Cc: Peter Zijlstra <peterz@infradead.org>
     Cc: Linus Torvalds <torvalds@linux-foundation.org>
     Cc: Andrew Morton <akpm@linux-foundation.org>
     Cc: Thomas Gleixner <tglx@linutronix.de>
     Cc: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
     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 <peterz@infradead.org>
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 <jason.low2@hpe.com>
    Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
    Cc: Andrew Morton <akpm@linux-foundation.org>
    Cc: Linus Torvalds <torvalds@linux-foundation.org>
    Cc: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
    Cc: Peter Zijlstra <peterz@infradead.org>
    Cc: Thomas Gleixner <tglx@linutronix.de>
    Cc: linux-arch@vger.kernel.org
    Cc: linux-kernel@vger.kernel.org
    Signed-off-by: Ingo Molnar <mingo@kernel.org>

 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(-)

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

* Re: Bad MAINTAINERS pattern in section 'LOCKING PRIMITIVES'
  2018-09-28 21:54 Bad MAINTAINERS pattern in section 'LOCKING PRIMITIVES' Joe Perches
@ 2018-10-01 14:28 ` Will Deacon
  2018-10-02 10:15   ` [tip:locking/urgent] MAINTAINERS: Remove dead path from LOCKING PRIMITIVES entry tip-bot for Will Deacon
  0 siblings, 1 reply; 3+ messages in thread
From: Will Deacon @ 2018-10-01 14:28 UTC (permalink / raw)
  To: Joe Perches
  Cc: linux-kernel, Peter Zijlstra, Ingo Molnar, Jason Low, Ingo Molnar

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 <will.deacon@arm.com>
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 <joe@perches.com>
Signed-off-by: Will Deacon <will.deacon@arm.com>
---
 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


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

* [tip:locking/urgent] MAINTAINERS: Remove dead path from LOCKING PRIMITIVES entry
  2018-10-01 14:28 ` Will Deacon
@ 2018-10-02 10:15   ` tip-bot for Will Deacon
  0 siblings, 0 replies; 3+ messages in thread
From: tip-bot for Will Deacon @ 2018-10-02 10:15 UTC (permalink / raw)
  To: linux-tip-commits
  Cc: hpa, will.deacon, mingo, jason.low2, joe, peterz, torvalds,
	linux-kernel, tglx

Commit-ID:  6d348925b306ab0cc9757a09a8cea6bf288018e4
Gitweb:     https://git.kernel.org/tip/6d348925b306ab0cc9757a09a8cea6bf288018e4
Author:     Will Deacon <will.deacon@arm.com>
AuthorDate: Mon, 1 Oct 2018 15:28:56 +0100
Committer:  Ingo Molnar <mingo@kernel.org>
CommitDate: Tue, 2 Oct 2018 10:45:57 +0200

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 <joe@perches.com>
Signed-off-by: Will Deacon <will.deacon@arm.com>
Cc: Jason Low <jason.low2@hpe.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Link: http://lkml.kernel.org/r/20181001142856.GC9716@arm.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
---
 MAINTAINERS | 1 -
 1 file changed, 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index a255240d1452..3bd8913b2d78 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -8598,7 +8598,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

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

end of thread, other threads:[~2018-10-02 10:16 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-09-28 21:54 Bad MAINTAINERS pattern in section 'LOCKING PRIMITIVES' Joe Perches
2018-10-01 14:28 ` Will Deacon
2018-10-02 10:15   ` [tip:locking/urgent] MAINTAINERS: Remove dead path from LOCKING PRIMITIVES entry tip-bot for Will Deacon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

Powered by JetHome