Skip to content

Comments

Add a simple unikernel which checks the fact that we can link with switch-installed libraries (and run it)#159

Closed
dinosaure wants to merge 5 commits intomainfrom
compile-and-run-unikernel
Closed

Add a simple unikernel which checks the fact that we can link with switch-installed libraries (and run it)#159
dinosaure wants to merge 5 commits intomainfrom
compile-and-run-unikernel

Conversation

@dinosaure
Copy link
Member

As asked here: #155 (comment)

@dinosaure dinosaure force-pushed the compile-and-run-unikernel branch 7 times, most recently from 7e1a5d2 to 4b41ea4 Compare November 24, 2025 18:29
@dinosaure dinosaure force-pushed the compile-and-run-unikernel branch from 722fff1 to 474addb Compare November 24, 2025 19:05
@hannesm
Copy link
Member

hannesm commented Nov 25, 2025

looks good to me. thanks.

@hannesm
Copy link
Member

hannesm commented Nov 25, 2025

the only question I have is what the difference between "example" (directory in this repo) and "unikernel" is, and whether it is worth to describe (in README)? Is example still used?

Copy link
Member

@reynir reynir left a comment

Choose a reason for hiding this comment

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

I pushed a commit that adds a small README in the unikernel/ directory. When I come back to this repository in the future I would not be able to figure out why there is such a trivial example in unikernel/, and now it's documented in the README.

@hannesm
Copy link
Member

hannesm commented Nov 25, 2025

And example is used as well in the CI actions...

@dinosaure
Copy link
Member Author

I was not able to put the unikernel into the example directory due to dune which requires another manifest.c and another startup.c. I can try to merge the compilation of spt and xen into this example but I find the example/ quite complex in some way.

@reynir
Copy link
Member

reynir commented Nov 25, 2025

Would it be worthwhile to add a README in example/? Especially why we use fmt and not Printf.printf.

@dinosaure
Copy link
Member Author

Replaced by 4fdf10e which is green and test what we want (without duplicate of unikernels and use the example/ directory).

@dinosaure dinosaure closed this Nov 25, 2025
@hannesm hannesm deleted the compile-and-run-unikernel branch November 25, 2025 12:51
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.

3 participants