Bug #81673 [Com]: Tracing JIT causing various errors (trait related?)

From: Date: Tue, 02 May 2023 12:25:35 +0000
Subject: Bug #81673 [Com]: Tracing JIT causing various errors (trait related?)
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-244336@lists.php.net to get a copy of this message
Edit report at https://bugs.php.net/bug.php?id=81673&edit=1

 ID:                 81673
 Comment by:         joanwest23 at aol dot com
 Reported by:        eric at ericstern dot com
 Summary:            Tracing JIT causing various errors (trait related?)
 Status:             Open
 Type:               Bug
 Package:            JIT
 Operating System:   Alpine linux
 PHP Version:        8.0.13
 Block user comment: N
 Private report:     N

 New Comment:

(https://www.linkedin.com/pulse/augusta-precious-metals-review-best-gold-ira-company-jeffrey-keever/)github.com
(https://www.linkedin.com/pulse/goldco-review-gold-ira-rollover-precious-metal-company-jeffrey-keever/)github.com
(https://www.linkedin.com/pulse/how-apply-employee-retention-credit-erc-complete-guide-jeffrey-keever/)github.com
(https://www.linkedin.com/pulse/best-gold-ira-companies-investment-retirement-accounts-jeffrey-keever/)github.com
(https://www.linkedin.com/pulse/ira-allowable-precious-metals-best-ira-approved-metal-jeffrey-keever/)github.com
(https://www.linkedin.com/pulse/roth-gold-ira-account-best-companies-how-works-jeffrey-keever/)github.com
(https://www.linkedin.com/pulse/what-employee-retention-tax-credit-ertc-jeffrey-keever/)github.com
(https://www.linkedin.com/pulse/best-precious-metals-ira-2023-jeffrey-keever/)github.com
(https://www.linkedin.com/pulse/best-silver-ira-companies-2023-jeffrey-keever/)github.com
(https://www.linkedin.com/pulse/how-protect-your-401k-from-market-crash-recession-jeffrey-keever/)github.com
(https://www.linkedin.com/pulse/goldco-vs-augusta-precious-metals-which-best-gold-ira-jeffrey-keever/)github.com
(https://www.linkedin.com/pulse/how-move-401k-gold-ira-without-penalty-jeffrey-keever/)github.com 
(https://www.linkedin.com/pulse/how-buy-physical-gold-your-401k-jeffrey-keever/)github.com
(https://www.linkedin.com/pulse/noble-gold-investments-review-jeffrey-keever/)github.com
(https://www.linkedin.com/pulse/best-paying-jobs-precious-metals-jeffrey-keever/)github.com
(https://www.linkedin.com/pulse/how-buy-gold-ira-jeffrey-keever-1e/)github.com


Previous Comments:
------------------------------------------------------------------------
[2023-04-25 10:12:21] fariba dot shami326 at gmail dot com

Tracing JIT (Just-In-Time) compilers are a type of JIT compiler that compile sections of code at
runtime based on the execution patterns of that code. While Tracing JITs can improve performance by
generating optimized machine code, they can also introduce new errors and bugs that may be difficult
to diagnose.

One common issue with Tracing JITs is related to object-oriented programming and the use of traits.
Traits are a way to reuse code by encapsulating a set of methods that can be applied to different
classes. However, traits can make it difficult for Tracing JITs to optimize code, as they may need
to track multiple execution paths for each trait.

To avoid errors related to Tracing JITs and traits, you may need to modify your code to make it more
amenable to optimization. This may involve simplifying complex inheritance hierarchies, minimizing
the use of traits, or using other techniques to reduce the number of execution paths that need to be
tracked.
(https://www.questdiagnosticsbill.me/)github.com

------------------------------------------------------------------------
[2023-04-13 15:53:17] ascanpardinum at gmail dot com

(https://www.metooo.io/u/jasa-pembuatan-website).php.net
(https://steamcommunity.com/id/astraleliteforce/).php.net

------------------------------------------------------------------------
[2023-01-27 11:55:55] rtsonldspj11 at gmail dot com

A commitment of appreciation is all together for the data. I will endeavor to figure it out for
extra. (https://www.readypayonline.me/)github.com

------------------------------------------------------------------------
[2021-11-30 00:47:42] julien dot boudry at gmail dot com

Not exactly the same error, but the same context, only with tracing mode. And the code use trait
too.
https://bugs.php.net/bug.php?id=81432

------------------------------------------------------------------------
[2021-11-29 22:02:47] eric at ericstern dot com

Description:
------------
When enabling the JIT in a production environment, I started getting various errors in third-party
libraries (guzzlehttp/psr7, though I doubt it's relevant). After some poking and prodding, I
was able to narrow it down to the JIT running in tracing mode (via "on" or
explicitly "tracing"). Turning it back off or running it in "function" mode made
the errors disappear.

Notably, if I'm following my logs correctly, the first job seems to run fine. It's only on
the second+ job where the error comes up.

The errors that occurred included:

TypeError: GuzzleHttp\Psr7\Request::getBody(): Return value must be of type
Psr\Http\Message\StreamInterface, null returned in
/var/www/html/vendor/guzzlehttp/psr7/src/MessageTrait.php:127

Error: Cannot use object of type GuzzleHttp\Psr7\Uri as array in
/var/www/html/vendor/guzzlehttp/psr7/src/MessageTrait.php:161

(see linked file below)

In both cases, the errors make absolutely no sense for the given source code, especially the latter.
Which error I get seems basically random, though the latter seems more common for any given worker.
My gut tells me the issue is somehow specific to traits, but I don't have any practical way to
verify that.

INI settings:
opcache.enable="1"
opcache.enable_cli="1"
(typical opcache prod values, preloading is enabled)
opcache.jit="on"
opcache.jit_buffer_size="128M"

Test script:
---------------
https://github.com/Firehed/php-jit-tracing-error
- this doesn't actually reproduce the issue (yet), but it shows the approximate code structure
where it occurs.

The surrounding infrastructure/code in prod is fairly long and complex, so I'm not sure how
much needs to get added in to reproduce the issue. But it's a run loop for an async job worker
which performs a network request and then parses and stores the results in a database. Nothing too
special. The code in question has been live for months without ever having an issue.

Errors originate in this file:

https://github.com/guzzle/psr7/blob/2.1.0/src/MessageTrait.php

Expected result:
----------------
No errors

Actual result:
--------------
One backtrace:

TypeError: GuzzleHttp\Psr7\Request::getBody(): Return value must be of type
Psr\Http\Message\StreamInterface, null returned in
/var/www/html/vendor/guzzlehttp/psr7/src/MessageTrait.php:127
Stack trace:
#0 /var/www/html/vendor/guzzlehttp/psr7/src/Utils.php(222): GuzzleHttp\Psr7\Request->getBody()
#1 /var/www/html/vendor/guzzlehttp/guzzle/src/Client.php(440):
GuzzleHttp\Psr7\Utils::modifyRequest(Object(GuzzleHttp\Psr7\Request), Array)
#2 /var/www/html/vendor/guzzlehttp/guzzle/src/Client.php(326):
GuzzleHttp\Client->applyOptions(Object(GuzzleHttp\Psr7\Request), Array)
#3 /var/www/html/vendor/guzzlehttp/guzzle/src/Client.php(107):
GuzzleHttp\Client->transfer(Object(GuzzleHttp\Psr7\Request), Array)
#4 /var/www/html/vendor/guzzlehttp/guzzle/src/Client.php(123):
GuzzleHttp\Client->sendAsync(Object(GuzzleHttp\Psr7\Request), Array)
#5 /var/www/html/src/Worker/FetchStoreData.php(241):
GuzzleHttp\Client->send(Object(GuzzleHttp\Psr7\Request), Array)
#6 /var/www/html/src/Console/Worker/Run.php(213):
Slant\Normalizer\Worker\FetchStoreData->doJob(Array)
#7 [internal function]:
Slant\Normalizer\Console\Worker\Run->Slant\Normalizer\Console\Worker\{closure}(Object(Kicken\Gearman\Job\WorkerJob))
#8 /var/www/html/vendor/kicken/gearman-php/src/Worker.php(194): call_user_func(Object(Closure),
Object(Kicken\Gearman\Job\WorkerJob))
#9 /var/www/html/vendor/kicken/gearman-php/src/Worker.php(177):
Kicken\Gearman\Worker->processJob(Object(Kicken\Gearman\Protocol\Packet))
#10 /var/www/html/vendor/kicken/gearman-php/src/Worker.php(116):
Kicken\Gearman\Worker->processPacket(Object(Kicken\Gearman\Protocol\Packet))
#11 /var/www/html/src/Console/Worker/Run.php(145): Kicken\Gearman\Worker->workOnce()
#12 /var/www/html/vendor/symfony/console/Command/Command.php(299):
Slant\Normalizer\Console\Worker\Run->execute(Object(Symfony\Component\Console\Input\ArgvInput),
Object(Symfony\Component\Console\Output\ConsoleOutput))
#13 /var/www/html/vendor/symfony/console/Application.php(978):
Symfony\Component\Console\Command\Command->run(Object(Symfony\Component\Console\Input\ArgvInput),
Object(Symfony\Component\Console\Output\ConsoleOutput))
#14 /var/www/html/vendor/symfony/console/Application.php(295):
Symfony\Component\Console\Application->doRunCommand(Object(Slant\Normalizer\Console\Worker\Run),
Object(Symfony\Component\Console\Input\ArgvInput),
Object(Symfony\Component\Console\Output\ConsoleOutput))
#15 /var/www/html/vendor/symfony/console/Application.php(167):
Symfony\Component\Console\Application->doRun(Object(Symfony\Component\Console\Input\ArgvInput),
Object(Symfony\Component\Console\Output\ConsoleOutput))
#16 /var/www/html/bin/app(63): Symfony\Component\Console\Application->run()
#17 {main}


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



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


Thread (15 messages)

« previous php.bugs (#244336) next »