RE: Memory management requirements buckets/brigades, new pools

[Ian Holsman <Ia...@cnet.com>]

Hi Sander.
is it possible to post patches from the most recent CVS version.
that would make it much easier for people to apply them

Thanks
..Ian
On Sun, 2001-08-26 at 08:29, Sander Striker wrote:
> [this time including the attachment...]
> 
> > Hi,
> > 
> > It seems that the memory management requirements for
> > buckets is that they have to be able to control their
> > own lifetime.  In other words, they need to be allocated
> > and freed on an individual basis.
> > It seems that their lifetime is bound by the lifetime
> > of the connection.
> > 
> > The above let me believe that buckets need a free
> > function to complement apr_palloc.  Hence the
> > attached patch that introduces apr_pfree *).  I know
> > this patch introduces some extra overhead, although
> > not much, which could be unacceptable.  OTOH would
> > this make it possible to use one memory management
> > scheme throughout apache...
> > 
> > Maybe something to consider, maybe not.  I don't
> > even know if these are the criteria or not ;)
> > 
> > 
> > Sander
> > 
> > *) patch is against the recently posted possible
> >    replacement code for pools.
> 
-- 
Ian Holsman
Performance Measurement & Analysis
CNET Networks    -    415 364-8608

RE: Memory management requirements buckets/brigades, new pools

[Ian Holsman <Ia...@cnet.com>]

On Sun, 2001-08-26 at 09:02, Sander Striker wrote:
> > Hi Sander.
> > is it possible to post patches from the most recent CVS version.
> > that would make it much easier for people to apply them
> 
> Ok, I agree with you on applyability, but for readability that
> is quite different.  Want me to post some patches against recent
> CVS?
[removed httpd]
if you could.
that would make it easier for me to test tomorrow.
> 
> > Thanks
> > ...Ian
> 
> Sander
-- 
Ian Holsman
Performance Measurement & Analysis
CNET Networks    -    415 364-8608

RE: Memory management requirements buckets/brigades, new pools

[Sander Striker <st...@apache.org>]

> Hi Sander.
> is it possible to post patches from the most recent CVS version.
> that would make it much easier for people to apply them

Ok, I agree with you on applyability, but for readability that
is quite different.  Want me to post some patches against recent
CVS?

> Thanks
> ...Ian

Sander

RE: Memory management requirements buckets/brigades, new pools

[Sander Striker <st...@apache.org>]

> Hi Sander.
> is it possible to post patches from the most recent CVS version.
> that would make it much easier for people to apply them

Ok, I agree with you on applyability, but for readability that
is quite different.  Want me to post some patches against recent
CVS?

> Thanks
> ...Ian

Sander