diff --git a/Zend/zend_alloc.c b/Zend/zend_alloc.c index 4372409e28c..7f5f2dd67ca 100644 --- a/Zend/zend_alloc.c +++ b/Zend/zend_alloc.c @@ -312,10 +312,12 @@ struct _zend_mm_heap { size_t block_size; zend_mm_segment *segments_list; zend_mm_storage *storage; - size_t size; + size_t real_size; #if MEMORY_LIMIT - size_t peak; + size_t real_peak; size_t limit; + size_t size; + size_t peak; #endif #if ZEND_USE_MALLOC_MM int use_zend_alloc; @@ -514,7 +516,7 @@ static void zend_mm_del_segment(zend_mm_heap *heap, zend_mm_segment *segment) p = p->next_segment; } } - heap->size -= segment->size; + heap->real_size -= segment->size; ZEND_MM_STORAGE_FREE(segment); } @@ -595,10 +597,12 @@ ZEND_API zend_mm_heap *zend_mm_startup_ex(const zend_mm_mem_handlers *handlers, heap->use_zend_alloc = 1; #endif - heap->size = 0; + heap->real_size = 0; #if MEMORY_LIMIT - heap->peak = 0; + heap->real_peak = 0; heap->limit = 1<<30; + heap->size = 0; + heap->peak = 0; #endif heap->overflow = 0; @@ -996,8 +1000,10 @@ ZEND_API void zend_mm_shutdown(zend_mm_heap *heap, int full_shutdown, int silent } else { heap->segments_list = NULL; zend_mm_init(heap); - heap->size = 0; + heap->real_size = 0; #if MEMORY_LIMIT + heap->real_peak = 0; + heap->size = 0; heap->peak = 0; #endif heap->overflow = 0; @@ -1175,7 +1181,7 @@ zend_mm_finished_searching_for_block: #if MEMORY_LIMIT - if (heap->size + segment_size > heap->limit) { + if (heap->real_size + segment_size > heap->limit) { /* Memory limit overflow */ #if ZEND_DEBUG zend_mm_safe_error(heap, "Allowed memory size of %d bytes exhausted at %s:%d (tried to allocate %d bytes)", heap->limit, __zend_filename, __zend_lineno, size); @@ -1196,17 +1202,17 @@ zend_mm_finished_searching_for_block: #endif HANDLE_UNBLOCK_INTERRUPTIONS(); #if ZEND_DEBUG - zend_mm_safe_error(heap, "Out of memory (allocated %d) at %s:%d (tried to allocate %d bytes)", heap->size, __zend_filename, __zend_lineno, size); + zend_mm_safe_error(heap, "Out of memory (allocated %d) at %s:%d (tried to allocate %d bytes)", heap->real_size, __zend_filename, __zend_lineno, size); #else - zend_mm_safe_error(heap, "Out of memory (allocated %d) (tried to allocate %d bytes)", heap->size, size); + zend_mm_safe_error(heap, "Out of memory (allocated %d) (tried to allocate %d bytes)", heap->real_size, size); #endif return NULL; } - heap->size += segment_size; + heap->real_size += segment_size; #if MEMORY_LIMIT - if (heap->size > heap->peak) { - heap->peak = heap->size; + if (heap->real_size > heap->real_peak) { + heap->real_peak = heap->real_size; } #endif @@ -1242,6 +1248,14 @@ zend_mm_finished_searching_for_block: # endif ZEND_MM_SET_END_MAGIC(best_fit); #endif + +#if MEMORY_LIMIT + heap->size += true_size; + if (heap->peak < heap->size) { + heap->peak = heap->size; + } +#endif + HANDLE_UNBLOCK_INTERRUPTIONS(); return ZEND_MM_DATA_OF(best_fit); @@ -1278,6 +1292,11 @@ static void _zend_mm_free_int(zend_mm_heap *heap, void *p ZEND_FILE_LINE_DC ZEND #endif HANDLE_BLOCK_INTERRUPTIONS(); + +#if MEMORY_LIMIT + heap->size -= size; +#endif + if (ZEND_MM_PREV_BLOCK_IS_FREE(mm_block)) { next_block = ZEND_MM_NEXT_BLOCK(mm_block); if (ZEND_MM_IS_FREE_BLOCK(next_block)) { @@ -1349,6 +1368,14 @@ static void *_zend_mm_realloc_int(zend_mm_heap *heap, void *p, size_t size ZEND_ } mm_block = ZEND_MM_HEADER_OF(p); true_size = ZEND_MM_TRUE_SIZE(size); + +#if MEMORY_LIMIT + heap->size = heap->size + true_size - ZEND_MM_BLOCK_SIZE(mm_block); + if (heap->peak < heap->size) { + heap->peak = heap->size; + } +#endif + if (true_size <= ZEND_MM_BLOCK_SIZE(mm_block)) { size_t remaining_size = ZEND_MM_BLOCK_SIZE(mm_block) - true_size; @@ -1439,7 +1466,7 @@ realloc_segment: segment_copy = (zend_mm_segment *) ((char *)mm_block - ZEND_MM_ALIGNED_SEGMENT_SIZE); #if MEMORY_LIMIT - if (heap->size + segment_size - segment_copy->size > heap->limit) { + if (heap->real_size + segment_size - segment_copy->size > heap->limit) { HANDLE_UNBLOCK_INTERRUPTIONS(); #if ZEND_DEBUG zend_mm_safe_error(heap, "Allowed memory size of %d bytes exhausted at %s:%d (tried to allocate %d bytes)", heap->limit, __zend_filename, __zend_lineno, size); @@ -1453,16 +1480,16 @@ realloc_segment: if (!segment) { HANDLE_UNBLOCK_INTERRUPTIONS(); #if ZEND_DEBUG - zend_mm_safe_error(heap, "Out of memory (allocated %d) at %s:%d (tried to allocate %d bytes)", heap->size, __zend_filename, __zend_lineno, size); + zend_mm_safe_error(heap, "Out of memory (allocated %d) at %s:%d (tried to allocate %d bytes)", heap->real_size, __zend_filename, __zend_lineno, size); #else - zend_mm_safe_error(heap, "Out of memory (allocated %d) (tried to allocate %d bytes)", heap->size, size); + zend_mm_safe_error(heap, "Out of memory (allocated %d) (tried to allocate %d bytes)", heap->real_size, size); #endif return NULL; } - heap->size += segment_size - segment->size; + heap->real_size += segment_size - segment->size; #if MEMORY_LIMIT - if (heap->size > heap->peak) { - heap->peak = heap->size; + if (heap->real_size > heap->real_peak) { + heap->real_peak = heap->real_size; } #endif segment->size = segment_size; @@ -1778,15 +1805,27 @@ ZEND_API int zend_set_memory_limit(unsigned int memory_limit) #endif } -ZEND_API size_t zend_memory_usage(TSRMLS_D) +ZEND_API size_t zend_memory_usage(int real_usage TSRMLS_DC) { - return AG(mm_heap)->size; + if (real_usage) { + return AG(mm_heap)->real_size; + } else { +#if MEMORY_LIMIT + return AG(mm_heap)->size; +#else + return AG(mm_heap)->real_size; +#endif + } } #if MEMORY_LIMIT -ZEND_API size_t zend_memory_peak_usage(TSRMLS_D) +ZEND_API size_t zend_memory_peak_usage(int real_usage TSRMLS_DC) { - return AG(mm_heap)->peak; + if (real_usage) { + return AG(mm_heap)->real_peak; + } else { + return AG(mm_heap)->peak; + } } #endif diff --git a/Zend/zend_alloc.h b/Zend/zend_alloc.h index fc27ddd8f55..842a6e552d5 100644 --- a/Zend/zend_alloc.h +++ b/Zend/zend_alloc.h @@ -119,8 +119,8 @@ void zend_debug_alloc_output(char *format, ...); #endif #if MEMORY_LIMIT -ZEND_API size_t zend_memory_usage(TSRMLS_D); -ZEND_API size_t zend_memory_peak_usage(TSRMLS_D); +ZEND_API size_t zend_memory_usage(int real_usage TSRMLS_DC); +ZEND_API size_t zend_memory_peak_usage(int real_usage TSRMLS_DC); #endif END_EXTERN_C() diff --git a/ext/standard/var.c b/ext/standard/var.c index 386643d28db..7141d0c883b 100644 --- a/ext/standard/var.c +++ b/ext/standard/var.c @@ -1152,14 +1152,25 @@ PHP_FUNCTION(unserialize) /* {{{ proto int memory_get_usage() Returns the allocated by PHP memory */ PHP_FUNCTION(memory_get_usage) { + zend_bool real_usage = 0; + + if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "|b", &real_usage) == FAILURE) { + RETURN_FALSE; + } - RETURN_LONG(zend_memory_usage(TSRMLS_C)); + RETURN_LONG(zend_memory_usage(real_usage TSRMLS_CC)); } /* }}} */ /* {{{ proto int memory_get_peak_usage() Returns the peak allocated by PHP memory */ PHP_FUNCTION(memory_get_peak_usage) { - RETURN_LONG(zend_memory_peak_usage(TSRMLS_C)); + zend_bool real_usage = 0; + + if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "|b", &real_usage) == FAILURE) { + RETURN_FALSE; + } + + RETURN_LONG(zend_memory_peak_usage(real_usage TSRMLS_CC)); } /* }}} */ #endif diff --git a/sapi/apache/mod_php5.c b/sapi/apache/mod_php5.c index 543eb8ba907..be7e581597f 100644 --- a/sapi/apache/mod_php5.c +++ b/sapi/apache/mod_php5.c @@ -669,7 +669,7 @@ static int send_parsed_php(request_rec * r) char *mem_usage; TSRMLS_FETCH(); - mem_usage = ap_psprintf(r->pool, "%u", zend_memory_peak_usage(TSRMLS_C)); + mem_usage = ap_psprintf(r->pool, "%u", zend_memory_peak_usage(1 TSRMLS_CC)); ap_table_setn(r->notes, "mod_php_memory_usage", mem_usage); } #endif diff --git a/sapi/apache2filter/sapi_apache2.c b/sapi/apache2filter/sapi_apache2.c index 3fec5556290..a622d154adc 100644 --- a/sapi/apache2filter/sapi_apache2.c +++ b/sapi/apache2filter/sapi_apache2.c @@ -521,7 +521,7 @@ static int php_output_filter(ap_filter_t *f, apr_bucket_brigade *bb) { char *mem_usage; - mem_usage = apr_psprintf(ctx->r->pool, "%u", zend_memory_peak_usage(TSRMLS_C)); + mem_usage = apr_psprintf(ctx->r->pool, "%u", zend_memory_peak_usage(1 TSRMLS_CC)); apr_table_set(ctx->r->notes, "mod_php_memory_usage", mem_usage); } #endif diff --git a/sapi/apache2handler/sapi_apache2.c b/sapi/apache2handler/sapi_apache2.c index caaf0a3d8f8..35f504e4805 100644 --- a/sapi/apache2handler/sapi_apache2.c +++ b/sapi/apache2handler/sapi_apache2.c @@ -593,7 +593,7 @@ zend_first_try { { char *mem_usage; - mem_usage = apr_psprintf(ctx->r->pool, "%u", zend_memory_peak_usage(TSRMLS_C)); + mem_usage = apr_psprintf(ctx->r->pool, "%u", zend_memory_peak_usage(1 TSRMLS_CC)); apr_table_set(r->notes, "mod_php_memory_usage", mem_usage); } #endif diff --git a/sapi/apache_hooks/mod_php5.c b/sapi/apache_hooks/mod_php5.c index 1b4d7e598b3..52a6b3c143d 100644 --- a/sapi/apache_hooks/mod_php5.c +++ b/sapi/apache_hooks/mod_php5.c @@ -727,7 +727,7 @@ static int send_parsed_php(request_rec * r) char *mem_usage; TSRMLS_FETCH(); - mem_usage = ap_psprintf(r->pool, "%u", zend_memory_peak_usage(TSRMLS_C)); + mem_usage = ap_psprintf(r->pool, "%u", zend_memory_peak_usage(1 TSRMLS_CC)); ap_table_setn(r->notes, "mod_php_memory_usage", mem_usage); } #endif