linux-kernel - Re: [PATCH 0/2] Markers Implementation for RCU Tracing

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: <20071130090708.GA8208@in.ibm.com>
Date:	Fri, 30 Nov 2007 14:37:08 +0530
From:	"K. Prasad" <prasad@...ux.vnet.ibm.com>
To:	"Paul E. McKenney" <paulmck@...ux.vnet.ibm.com>
Cc:	"K. Prasad" <prasad@...ux.vnet.ibm.com>,
	linux-kernel@...r.kernel.org, dipankar@...ibm.com, ego@...ibm.com,
	mathieu.desnoyers@...ymtl.ca, mingo@...e.hu
Subject: Re: [PATCH 0/2] Markers Implementation for RCU Tracing

On Thu, Nov 29, 2007 at 05:36:05PM -0800, Paul E. McKenney wrote:
> On Fri, Nov 30, 2007 at 12:11:28AM +0530, K. Prasad wrote:
> > Hi,
> > 	Please review the ensuing set of patches which convert the
> > existing RCU tracing mechanism for Preempt RCU and RCU Boost into
> > markers.
> > 
> > These patches are based upon the 2.6.24-rc2-rt1 kernel tree.
> > 
> > Along with marker transition, the RCU Tracing infrastructure has also
> > been modularised to be built as a kernel module, thereby enabling
> > runtime changes to the RCU Tracing infrastructure.
> > 
> > Patch [1/2] - Patch that converts the Preempt RCU tracing in
> > rcupreempt.c into markers.
> > 
> > Patch [1/2] - Patch that converts the Preempt RCU Boost tracing in
> > rcupreempt-boost.c into markers.
> 
> Looks good to me, though I do not pretend to understand the markers
> implementation.  I presume that the markers implementation forces the
> varargs usage -- though the markers do seem quite a bit nicer in allowing
> the formatting to be specified more naturally.
> 

Paul,
	Thanks for the quick review.
Yes, varargs are required for marker handlers even with fixed set of
arguments.

Ingo,
	Can you please include these patches in your rt patchset?

Thanks,
K.Prasad

-
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/