[PATCH] Make loadparm more common

[Andrew Bartlett]

On Fri, 2014-03-28 at 09:21 +1300, Andrew Bartlett wrote:
> On Thu, 2014-03-27 at 16:04 +1300, Garming Sam wrote:
> > On 21/03/14 13:00, Andrew Bartlett wrote:
> > > On Thu, 2014-03-20 at 13:12 +1300, Andrew Bartlett wrote:
> > >> On Thu, 2014-03-13 at 16:08 +1300, Andrew Bartlett wrote:
> > >>> Garming,
> > >>>
> > >>> I'm really impressed by the patches you just showed me at
> > >>> http://git.catalyst.net.nz/gitweb?p=samba.git;a=shortlog;h=refs/heads/loadparm-talloc-polish-second
> > >>> Aside from these small comments, easily sorted out, can I get comments,
> > >>> thoughts and perhaps a second team reviewer for this?
> > >> These issues have been fixed up, and I've uploaded it to gerrit.  The
> > >> series starts at https://gerrit.samba.org/#/c/97/ and finishes at
> > >> https://gerrit.samba.org/203
> > >>
> > >> Most of the earlier set of patches have been reviewed, but these still
> > >> need a second reviewer:
> > >> https://gerrit.samba.org/#/c/156/
> > >> https://gerrit.samba.org/#/c/157/
> > >> https://gerrit.samba.org/#/c/119/
> > >> https://gerrit.samba.org/#/c/158/
> > >> https://gerrit.samba.org/#/c/159/
> > >> https://gerrit.samba.org/#/c/160/
> > >>
> > >> Also, the new changes start at https://gerrit.samba.org/166
> > > It looks like we still have some issues with the change across to
> > > talloc, which was always going to be the most delicate part of this
> > > patch series.
> > >
> > > A bisect shows that the s3dc environment fails to start with
> > > commit 9ea68b9f954dac6b222bf6d63cf7329d40659310
> > > Author: Garming Sam <garming at catalyst.net.nz>
> > > Date:   Fri Feb 14 12:46:41 2014 +1300
> > >
> > >      s3:loadparm: change memory allocations to use talloc
> > >      
> > >      Change-Id: I9503eee037eadd07c4dd8f5481f65b569dc2dac8
> > >      Signed-off-by: Garming Sam <garming at catalyst.net.nz>
> > >      Reviewed-by: Andrew Bartlett <abartlet at samba.org>
> > >
> > > Sorry,
> > >
> > > Andrew Bartlett
> > >
> > 
> > The issues regarding the s3dc have been fixed. It stems from an issue 
> > where the talloc context for the globals exists independently from the 
> > service pointers.
> > 
> > The changes are in:
> > git://git.catalyst.net.nz/samba.git loadparm-talloc-polish-secondv2
> > 
> > http://git.catalyst.net.nz/gitweb?p=samba.git;a=shortlog;h=refs/heads/loadparm-talloc-polish-secondv2
> > 
> > Apologies for being unable to put this in gerrit, it seems to be having 
> > some issues processing the changes.
> > 
> > Please review and push.
> 
> Once we get the first 12 or so patches into master, I think I'll be able
> to push the rest to gerrit, but for now we will have to review these the
> old-fashioned way.

Even more patches are at
http://git.catalyst.net.nz/gitweb?p=samba.git;a=shortlog;h=refs/heads/loadparm-talloc-to-review3

Please review!  (I do realise that reviewing 98 patches will take some
time, but if we can make a start on the first 14 that would be most
helpful).

Thanks,

Andrew Bartlett

-- 
Andrew Bartlett
http://samba.org/~abartlet/
Authentication Developer, Samba Team  http://samba.org
Samba Developer, Catalyst IT          http://catalyst.net.nz/services/samba