Skip to content

feat(sigsafe): resolve Linux descriptor paths - #610

Merged
wan9chi merged 1 commit into
agent/sigsafe-fs-wrappersfrom
agent/sigsafe-linux-readlink
Aug 9, 2026
Merged

feat(sigsafe): resolve Linux descriptor paths#610
wan9chi merged 1 commit into
agent/sigsafe-fs-wrappersfrom
agent/sigsafe-linux-readlink

Conversation

@wan9chi

@wan9chi wan9chi commented Aug 9, 2026

Copy link
Copy Markdown
Member

Motivation

Linux descriptor path resolution still uses nix::readlink and heap-backed formatting. Add a raw caller-buffer readlinkat wrapper and allocator adapter, then immediately replace that path with stack formatting and arena storage.

The low-level wrapper preserves readlink truncation semantics; the allocating adapter retries only when the buffer is filled.

@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown

fspy benchmark

linux

macos

dynamic/launch             change  +0.04%  [ -4.41% ..  +5.03%]  overhead  +230.24%
dynamic/access             change  +1.32%  [ -7.65% .. +137.22%]  overhead    +6.55%
dynamic/access-relative    change  -0.31%  [ -6.02% ..  +3.01%]  overhead  +379.59%

windows

dynamic/launch             change  -0.03%  [ -2.58% ..  +1.50%]  overhead   +27.91%
dynamic/access             change  +0.00%  [ -0.76% ..  +1.17%]  overhead    +1.76%
dynamic/access-relative    change  +0.19%  [ -0.57% ..  +0.96%]  overhead    +1.32%

Co-authored-by: GPT-5 Codex <codex@openai.com>
@wan9chi
wan9chi force-pushed the agent/sigsafe-linux-readlink branch from 5fbbf64 to 46f3e38 Compare August 9, 2026 11:26
Base automatically changed from agent/sigsafe-getcwd to agent/sigsafe-fs-wrappers August 9, 2026 11:26
@wan9chi
wan9chi merged commit 46f3e38 into main Aug 9, 2026
1 check passed
@wan9chi
wan9chi deleted the agent/sigsafe-linux-readlink branch August 9, 2026 11:26
@wan9chi
wan9chi restored the agent/sigsafe-linux-readlink branch August 9, 2026 11:27
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