Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 927461 - dev-lang/php-8.2.15 fails tests on big-endian: Bug #78987 High memory usage during encoding detection [ext/fileinfo/tests/bug78987.phpt]
Summary: dev-lang/php-8.2.15 fails tests on big-endian: Bug #78987 High memory usage d...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: PPC64 Linux
: Normal normal (vote)
Assignee: PHP Bugs
URL: https://github.com/php/php-src/issues...
Whiteboard:
Keywords:
Depends on:
Blocks: 920701
  Show dependency tree
 
Reported: 2024-03-21 22:31 UTC by matoro
Modified: 2024-04-11 20:57 UTC (History)
4 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
build.log and emerge --info (build.log.gz,294.11 KB, application/gzip)
2024-03-21 22:32 UTC, matoro
Details
tarball of failed test logs (bug927461.tar.gz,8.70 KB, application/gzip)
2024-03-21 22:34 UTC, matoro
Details
build.log and emerge --info (build.log.gz,281.12 KB, application/gzip)
2024-04-03 14:56 UTC, matoro
Details
tarball of failed test logs (bug-927461.tar.gz,8.62 KB, application/gzip)
2024-04-04 16:25 UTC, matoro
Details

Note You need to log in before you can comment on or make changes to this bug.
Description matoro archtester 2024-03-21 22:31:56 UTC
Does not fail on LE.

=====================================================================
FAILED TEST SUMMARY
---------------------------------------------------------------------
Bug #78987 High memory usage during encoding detection [ext/fileinfo/tests/bug78987.phpt]
=====================================================================

---- EXPECTED OUTPUT
131072   => ok
262144   => ok
524288   => ok
1048576  => ok
2097152  => ok
4194304  => ok
8388608  => ok
16777216 => ok
---- ACTUAL OUTPUT
131072   => 9175040
262144   => 9175040
524288   => ok
1048576  => 11272192
2097152  => 13434880
4194304  => 17694720
8388608  => 23986176
16777216 => 36569088
---- FAILED

Reproducible: Always
Comment 1 matoro archtester 2024-03-21 22:32:56 UTC
Created attachment 888120 [details]
build.log and emerge --info
Comment 2 matoro archtester 2024-03-21 22:34:55 UTC
Created attachment 888121 [details]
tarball of failed test logs
Comment 3 matoro archtester 2024-03-21 22:36:06 UTC
Affects sparc as well, so assuming all big-endian.
Comment 4 Michael Orlitzky gentoo-dev 2024-03-23 12:09:25 UTC
Could you please try php-8.3.3?

If this is already fixed in 8.3 (and not easy to backport), I'll disable the test in 8.2. Otherwise, it will be easier to report upstream if it's confirmed in the latest release.
Comment 5 matoro archtester 2024-03-23 23:10:08 UTC
(In reply to Michael Orlitzky from comment #4)
> Could you please try php-8.3.3?
> 
> If this is already fixed in 8.3 (and not easy to backport), I'll disable the
> test in 8.2. Otherwise, it will be easier to report upstream if it's
> confirmed in the latest release.

It's not fixed in dev-lang/php-8.3.3-r1.
Comment 6 Michael Orlitzky gentoo-dev 2024-03-23 23:42:31 UTC
(In reply to matoro from comment #5)
> It's not fixed in dev-lang/php-8.3.3-r1.

Thanks, I reported it upstream at $URL.
Comment 7 Michael Orlitzky gentoo-dev 2024-04-02 12:35:31 UTC
I should have asked you for the updated expected/actual numbers from 8.3.3 because they changed recently.

Would you mind posting them? It would be 8.3.4 now.
Comment 8 matoro archtester 2024-04-03 14:56:51 UTC
Created attachment 889361 [details]
build.log and emerge --info

Sure, here's latest log for 8.3.4.
Comment 9 Michael Orlitzky gentoo-dev 2024-04-03 15:11:21 UTC
(In reply to matoro from comment #8)
> 
> Sure, here's latest log for 8.3.4.

If you still have that build directory, can you post ext/fileinfo/tests/bug78987.* ?

PHP helpfully hides the actual error message in a bunch of other files named after the test that failed.
Comment 10 matoro archtester 2024-04-04 16:25:59 UTC
Created attachment 889446 [details]
tarball of failed test logs

Done, here
Comment 11 matoro archtester 2024-04-04 16:26:50 UTC
---- EXPECTED OUTPUT
131072   => ok
262144   => ok
524288   => ok
1048576  => ok
2097152  => ok
4194304  => ok
8388608  => ok
16777216 => ok
---- ACTUAL OUTPUT
131072   => 10092544
262144   => 10092544
524288   => 12189696
1048576  => 12189696
2097152  => 14352384
4194304  => 18612224
8388608  => 24903680
16777216 => 37486592
---- FAILED
Comment 12 Larry the Git Cow gentoo-dev 2024-04-09 13:51:24 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=ccc5cdc89ecf3ad501acfa34650e63b3329e18da

commit ccc5cdc89ecf3ad501acfa34650e63b3329e18da
Author:     Michael Orlitzky <mjo@gentoo.org>
AuthorDate: 2024-04-09 13:08:47 +0000
Commit:     Michael Orlitzky <mjo@gentoo.org>
CommitDate: 2024-04-09 13:50:55 +0000

    dev-lang/php: fix a failing test and a musl compatibility issue
    
    Closes: https://bugs.gentoo.org/927461
    Closes: https://bugs.gentoo.org/928072
    Signed-off-by: Michael Orlitzky <mjo@gentoo.org>

 dev-lang/php/files/fix-musl-llvm.patch | 37 ++++++++++++++++++++++++++++++++++
 dev-lang/php/php-8.1.27-r2.ebuild      |  5 +++--
 dev-lang/php/php-8.2.17.ebuild         |  7 +++++++
 dev-lang/php/php-8.3.4.ebuild          |  7 +++++++
 4 files changed, 54 insertions(+), 2 deletions(-)
Comment 13 matoro archtester 2024-04-11 19:45:30 UTC
Can you extend this fix to 8.2 to unblock bug 920701?
Comment 14 Larry the Git Cow gentoo-dev 2024-04-11 20:57:26 UTC
The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=7d56cd36ac6b6d468e20d6333ae2917c6c39c146

commit 7d56cd36ac6b6d468e20d6333ae2917c6c39c146
Author:     Michael Orlitzky <mjo@gentoo.org>
AuthorDate: 2024-04-11 19:50:37 +0000
Commit:     Michael Orlitzky <mjo@gentoo.org>
CommitDate: 2024-04-11 20:55:06 +0000

    dev-lang/php: skip doomed test in 8.2.15, too
    
    Bug: https://bugs.gentoo.org/927461
    Signed-off-by: Michael Orlitzky <mjo@gentoo.org>

 dev-lang/php/php-8.2.15.ebuild | 6 ++++++
 1 file changed, 6 insertions(+)