Bug #79519 [Com]: Memory leak in PHP 7.4

From: Date: Thu, 09 Dec 2021 12:20:14 +0000
Subject: Bug #79519 [Com]: Memory leak in PHP 7.4
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-238301@lists.php.net to get a copy of this message
Edit report at https://bugs.php.net/bug.php?id=79519&edit=1

 ID:                 79519
 Comment by:         brendt at stitcher dot io
 Reported by:        kieran at miami-nice dot co dot uk
 Summary:            Memory leak in PHP 7.4
 Status:             Closed
 Type:               Bug
 Package:            Performance problem
 Operating System:   Linux
 PHP Version:        7.4.5
 Assigned To:        nikic
 Block user comment: N
 Private report:     N

 New Comment:

I want to follow up on this one, since the issue doesn't seem to be resolved.

Tested on PHP 8.1.0 and PHPUnit 9.5.10; running a Laravel test suite of 1400 tests results in a
rather significant memory leak. Here's a screenshot of the memory consumption plotted in a
chart: https://user-images.githubusercontent.com/6905297/145394833-dc488206-238d-4164-86e4-e2ad8c824590.png

This is the issue related to Laravel discussing the problem: https://github.com/laravel/framework/issues/30736
And here's the phpunit issue: https://github.com/sebastianbergmann/phpunit/issues/3915


Previous Comments:
------------------------------------------------------------------------
[2021-06-09 13:01:12] nikic@php.net

In PHP 8.1, the memory leak for repeated included anonymous functions has been addressed by https://github.com/php/php-src/pull/5595 (or at
least, mostly addressed).

The GC with destructors issue has been addressed by https://github.com/php/php-src/pull/5581.

I believe that covers the issues raised here.

------------------------------------------------------------------------
[2020-09-22 21:27:39] nikic@php.net

Related To: Bug #80140

------------------------------------------------------------------------
[2020-08-10 20:30:00] nikic@php.net

I did some work on this in https://github.com/php/php-src/pull/5593 and https://github.com/php/php-src/pull/5595, but
unfortunately it's not easy to fix.

------------------------------------------------------------------------
[2020-08-10 20:19:32] kieran at supportpal dot com

Will there be a fix for this issue in PHP 8? Just tested against the PHP 8.0-beta.1 docker image and
memory usage matches PHP 7.4

------------------------------------------------------------------------
[2020-05-21 19:32:20] kieran at supportpal dot com

@nikita I think you're probably right:
> PHP 7.4 fixed some issues related to anonymous functions "overwriting" previous
> anonymous functions in the same file

I "memorised" the faker requires as suggested and it cut usage from 3.8G to 2.6G. So I
assume similar changes elsewhere would further reduce memory usage. The only problem is anonymous
function usage is rife in Laravel!

Not sure if you want to close this as duplicate of https://bugs.php.net/bug.php?id=76982 or repurpose
following below comment:
> The master branch case is still potentially actionable from our side.

------------------------------------------------------------------------


The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at

    https://bugs.php.net/bug.php?id=79519


--
Edit this bug report at https://bugs.php.net/bug.php?id=79519&edit=1


Thread (25 messages)

« previous php.bugs (#238301) next »