Everything

php

RC3

php.internals

Andi Gutmans

22 years ago

Anything else that needs to go into RC3?

Andi

Wolfgang Drews

22 years ago

*This message was transferred with a trial version of CommuniGate(tm) Pro*
Hi Andi,

> Anything else that needs to go into RC3?

maybe not so important for *nixer, but for Windows: I have a
really annoying Memory Leak here on my Dev-Station with WinXP
and the latest snaps-built (from tomorrow). Every call of a php-
script increases the memory-usage of apache about 3-4 MB, so that
in consequence the apache takes up about 700MB memory after some
reloads of a php-script. Killing this apache-process is the only way to
free memory again.
This behavior appears since some earlier CVS-Builts (Don't know the
exact one).

Sorry, that i have no more details to give, but if you tell me what
to do, i would be happy to support you with finding a solution.

cheers

-Wolfgang

--
PHP-Knotenpunkt Dynamic Web Pages: http://www.dynamicwebpages.de/
Professionelle Lösungen für dynamisches Webpublishing: http://php-buch.de/
Thanks Wolfgang: http://www.amazon.de/exec/obidos/wishlist/C1MRGJEWB3SB/

Andi Gutmans

22 years ago

This is indeed very strange. I made some fixes to the memory manager a few
days ago. Have you tried a version since then?

Andi

At 02:50 PM 6/1/2004 +0200, Wolfgang Drews wrote:

Wolfgang Drews

22 years ago

*This message was transferred with a trial version of CommuniGate(tm) Pro*
Hi Andi,

> This is indeed very strange. I made some fixes to the memory
> manager a few
> days ago. Have you tried a version since then?

before writing to this list, i've download the latest CVS-Snap
Win-Built from snaps.php.net. Problem still existed, so i
decided to tell you (Build Date  Jun 1 2004 12:13:54).

cheers

-Wolfgang

--
PHP-Knotenpunkt Dynamic Web Pages: http://www.dynamicwebpages.de/
Professionelle Lösungen für dynamisches Webpublishing: http://php-buch.de/
Thanks Wolfgang: http://www.amazon.de/exec/obidos/wishlist/C1MRGJEWB3SB/

Sterling Hughes

22 years ago

Wolfgang Drews wrote:

> *This message was transferred with a trial version of CommuniGate(tm) Pro*
> Hi Andi,
>
>
>>This is indeed very strange. I made some fixes to the memory
>>manager a few
>>days ago. Have you tried a version since then?
>
>
> before writing to this list, i've download the latest CVS-Snap
> Win-Built from snaps.php.net. Problem still existed, so i
> decided to tell you (Build Date  Jun 1 2004 12:13:54).
>

can we drop this "ts memory manager," its been causing leaks for quite a
while and, from my understanding, only speeds up the threaded case.

-sterling

Andi Gutmans

22 years ago

On Windows the ZEND_MM isn't being used (we use Windows specific memory
pools), so it's really not relevant. With my tests ZEND_MM is faster than
malloc() even on non-threaded servers.
In any case, the important thing right now is to try and track down the
Windows problem which is not related. I'm not quite sure when it was
introduced. My latest fixes weren't supposed to touch that code.
In any case, if ZEND_MM ends up causing problems I don't mind leaving it
out but right now I prefer leaving it in because of its advantages.

Andi

At 09:26 AM 6/1/2004 -0700, Sterling Hughes wrote:

Wolfgang Drews

22 years ago

*This message was transferred with a trial version of CommuniGate(tm) Pro*
Hi Andi,

> In any case, the important thing right now is to try and
> track down the
> Windows problem which is not related. I'm not quite sure when it was
> introduced. My latest fixes weren't supposed to touch that code.

were you able to reproduce the memleak? Or need any more infos? I
just looked up my php5-download-dir and for sure the leak was there
with this built: May 21 2004 20:14:07

let me know, if i can help

cheers

-Wolfgang

--
PHP-Knotenpunkt Dynamic Web Pages: http://www.dynamicwebpages.de/
Professionelle Lösungen für dynamisches Webpublishing: http://php-buch.de/
Thanks Wolfgang: http://www.amazon.de/exec/obidos/wishlist/C1MRGJEWB3SB/

Andi Gutmans

22 years ago

I made some changes about 4 days ago. Can you try a build from the past day
or two?

Andi

At 06:59 PM 6/1/2004 +0200, Wolfgang Drews wrote:

Wolfgang Drews

22 years ago

*This message was transferred with a trial version of CommuniGate(tm) Pro*
Hi Andi,

> I made some changes about 4 days ago. Can you try a build
> from the past day
> or two?

sorry, i have no snapshot from this date. As i said before,
my last snapshot is from 21.05.2004 and there the problem
existed already. so the bug must have been intoduced earlier
i guess.

-Wolfgang

--
PHP-Knotenpunkt Dynamic Web Pages: http://www.dynamicwebpages.de/
Professionelle Lösungen für dynamisches Webpublishing: http://php-buch.de/
Thanks Wolfgang: http://www.amazon.de/exec/obidos/wishlist/C1MRGJEWB3SB/

Andi Gutmans

22 years ago

Can you get a June 1 snapshot from http://snaps.php.net/ and let me know?
Thanks,

Andi

At 07:07 PM 6/1/2004 +0200, Wolfgang Drews wrote:

Wolfgang Drews

22 years ago

*This message was transferred with a trial version of CommuniGate(tm) Pro*

> Can you get a June 1 snapshot from http://snaps.php.net/ and
> let me know?
> Thanks,

np. but as i said before, i've got the latest already, BEFORE i startet
this discussion :). Nevertheless i once more took the VERY latest
(php5-win32-200406011430) but memleak is still there.

cheers

-Wolfgang

--
PHP-Knotenpunkt Dynamic Web Pages: http://www.dynamicwebpages.de/
Professionelle Lösungen für dynamisches Webpublishing: http://php-buch.de/
Thanks Wolfgang: http://www.amazon.de/exec/obidos/wishlist/C1MRGJEWB3SB/

Andi Gutmans

22 years ago

Seems to be some problem with using ZEND_MM in MT servers. I checked on
Apache and this behavior doesn't exist.
In any case, on windows we have and use a native Win32 per-thread pool so
there's no reason to use ZEND_MM.
I commited a patch
Andi

At 07:43 PM 6/1/2004 +0200, Wolfgang Drews wrote:

Wolfgang Drews

22 years ago

*This message was transferred with a trial version of CommuniGate(tm) Pro*

> I commited a patch
> Andi

great, I'll give you feedback when the next snapshot is ready.

cheers

-Wolfgang

--
PHP-Knotenpunkt Dynamic Web Pages: http://www.dynamicwebpages.de/
Professionelle Lösungen für dynamisches Webpublishing: http://php-buch.de/
Thanks Wolfgang: http://www.amazon.de/exec/obidos/wishlist/C1MRGJEWB3SB/

Wolfgang Drews

22 years ago

*This message was transferred with a trial version of CommuniGate(tm) Pro*
btw., you may have a look at snaps.php.net:

Next STABLE Win32 snapshot in: 9 minute(s)
Next Latest CVS Win32 snapshot in: please consult /dev/urandom

cheers

-Wolfgang

Wolfgang Drews

22 years ago

*This message was transferred with a trial version of CommuniGate(tm) Pro*
Hi Andi,

> Seems to be some problem with using ZEND_MM in MT servers. I
> checked on
> Apache and this behavior doesn't exist.
> In any case, on windows we have and use a native Win32
> per-thread pool so
> there's no reason to use ZEND_MM.
> I commited a patch

yes, this patch works for me with the latest snapshot from
06:30 today. Thanks!

I think "Bug #27758 High memory usage/leak" can now be closed.
http://bugs.php.net/bug.php?id=27758

cheers

-Wolfgang

--
PHP-Knotenpunkt Dynamic Web Pages: http://www.dynamicwebpages.de/
Professionelle Lösungen für dynamisches Webpublishing: http://php-buch.de/
Thanks Wolfgang: http://www.amazon.de/exec/obidos/wishlist/C1MRGJEWB3SB/