Re: cvs: php-src(PHP_5_3) /main/streams mmap.c
| From: | Ilia Alshanetsky | Date: | Fri, 21 Mar 2008 19:09:22 +0000 |
| Subject: | Re: cvs: php-src(PHP_5_3) /main/streams mmap.c | ||
| References: | 1 2 3 4 5 | Groups: | php.cvs |
| Request: | Send a blank email to php-cvs+get-49332@lists.php.net to get a copy of this message | ||
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:Ilia AlshanetskyThe 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? marcusOn 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 UTCModified files: (Branch: PHP_5_3) /php-src/main/streams mmap.cLog: - MFH Increase max mmap size to 8 MBIlia AlshanetskyBest regards, MarcusIlia AlshanetskyBest regards, Marcus