Bug #81275 [PATCH]: status page, contain bugus value in request duration sometimes

Edit report at https://bugs.php.net/bug.php?id=81275&edit=1 ID: 81275 Patch added by: bukka@php.net Reported by: tsmtgdi at gmail dot com Summary: status page, contain bugus value in request duration sometimes Status: Assigned Type: Bug Package: FPM related Operating System: Suse PHP Version: 8.1Git-2021-07-19 (Git) Assigned To: bukka Block user comment: N Private report: N New Comment: The following pull request has been associated: Patch Name: Fix #76109: Implement fpm_scoreboard_copy On GitHub: https://github.com/php/php-src/pull/8049 Patch: https://github.com/php/php-src/pull/8049.patch Previous Comments: ------------------------------------------------------------------------ [2022-02-08 22:42:17] bukka@php.net Thanks for the reproducing script which worked perfectly for me in this setup https://github.com/bukka/php-util/tree/7ac45d4e51465f60ad69589f9e06a8ef5b5cec16/tests/fpm/scoreboard-race . I was able to recreate the issue with PHP-8.0 and after applying https://github.com/php/php-src/pull/8049 , the issue is no longer present. I will merge it tomorrow. ------------------------------------------------------------------------ [2021-09-05 19:23:26] bukka@php.net Ah so it's not maybe such a theoretical issue that I thought initially. That PR looks like a potential fix indeed. I will need to test it to confirm though. Will also think about creating some test for this (most likely the slow one if possible). ------------------------------------------------------------------------ [2021-08-03 13:10:14] nikic@php.net This is the PR I had in mind: https://github.com/php/php-src/pull/3188 ------------------------------------------------------------------------ [2021-08-03 13:05:24] ramsey@php.net bukka, can you take a look? nikic thinks there might already be a PR for this, but I can't find it. ------------------------------------------------------------------------ [2021-07-20 01:20:27] tsmtgdi at gmail dot com I tried doing int scoreboard_size = sizeof(struct fpm_scoreboard_s) + (scoreboard_p->nprocs) * sizeof(struct fpm_scoreboard_proc_s*); int scoreboard_nprocs_size = sizeof(struct fpm_scoreboard_proc_s) * scoreboard_p->nprocs; scoreboardCopy = (struct fpm_scoreboard_s*)emalloc(scoreboard_size + scoreboard_nprocs_size); memcpy(scoreboardCopy, scoreboard_p, scoreboard_size + scoreboard_nprocs_size); To have a full copy, both of the fpm_scoreboard_s and the array of fpm_scoreboard_proc_s, but the problem is still present. ------------------------------------------------------------------------ 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=81275 -- Edit this bug report at https://bugs.php.net/bug.php?id=81275&edit=1