Skip to content

vm: expose child processes through procfs - #47

Open
zbarsky-openai wants to merge 1 commit into
hermeticbuild:mainfrom
zbarsky-openai:zbarsky/kernel-proc-children
Open

zbarsky-openai wants to merge 1 commit into
hermeticbuild:mainfrom
zbarsky-openai:zbarsky/kernel-proc-children

Conversation

@zbarsky-openai

Copy link
Copy Markdown

Subprocess supervisors fail inside actiond guests because the compact
kernel omits the procfs child-process list:

RuntimeError: cannot inspect adopted subprocesses through
/proc/thread-self/children

SupervisedProcessCleanupError: subprocess supervisor did not confirm
descendant cleanup

Linux exposes /proc/<pid>/task/<tid>/children only when
CONFIG_PROC_CHILDREN is enabled. The option defaults to disabled with
allnoconfig, even when procfs itself is available.

Enable CONFIG_PROC_CHILDREN in both ARM64 and x86_64 guest kernels.
The rebuilt ARM64 kernel exposes actual child PIDs, all three previously
failing subprocess-supervisor cases pass inside the guest, and the full
downstream suite completes with 898 passing tests and one existing skip.

Subprocess supervisors fail inside actiond guests because the compact
kernel omits the procfs child-process list:

```text
RuntimeError: cannot inspect adopted subprocesses through
/proc/thread-self/children

SupervisedProcessCleanupError: subprocess supervisor did not confirm
descendant cleanup
```

Linux exposes `/proc/<pid>/task/<tid>/children` only when
`CONFIG_PROC_CHILDREN` is enabled. The option defaults to disabled with
`allnoconfig`, even when procfs itself is available.

Enable `CONFIG_PROC_CHILDREN` in both ARM64 and x86_64 guest kernels.
The rebuilt ARM64 kernel exposes actual child PIDs, all three previously
failing subprocess-supervisor cases pass inside the guest, and the full
downstream suite completes with 898 passing tests and one existing skip.
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