Review of brl scaling fix patches.

[Michael Adam]

Volker Lendecke wrote:
> On Sat, Feb 06, 2010 at 05:06:21PM +1100, tridge at samba.org wrote:
> > Hi Jeremy,
> > 
> >  > Do these really need to be "static int"'s ?
> > 
> > I had forgotten that you were trying to reduce static vars in s3. I
> > only used one as lp_parm_int() uses get_parametrics() which is O(N) in
> > the number of non-fixed parameters. That means we pay a bit extra to
> > get the param when a child dies. I doubt it would be measurable
> > though, so it's fine just to init each time of course.
> 
> Why not make it a real parameter?

Absolutely. Parametric options are fine for module-specific
purposes, but other than that, we are trying to get rid
of parametric options.

Michael

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 206 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20100207/0416138b/attachment.pgp>