Skip to content

Ignore /mnt prefix when scanning for multicall binaries on wsl2#96

Closed
dwt wants to merge 2 commits intoUse-Tusk:mainfrom
dwt:fix/slow-sandbox-start-windows
Closed

Ignore /mnt prefix when scanning for multicall binaries on wsl2#96
dwt wants to merge 2 commits intoUse-Tusk:mainfrom
dwt:fix/slow-sandbox-start-windows

Conversation

@dwt
Copy link
Copy Markdown
Collaborator

@dwt dwt commented Apr 1, 2026

closes #95

This is is problematic because /mnt is the wsl2 bridge into the windows file system, which is shared via the 9p protocol.

This is really slow and can take multiple minutes for the thousands of windows binaries that are potentially found there.

Even though it would be correct to search for multicall binaries there, it is prohibitively expensive. So we don't for now.

This is is problematic because /mnt is the wsl2 bridge into the windows
file system, which is shared via the 9p protocol.

This is really slow and can take multiple minutes for the thousands of
windows binaries that are potentially found there.

Even though it would be correct to search for multicall binaries there,
it is prohibitively expensive. So we don't for now.
@dwt dwt requested a review from jy-tan as a code owner April 1, 2026 17:44
@dwt
Copy link
Copy Markdown
Collaborator Author

dwt commented Apr 1, 2026

Not quite sure this is the correct way to do it - feedback appreciated as always. Since my dev box is a Mac, I'm not sure yet how to reliably test on linux locally. Thus the code is a bit speculative. Ah well. Better try than not try.

Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 3 files

@dwt
Copy link
Copy Markdown
Collaborator Author

dwt commented Apr 2, 2026

closing for now in favor of #98

@dwt dwt closed this Apr 2, 2026
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.

Fence hangs after printing "Sandbox manager initialized" and never executes the sandboxed command (WSL2)

1 participant