I think 4mb would be a better value.

On 21-Mar-08, at 2:44 PM, Marcus Boerger wrote:
Hello Ilia,

Friday, March 21, 2008, 7:28:36 PM, you wrote:

The reason a 2 meg limit was imposed originally was partially due to
memory_limit value set in PHP, which at the time was 8 megs. By
bumping it to a much higher value, you are quite easily exceeding the
memory limit. I am not opposed to increasing the value, but 8megs
seems a bit high...

Well if it tracks down to mmap() then it doesn't go to emalloc at all.
But well yeah, maybe we want a lower value. The pear installer right now
uses 3.5 MB, should we set it to 4 MB?

marcus


On 21-Mar-08, at 2:23 PM, Marcus Boerger wrote:
Hello Ilia,

mmapping of larger phars, like the pear installer :-)

marcus

Friday, March 21, 2008, 6:39:10 PM, you wrote:

Why the increase?


On 21-Mar-08, at 1:13 PM, Marcus Boerger wrote:
helly         Fri Mar 21 17:13:08 2008 UTC

Modified files:              (Branch: PHP_5_3)
 /php-src/main/streams      mmap.c
Log:
- MFH Increase max mmap size to 8 MB

Ilia Alshanetsky








Best regards,
Marcus


Ilia Alshanetsky








Best regards,
Marcus


Ilia Alshanetsky





--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to