Skip to content

[PHP] fix: split fork/process/shared-memory runtime fixes#737

Closed
brandonpayton wants to merge 8 commits into
Automattic:integration/kd-6nz-php-phpt-socket-network-basefrom
brandonpayton:gascity/kd-6nz/kd-e0l-pr717-fork-process-shm-split
Closed

[PHP] fix: split fork/process/shared-memory runtime fixes#737
brandonpayton wants to merge 8 commits into
Automattic:integration/kd-6nz-php-phpt-socket-network-basefrom
brandonpayton:gascity/kd-6nz/kd-e0l-pr717-fork-process-shm-split

Conversation

@brandonpayton

@brandonpayton brandonpayton commented Jun 20, 2026

Copy link
Copy Markdown
Member

Summary

Stack

Preserved commits

  • 6ecec96 fix: isolate PHPT workers and preserve process status
  • 3d27d6e fix: share anonymous mmap across fork
  • 74f7375 fix: honor mmap address hints
  • 940d885 fix: improve shared memory and fpm socket semantics
  • 5368b79 fix: restore fork socket exec reader
  • 8496bd0 fix: support fork from dlopened wasm side modules
  • fb768e0 Preserve shared mappings across fork for PHP PHPTs
  • 9a7ec18 host: avoid single-observer shared-memory sync churn

Verification

  • git diff --check origin/integration/kd-6nz-php-phpt-socket-network-base...HEAD

@brandonpayton brandonpayton force-pushed the integration/kd-6nz-php-phpt-socket-network-base branch from f41e20d to 7450c64 Compare June 20, 2026 21:18
Kandelo Agent and others added 8 commits June 20, 2026 17:21
(cherry picked from commit 079b089)
Port the PR #2 shared-memory/fork fixes onto current Automattic/kandelo. Centralized host workers now inherit and synchronize MAP_SHARED and SysV shared memory across fork, wait, detach, and exit boundaries so parent/child processes observe real shared state.

Grow exec-state serialization instead of failing at a fixed 64 KiB buffer, keep Node/browser fork paths in parity, refresh the ABI snapshot for ABI 16, and update focused host/kernel fixtures and tests for the port.
@brandonpayton

Copy link
Copy Markdown
Member Author

Closing as superseded by origin-headed replacement #756: #756

This PR head was in brandonpayton/kandelo-agentic-dev. The replacement uses the same head SHA (9a7ec18) on an Automattic/kandelo origin branch and has an expanded description explaining why the change is needed and why it belongs in Kandelo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant