linux-kernel - Re: [PATCH] locktorture: Fix Oops when reader/writer count is 0

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <20171107221555.GX3624@linux.vnet.ibm.com>
Date:   Tue, 7 Nov 2017 14:15:55 -0800
From:   "Paul E. McKenney" <paulmck@...ux.vnet.ibm.com>
To:     Peter Zijlstra <peterz@...radead.org>
Cc:     Jeremy Linton <jeremy.linton@....com>,
        linux-kernel@...r.kernel.org, mingo@...hat.com, dave@...olabs.net,
        josh@...htriplett.org
Subject: Re: [PATCH] locktorture: Fix Oops when reader/writer count is 0

On Tue, Nov 07, 2017 at 10:07:48PM +0100, Peter Zijlstra wrote:
> On Tue, Nov 07, 2017 at 02:01:58PM -0600, Jeremy Linton wrote:
> > Hi,
> > 
> > On 10/10/2017 10:52 AM, Jeremy Linton wrote:
> > >If nwriters_stress=0 is passed to the lock torture test
> > >it will panic in:
> > 
> > Ping?
> > 
> > Has anyone had a chance to look at this?
> 
> Helps if you Cc the people actually working on this stuff of course...

Thank you for the forward, Peter, I have queued Jeremy's patch for
testing and review.

But Jeremy's list of email addresses is what you would expect from
looking at MAINTAINERS, so how about the following patch?

							Thanx, Paul

------------------------------------------------------------------------

commit 58322063498c8f5a3cc88f95bee237a0ce81f70a
Author: Paul E. McKenney <paulmck@...ux.vnet.ibm.com>
Date:   Tue Nov 7 14:10:03 2017 -0800

    torture: Place all torture-test modules in one MAINTAINERS group
    
    There is some confusion about where patches to kernel/torture.c
    and kernel/locking/locktorture.c should be sent.  This commit
    therefore updates MAINTAINERS appropriately.
    
    Reported-by: Peter Zijlstra <peterz@...radead.org>
    Signed-off-by: Paul E. McKenney <paulmck@...ux.vnet.ibm.com>

diff --git a/MAINTAINERS b/MAINTAINERS
index 2d3d750b19c0..eab868adedc6 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -8091,6 +8091,7 @@ F:	arch/*/include/asm/rwsem.h
 F:	include/linux/seqlock.h
 F:	lib/locking*.[ch]
 F:	kernel/locking/
+X:	kernel/locking/locktorture.c
 
 LOGICAL DISK MANAGER SUPPORT (LDM, Windows 2000/XP/Vista Dynamic Disks)
 M:	"Richard Russon (FlatCap)" <ldm@...tcap.org>
@@ -11318,15 +11319,6 @@ L:	linux-wireless@...r.kernel.org
 S:	Orphan
 F:	drivers/net/wireless/ray*
 
-RCUTORTURE MODULE
-M:	Josh Triplett <josh@...htriplett.org>
-M:	"Paul E. McKenney" <paulmck@...ux.vnet.ibm.com>
-L:	linux-kernel@...r.kernel.org
-S:	Supported
-T:	git git://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu.git
-F:	Documentation/RCU/torture.txt
-F:	kernel/rcu/rcutorture.c
-
 RCUTORTURE TEST FRAMEWORK
 M:	"Paul E. McKenney" <paulmck@...ux.vnet.ibm.com>
 M:	Josh Triplett <josh@...htriplett.org>
@@ -13558,6 +13550,18 @@ L:	platform-driver-x86@...r.kernel.org
 S:	Maintained
 F:	drivers/platform/x86/topstar-laptop.c
 
+TORTURE-TEST MODULES
+M:	Davidlohr Bueso <dave@...olabs.net>
+M:	"Paul E. McKenney" <paulmck@...ux.vnet.ibm.com>
+M:	Josh Triplett <josh@...htriplett.org>
+L:	linux-kernel@...r.kernel.org
+S:	Supported
+T:	git git://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu.git
+F:	Documentation/RCU/torture.txt
+F:	kernel/torture.c
+F:	kernel/rcu/rcutorture.c
+F:	kernel/locking/locktorture.c
+
 TOSHIBA ACPI EXTRAS DRIVER
 M:	Azael Avalos <coproscefalo@...il.com>
 L:	platform-driver-x86@...r.kernel.org