Skip to content

test: unikraft and app-elfloader bump - #104

Closed
danbugs wants to merge 6 commits into
mainfrom
test/unikraft-elfloader-bump
Closed

test: unikraft and app-elfloader bump#104
danbugs wants to merge 6 commits into
mainfrom
test/unikraft-elfloader-bump

Conversation

@danbugs

@danbugs danbugs commented Jun 28, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings June 28, 2026 03:56

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Kraft example configurations to use newer Unikraft and app-elfloader revisions intended to support Hyperlight (notably around vfork/execve-related behavior).

Changes:

  • Swaps Unikraft source/version in multiple examples/*/kraft.yaml files from unikraft/unikraft@plat-hyperlight to danbugs/unikraft@feat/vfork-stack-save.
  • Swaps app-elfloader source/version from unikraft/app-elfloader@plat-hyperlight to danbugs/app-elfloader@fix/vfork-execve.

Reviewed changes

Copilot reviewed 18 out of 18 changed files in this pull request and generated 18 comments.

Show a summary per file
File Description
examples/shell/kraft.yaml Updates Unikraft + app-elfloader refs for the shell example.
examples/rust/kraft.yaml Updates Unikraft + app-elfloader refs for the Rust example.
examples/python/kraft.yaml Updates Unikraft + app-elfloader refs for the Python example.
examples/python-tools/kraft.yaml Updates Unikraft + app-elfloader refs for the Python tools example.
examples/python-agent/kraft.yaml Updates Unikraft + app-elfloader refs for the Python agent example.
examples/python-agent-driver/kraft.yaml Updates Unikraft + app-elfloader refs for the Python agent driver example.
examples/powershell/kraft.yaml Updates Unikraft + app-elfloader refs for the PowerShell example.
examples/nodejs/kraft.yaml Updates Unikraft + app-elfloader refs for the Node.js example.
examples/networking-py/kraft.yaml Updates Unikraft + app-elfloader refs for the Python networking example.
examples/multifn-c/kraft.yaml Updates Unikraft + app-elfloader refs for the multi-function C example.
examples/hostfs-posix-py/kraft.yaml Updates Unikraft + app-elfloader refs for the hostfs POSIX Python example.
examples/hostfs-posix-c/kraft.yaml Updates Unikraft + app-elfloader refs for the hostfs POSIX C example.
examples/helloworld-c/kraft.yaml Updates Unikraft + app-elfloader refs for the hello world C example.
examples/go/kraft.yaml Updates Unikraft + app-elfloader refs for the Go example.
examples/go-http/kraft.yaml Updates Unikraft + app-elfloader refs for the Go HTTP example.
examples/dotnet/kraft.yaml Updates Unikraft + app-elfloader refs for the .NET example.
examples/dotnet-nativeaot/kraft.yaml Updates Unikraft + app-elfloader refs for the .NET NativeAOT example.
examples/dotnet-http/kraft.yaml Updates Unikraft + app-elfloader refs for the .NET HTTP example.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread examples/shell/kraft.yaml Outdated
Comment thread examples/rust/kraft.yaml Outdated
Comment thread examples/python/kraft.yaml Outdated
Comment thread examples/python-tools/kraft.yaml Outdated
Comment thread examples/python-agent/kraft.yaml Outdated
Comment thread examples/go/kraft.yaml Outdated
Comment thread examples/go-http/kraft.yaml Outdated
Comment thread examples/dotnet/kraft.yaml Outdated
Comment thread examples/dotnet-nativeaot/kraft.yaml Outdated
Comment thread examples/dotnet-http/kraft.yaml Outdated
@danbugs
danbugs force-pushed the test/unikraft-elfloader-bump branch 4 times, most recently from 7f1bc77 to 642f609 Compare June 28, 2026 06:32

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Linux Benchmarks

Details
Benchmark suite Current: 310f4dc Previous: 682ad20 Ratio
hello_world (median) 42 ms 20 ms 2.10
pandas (median) -1 ms 120 ms -0.008333333333333333
density (per VM) 7 MB 8 MB 0.88
snapshot (disk) 653 MiB 653 MiB 1

This comment was automatically generated by workflow using github-action-benchmark.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Windows Benchmarks

Details
Benchmark suite Current: 310f4dc Previous: 682ad20 Ratio
hello_world (median) 241 ms 291 ms 0.83
pandas (median) 725 ms 1024 ms 0.71
density (per VM) 7 MB 7 MB 1
snapshot (disk) 661 MiB 661 MiB 1

This comment was automatically generated by workflow using github-action-benchmark.

@danbugs
danbugs force-pushed the test/unikraft-elfloader-bump branch 2 times, most recently from 53d4fdf to 22a8f8e Compare June 29, 2026 17:14
danbugs added 6 commits June 30, 2026 06:28
Signed-off-by: danbugs <danilochiarlone@gmail.com>
Signed-off-by: danbugs <danilochiarlone@gmail.com>
Signed-off-by: danbugs <danilochiarlone@gmail.com>
Signed-off-by: danbugs <danilochiarlone@gmail.com>
After snapshot restore, the first subprocess.run with capture_output
crashes on an unresolvable CoW page fault. Prepending two no-op
subprocess.run(['true']) calls in the same PyRun_String resolves the
needed CoW pages before the piped subprocess runs.

Signed-off-by: danbugs <danilochiarlone@gmail.com>
Signed-off-by: danbugs <danilochiarlone@gmail.com>
@danbugs
danbugs force-pushed the test/unikraft-elfloader-bump branch from a5754e6 to 9a3f5a9 Compare June 30, 2026 06:28
@danbugs danbugs closed this Jul 3, 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.

2 participants