Skip to content

Fix reforge workspace isolation and durable result publication - #1

Draft
0z5a wants to merge 3 commits into
kacper-daftcode:masterfrom
0z5a:fix/reforge-workspace
Draft

0z5a wants to merge 3 commits into
kacper-daftcode:masterfrom
0z5a:fix/reforge-workspace

Conversation

@0z5a

@0z5a 0z5a commented Sep 24, 2026 •

Copy link
Copy Markdown

Concurrent reforge calls currently overwrite /tmp/reforge_current.cubin and /tmp/reforge_best.cubin. Give each call an owned temporary workspace and atomically publish a durable best cubin without overwriting existing output or the input.

Adds optional --output / --work-dir and state.best_cubin_path; existing invocations retain defaults. Mutation/search policy is unchanged.

Validation Result
Forced 2/8-process baseline race Reproduced token corruption
Workspace/publication/CLI tests 11 passed
Numerical SM120 integration 1/2/4/8 GPU workers passed
Original fixture tests 7 skipped; original fixture unavailable

Validation details. Multi-GPU evidence uses the integrated snapshot/swap fixes and a known numerical ABI. No kernel speedup is claimed for file isolation.

Replayable integration fixtures are preserved on the integration branch.

…ring

Signed-off-by: 0z5a <dezhen.lu@student.uni-tuebingen.de>
Signed-off-by: 0z5a <dezhen.lu@student.uni-tuebingen.de>
Signed-off-by: 0z5a <dezhen.lu@student.uni-tuebingen.de>
@0z5a
0z5a force-pushed the fix/reforge-workspace branch from 925adb6 to c8f498d Compare September 24, 2026 10:29
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