php-in-php: JIT StreamNotificationRuntime — route stream_notification_callback through ext/standard PHP not LLVM global (#1492)

[PurHur]

Category

php-in-php · stdlib · streams (#1492)

Problem

stream_notification_callback() stores the user notifier in a JIT/AOT LLVM global via lib/JIT/Builtin/StreamNotificationRuntime.php, while the VM builtin already lives in ext/standard/stream_notification_callback.php.

Stream wrappers invoke the callback from both VM and JIT paths; duplicated storage causes risk of VM-set callbacks not visible to JIT stream I/O (and vice versa).

php-src reference

• php/php-src ext/standard/streams.c — PHP_FUNCTION(stream_notification_callback), php_stream_notification helpers
• php/php-src main/streams/streams.c — notification dispatch

Repro

<?php
stream_notification_callback(function ($ncode, $severity, $message, $code, $transferred, $max) {
    echo "notify:$ncode\n";
});
$ctx = stream_context_create(['http' => ['timeout' => 1]]);
@file_get_contents('http://127.0.0.1:1/', false, $ctx);
echo "done\n";

./script/docker-exec.sh -- php bin/vm.php repro.php
./script/docker-exec.sh -- bash -lc 'source script/php-env.sh && php bin/jit.php repro.php'

Callback must fire identically on VM and JIT; today JIT may read a different global slot than VM PHP storage.

Scope (this repo)

Layer	Path
VM SSOT	ext/standard/stream_notification_callback.php, stream context state
LLVM today	lib/JIT/Builtin/StreamNotificationRuntime.php (__phpc_stream_notification_callback_set)
JIT streams	lib/JIT/Builtin/StreamReadJit.php, StreamIoJit.php — call PHP getter

Done when

• Single PHP-owned callback registry shared by VM + JIT/AOT
• StreamNotificationRuntime.php deleted or link stub only
• Compliance guard with VM + JIT .phpt under test/compliance/cases/stdlib/stream_notification_callback*.phpt
• PR cites php-src + LLVM lines removed

Related

• Stdlib: stream_notification_callback() — stream progress notifications missing (ext/standard/streams.c) #6055 · Stdlib: stream_filter_append() / stream_filter_prepend() / stream_filter_register() (ext/standard/streams.c parity) #3283 · Epic: Self-host critical path — compiler compiles itself (M3→M5) #1492

[PurHur]

claim: PHP Runtime → PHP — batch (issues #9181 getdate slice, #9478 stream notification)

[PurHur]

Merged via #10963.

Verification: StreamNotificationRuntimeShrinkTest + StreamNotificationCallbackBuiltinTest + spine sync green.

LLVM shrink: removed phpc_stream_notification_callback LLVM global; JIT set path exchanges via compiled StreamNotificationJitHelper::jitCallbackSlot() shared with VM dispatch.