Skip to content

Commit 23ae029

Browse files
committed
docs(perf): record rejected owned server batch screen
1 parent d2534a8 commit 23ae029

5 files changed

Lines changed: 833 additions & 0 deletions

File tree

scripts/perf/README.md

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -843,6 +843,19 @@ it does not establish a performance gain or neutrality. Keep endpoint readiness,
843843
cache completion and offline witness cost separate, and never extrapolate a
844844
processing-probe gain to the shipping HTTP artifact.
845845
[Sanitized screen and limitations](evidence/server-json-roundtrip-4064/README.md).
846+
### Rejected owned server mesh-batch transfer (#4066)
847+
848+
An owned sink in the canonical processing loop removed the server bridge's deep
849+
mesh-buffer copy while preserving borrowed callers, retained output, batching,
850+
progress, styling and cancellation. Exact output and actual cache replay passed,
851+
but the prespecified HTTP readiness continuation gate failed: the small and MEP
852+
models were slower in their single pairs, and the largest model's modest time
853+
improvement accompanied higher sampled RSS. The implementation was not landed
854+
or expanded to the wider corpus. A source-level copy removal is not an
855+
end-to-end gain; unbounded downstream ownership and other pipeline work remain
856+
relevant costs. This screen does not establish precise attribution or a physical
857+
memory benefit. [Complete verdict, limits and reproducible rejected source](evidence/rejected-owned-batches-2026-09-07/README.md).
858+
846859
## Current-source native PGO qualification (#4059, not shipped)
847860

848861
Fresh profile training on five public fixtures produced a broad held-out
Lines changed: 55 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,55 @@
1+
# Rejected owned server mesh batches (#4066)
2+
3+
The candidate transferred non-retained mesh batches into the existing server
4+
channel instead of cloning their owned buffers. It used the same canonical
5+
processing loop and preserved the borrowed API, retained output, batch/progress
6+
boundaries, styling and cancellation. It was rejected by the prespecified actual
7+
HTTP continuation rule. No production implementation or wider corpus run landed.
8+
9+
[screen.json](screen.json) contains every arm of the initial three-model screen,
10+
its exact gates, separate startup/readiness clocks, sampled RSS, continuation
11+
rule, source/artifact/compiler identities and validation limits. The small model
12+
and MEP model were slower in their single pairs; the largest model improved
13+
modestly while its sampled RSS increased. These observations neither establish
14+
precise causal contributions nor justify a general claim that ownership transfer
15+
cannot help. Removing a real copy does not by itself establish an end-to-end win.
16+
17+
All three models passed the declared complete semantic witness, strict raw
18+
geometry/data-model comparisons, actual hash-only cache replay and cleanup gates.
19+
No diagnostic mismatch was waived. The primary clock ends when complete SSE and
20+
data-model receipt both finish; startup is separate. This is HTTP qualification,
21+
not browser readiness or physical-footprint evidence. Every offline decode and
22+
retention operation occurred after both timed arms. The isolated verifier reused
23+
only complete byte-equal inputs with validated witness/version identities;
24+
current completion and transport fields were independently checked.
25+
26+
The workspace tests and strict workspace/all-target Clippy passed on the exact
27+
candidate. Downloaded external fixtures were absent: the preserved test evidence
28+
records ignored tests and fixture-skip messages. Inline/committed-fixture tests
29+
exercise ownership, both APIs, retained output, styling/progress/bootstrap
30+
sequences and cancellation. They complement the actual HTTP cases and are not a
31+
substitute for corpus qualification.
32+
33+
## Reproduce the rejected source
34+
35+
[measured-candidate.patch](measured-candidate.patch) archives the complete local
36+
candidate, including its tests and Rust API documentation. It uses zero-context
37+
full-index Git format so patch-context blank lines do not trigger the repository
38+
whitespace gate. In an isolated checkout at the public base recorded in
39+
[source-archive.json](source-archive.json), run:
40+
41+
```sh
42+
git apply --unidiff-zero /path/to/measured-candidate.patch
43+
```
44+
45+
Temporary-index application reconstructed the **exact measured candidate tree**.
46+
The public base's Rust/server/Cargo source also matches the measured control.
47+
The archive records these identities and the patch digest; applying it is not a
48+
new build or performance measurement. The candidate used the same pinned
49+
compiler, explicit standard-library build configuration, target and release
50+
profile as the control, with no PGO inputs.
51+
52+
Raw captures and complete private logs remain retained; repository evidence
53+
contains sanitized measurements and their hash references, not those large
54+
payloads. Exact duplicate evidence files share APFS extents while retaining
55+
separate paths and inodes. No unique raw evidence was removed.

0 commit comments

Comments
 (0)