linux-kernel - Re: [PATCH tip/core/rcu 0/12] TINY

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] [day] [month] [year] [list]

Message-ID: <20130413193854.GZ29861@linux.vnet.ibm.com>
Date:	Sat, 13 Apr 2013 12:38:54 -0700
From:	"Paul E. McKenney" <paulmck@...ux.vnet.ibm.com>
To:	Josh Triplett <josh@...htriplett.org>
Cc:	linux-kernel@...r.kernel.org, mingo@...e.hu, laijs@...fujitsu.com,
	dipankar@...ibm.com, akpm@...ux-foundation.org,
	mathieu.desnoyers@...icios.com, niv@...ibm.com, tglx@...utronix.de,
	peterz@...radead.org, rostedt@...dmis.org, Valdis.Kletnieks@...edu,
	dhowells@...hat.com, edumazet@...gle.com, darren@...art.com,
	fweisbec@...il.com, sbw@....edu
Subject: Re: [PATCH tip/core/rcu 0/12] TINY_RCU changes for 3.11

On Sat, Apr 13, 2013 at 11:30:49AM -0700, Josh Triplett wrote:
> On Fri, Apr 12, 2013 at 04:48:10PM -0700, Paul E. McKenney wrote:
> > This series removes TINY_PREEMPT_RCU, as promised/threatened at
> > http://lwn.net/Articles/541037/ and https://lkml.org/lkml/2012/11/12/545.
> > 
> > 1.	Remove TINY_PREEMPT_RCU.  This is a straight syntactic removal,
> > 	with no attempt at cleanup.  The remaining patches do the cleanup.
> > 
> > 2.	Inline the now-empty show_tiny_preempt_stats() function.
> > 
> > 3.	Inline the now-empty rcu_preempt_check_callbacks() function.
> > 
> > 4.	Inline the now-empty rcu_preempt_remove_callbacks() function.
> > 
> > 5.	Inline the now-empty rcu_preempt_process_callbacks() function.
> > 
> > 6.	Because TINY_RCU no longer has kthreads, remove the code that
> > 	used to abstract away kthread vs. softirq invocation.
> > 
> > 7.	Inline the now-empty check_cpu_stall_preempt() function.
> > 
> > 8.	Remove CONFIG_TINY_RCU ifdefs from include/linux/rcutiny.h
> > 
> > 9.	Inline the now-empty rcu_preempt_note_context_switch() function.
> > 
> > 10.	Move code to allow consolidating ifdefs in kernel/rcutiny_plugin.h.
> > 
> > 11.	Remove TINY_PREEMPT_RCU's tracing formats from documentation.
> > 
> > 12.	Shrink TINY_RCU a bit by moving exit_rcu() to TREE_RCU, leaving
> > 	TINY_RCU with a static inline empty function.
> 
> For 2-7 and 9-12:
> Reviewed-by: Josh Triplett <josh@...htriplett.org>
> 
> I responded to patch 8 with a note about moving part of it to patch 1;
> with that changed,
> Reviewed-by: Josh Triplett <josh@...htriplett.org>
> for those two as well.

Thank you for the review, and good point on merging patch 8 into patch 1,
will do!

							Thanx, Paul

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/