Skip to content

[port #1062] remove Linux runtime VMA labels in tiny mode - #12

Open
luantak wants to merge 2 commits into
masterfrom
cursor/maremagnum-pr-1062-tiny-linux-vma-labels-2676
Open

[port #1062] remove Linux runtime VMA labels in tiny mode#12
luantak wants to merge 2 commits into
masterfrom
cursor/maremagnum-pr-1062-tiny-linux-vma-labels-2676

Conversation

@luantak

@luantak luantak commented Aug 11, 2026

Copy link
Copy Markdown
Owner

Validity: valid

Linux runtime names anonymous mappings (Go: heap arena etc.) visible in /proc/PID/maps. Disabling setVMAName for tiny Linux builds removes an OS-visible runtime trace without changing mapping behavior.

Upstream

burrowers#1062 (was dirty vs master)

Commit layout

Commit 1 is Maremagnum's from burrowers#1062 (author preserved as on the upstream commit: autoprofiler <autoprofiler@local.invalid>); following commit(s) address luantak/mvdan review feedback and are authored by the Cursor/luantak port side.

  • e26e4a0 autoprofiler | remove Linux runtime VMA labels in tiny mode
  • 23fda5e Cursor Agent | Address review feedback: named VMA return, no map, binsubstr Go:

What this ports

  • Empty setVMAName in set_vma_name_linux.go under tiny
  • Fail-closed check that the strip matched on Linux
  • Cross-build script tiny-vma.txtar (GOOS=linux)

Feedback applied (luantak requested changes)

  1. Named return strippedVMAName so it does not look like a general success condition (still returns master's strippedFunctions map for other direct-strip validation).
  2. No nested ifs for the VMA check — panic immediately when processing set_vma_name_linux.go on Linux.
  3. No separate VMA map — use the named bool at the call site.
  4. Copilot: binsubstr matches Go: not Go:.

Testing

  • go test -run 'TestScript/(tiny|tiny-vma)$' .

Remaining risks

  • Low-medium. Narrow Linux-only change; validate that Go still ships set_vma_name_linux.go with that symbol name.

Notes for Paul

Clean, small; good companion to other tiny hardening ports.

Open in Web Open in Cursor 

autoprofiler and others added 2 commits August 11, 2026 21:04
Co-authored-by: Paul <luantak@pm.me>
luantak: use a named return for strippedVMAName; panic immediately
without a VMA map / nested ifs. Copilot: binsubstr matches "Go:" not
" Go: ". Adapted to master's stripRuntime map return.

Co-authored-by: Paul <luantak@pm.me>
@cursor
cursor Bot force-pushed the cursor/maremagnum-pr-1062-tiny-linux-vma-labels-2676 branch from 07cf595 to 23fda5e Compare August 11, 2026 21:05
@luantak
luantak marked this pull request as ready for review August 12, 2026 00:28
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.

2 participants