Skip to content

refactor(fspy-shm): open backing files through sigsafe - #630

Draft
wan9chi wants to merge 1 commit into
mainfrom
feat/fspy-shm-injected-runtime
Draft

refactor(fspy-shm): open backing files through sigsafe#630
wan9chi wants to merge 1 commit into
mainfrom
feat/fspy-shm-injected-runtime

Conversation

@wan9chi

@wan9chi wan9chi commented Aug 10, 2026

Copy link
Copy Markdown
Member

Motivation

The injected runtime may attach before libc initialization, while fspy_shm::open used std::fs::OpenOptions. This PR routes Linux backing-file opens through the sigsafe raw openat wrapper with a caller-owned path buffer, preserving the existing fspy_shm API and storage behavior.

Co-authored-by: GPT-5 Codex <codex@openai.com>
@wan9chi wan9chi changed the title feat/fspy shm injected runtime refactor(fspy-shm): open backing files through sigsafe Aug 10, 2026
@github-actions

Copy link
Copy Markdown

fspy benchmark

linux

dynamic/launch             change  +1.33%  [ -7.44% ..  +8.59%]  overhead   +61.14%
dynamic/access             change  +0.39%  [-17.42% .. +13.88%]  overhead    +7.94%
dynamic/access-relative    change  +0.29%  [-21.57% .. +21.49%]  overhead   +43.14%
static/launch              change  +0.98%  [ -7.49% ..  +9.59%]  overhead  +181.74%
static/access              change  -0.74%  [ -7.39% ..  +3.56%]  overhead  +874.06%
static/access-relative     change  +0.18%  [ -4.62% ..  +6.40%]  overhead +1335.18%

macos

dynamic/launch             change  -0.39%  [ -5.67% ..  +5.60%]  overhead  +241.05%
dynamic/access             change  -0.86%  [-10.69% ..  +8.36%]  overhead    +4.32%
dynamic/access-relative    change  +3.32%  [ -9.34% .. +59.79%]  overhead  +246.20%

windows

dynamic/launch             change  +0.44%  [ -4.83% ..  +5.71%]  overhead   +28.77%
dynamic/access             change  -0.54%  [ -5.88% ..  +2.34%]  overhead    +1.79%
dynamic/access-relative    change  -0.17%  [ -1.91% ..  +1.25%]  overhead    +1.60%

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