Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add new version of the restricted kernel simple_io and initrd #4906

Merged

Conversation

jul-sh
Copy link
Contributor

@jul-sh jul-sh commented Mar 14, 2024

the new oak_restricted_kernel_simple_io_init_rd_wrapper will exist alongside oak_restricted_kernel_simple_io_wrapper for a while. then internal dependencies will be updated, and the the former will be removed.

this approach is significantly simpler than #4828. For #4828 we would need to attempt to synchronize its merging with the updates to the internal launcher and the transparent release imports of the binary. It'd be too flaky.

the new oak_restricted_kernel_simple_io_init_rd_wrapper will exist alongside oak_restricted_kernel_simple_io_wrapper for a while. then internal dependencies will be updated, and the the former will be removed.

this approach is significatnly simpler than #. For project-oak#4828 we would need to attempt to synchronize its merging with the updates to the internal launcher and the transparent release imports of the binary. It'd be too flaky.
@@ -42,6 +43,7 @@ readonly generated_binaries=(
)
readonly binary_names=(
oak_restricted_kernel_simple_io_wrapper_bin
oak_restricted_kernel_simple_io_init_rd_wrapper_bin
Copy link
Collaborator

Choose a reason for hiding this comment

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

I assume the new one would also behave like a bzImage (i.e. a kernel-type endorsement) since it is named "wrapper".

Copy link
Collaborator

Choose a reason for hiding this comment

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

Yes

@@ -42,6 +43,7 @@ readonly generated_binaries=(
)
readonly binary_names=(
oak_restricted_kernel_simple_io_wrapper_bin
oak_restricted_kernel_simple_io_init_rd_wrapper_bin
Copy link
Collaborator

Choose a reason for hiding this comment

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

Yes

@jul-sh jul-sh merged commit 6f8a21f into project-oak:main Mar 15, 2024
24 checks passed
@jul-sh jul-sh deleted the oak_restricted_kernel_simple_io_initrd_bin branch March 15, 2024 09:22
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