Re: [PATCH] v4 RCU: the bloatwatch edition

From: Andrew Morton
Date: Wed May 06 2009 - 14:32:12 EST


On Wed, 6 May 2009 14:09:09 +0200
Ingo Molnar <mingo@xxxxxxx> wrote:

>
> * David Howells <dhowells@xxxxxxxxxx> wrote:
>
> > Paul E. McKenney <paulmck@xxxxxxxxxxxxxxxxxx> wrote:
> >
> > > This patch is a version of RCU designed for (!SMP && EMBEDDED)
> > > provided as a proof of concept of a small-footprint RCU implementation.
> > > In particular, the implementation of synchronize_rcu() is extremely
> > > lightweight and high performance. It passes rcutorture testing in each
> > > of the four relevant configurations (combinations of NO_HZ and PREEMPT)
> > > on x86. This saves about 900 bytes compared to Classic RCU, and a
> > > couple kilobytes compared to Hierarchical RCU (updated to 2.6.29):
> > > ...
> > > Signed-off-by: Paul E. McKenney <paulmck@xxxxxxxxxxxxxxxxxx>
> >
> > Acked-by: David Howells <dhowells@xxxxxxxxxx>
>
> i'm wondering what Andrew thinks - he had objections, right?
>

More like "concerns". It's unobvious to me that the modest .text
savings justify the costs of an additional RCU implementation. Where
those costs include

- additional maintenance work and

- the reduced code reliability which comes from fragmenting the
tester base. This will mostly affect users of the less popular RCU
implementations.


But hey, maybe I'm wrong. And maybe I'm right, but we'll merge it anyway ;)
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/