[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20070123163259.0d1d5ffc.akpm@osdl.org>
Date: Tue, 23 Jan 2007 16:32:59 -0800
From: Andrew Morton <akpm@...l.org>
To: dipankar@...ibm.com
Cc: Ingo Molnar <mingo@...e.hu>,
Paul E McKenney <paulmck@...ux.vnet.ibm.com>,
linux-kernel@...r.kernel.org
Subject: Re: [mm PATCH 4/6] RCU: preemptible RCU
On Tue, 16 Jan 2007 00:58:58 +0530
Dipankar Sarma <dipankar@...ibm.com> wrote:
> This patch implements a new version of RCU which allows its read-side
> critical sections to be preempted.
Why is it selectable if CONFIG_PREEMPT=n?
> It uses a set of counter pairs
> to keep track of the read-side critical sections and flips them
> when all tasks exit read-side critical section. The details
> of this implementation can be found in this paper -
>
> http://www.rdrop.com/users/paulmck/RCU/OLSrtRCU.2006.08.11a.pdf
>
> This patch was developed as a part of the -rt kernel
> development and meant to provide better latencies when
> read-side critical sections of RCU don't disable preemption.
Does it succeed in that attempt? Thus far you've given no reason for
merging this code..
This is a pile of tricky new core kernel code for us to test, maintain,
understand, debug, etc. It needs to provide a substantial benefit. Does
it?
> As a consequence of keeping track of RCU readers, the readers
> have a slight overhead (optimizations in the paper).
> This implementation co-exists with the "classic" RCU
> implementations and can be switched to at compiler.
That's yet another question we need to ask people when their kernel dies,
and yet another deviation between the kernels which we all test, causing
more dilution of testing efforts. It would be much better if we could
remove classic RCU. You say this would incur extra cost, but the magnitude
of that cost is not clear. Please help us make that decision.
-
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/
Powered by blists - more mailing lists