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

Update the Secure Kernel Loader package support for QubesOS #19

Closed
6 of 7 tasks
BeataZdunczyk opened this issue Apr 3, 2023 · 5 comments
Closed
6 of 7 tasks
Assignees
Labels
P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. T: feature request Type: feature reguest. A new feature for the project. W: done Workflow: done. This issue is done/close.

Comments

@BeataZdunczyk
Copy link
Member

BeataZdunczyk commented Apr 3, 2023

Is your feature request related to a problem? Please describe.

Since the initial work done by 3mdeb engineers for AMD AEM in Qubes OS, the Secure Kernel Loader (formerly Landing Zone) package support has not been updated. The package has undergone significant improvements and added new features. SKL is an open-source module written by TrenchBoot developers required by AMD Secure Startup technology to perform DRTM launch. The task aims to refresh the previous work and update the SKL package for Qubes OS to the newest revision.

Is your feature request related to a new idea or technology that
would benefit the project? Please describe.

Updating the Secure Kernel Loader package support for QubesOS on AMD would benefit the project by providing a more up-to-date and reliable version of the package, which is essential for AMD Secure Startup technology to perform DRTM launch.

Describe the solution you'd like

Update the Secure Kernel Loader (SKL) package support for QubesOS on AMD to the newest revision.

Describe alternatives you've considered

N/A

Additional context

This feature request is part of Phase 4 in TrenchBoot as Anti Evil Maid project, as outlined in the documentation: https://docs.dasharo.com/projects/trenchboot-aem-v2/.

Checklist:

Relevant documentation you've consulted

N/A

@BeataZdunczyk BeataZdunczyk added T: feature request Type: feature reguest. A new feature for the project. P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. W: todo Workflow: todo. The issue is in the initial to do state. labels Apr 3, 2023
@macpijan
Copy link
Member

macpijan commented Nov 20, 2023

Since the initial work done by 3mdeb engineers for AMD AEM in Qubes OS

I'm not really certain, if we had done some support for QubesOS AMD AEM before?
As far as I remember, the https://github.com/trenchboot/landing-zone has not been packaged for QubesOS before, @miczyg1 @krystian-hebel ?

I cannot find such traces in landing-zone repo inn either trenchboot, nor 3mdeb repositories.

So the goal of this task would be to package the https://github.com/TrenchBoot/secure-kernel-loader (replacement for Landing Zone) for QubesOS. Including adding github CI ensuring package can still be built, and artifact can be stored for easier deployment.

The operational state of the SKL repo is really unknown I guess, especially in terms of changes in other repositories happening in parallel. I would say that the goal would be here to have something that builds reliably, so we can work on that in the next tasks.

@macpijan
Copy link
Member

macpijan commented Nov 20, 2023

There are some workflows already: https://github.com/TrenchBoot/secure-kernel-loader/tree/master/.github/workflows
But I do not see them being built on the latest commits. This needs to be checked/updated/improved.

There were also some discussion with @andyhhp in TrenchBoot/secure-kernel-loader#12 we might want to go back to that here to decide on pursuing that further, or dropping.

@macpijan
Copy link
Member

@SergiiDmytruk I have updated checklist in the first comment. Feel free to adjust/extend it if necessary, or discuss here.

@SergiiDmytruk
Copy link
Member

@krystian-hebel
Copy link

krystian-hebel commented Jan 18, 2024

I've approved PR for building RPMs (TrenchBoot/secure-kernel-loader#14), but we still need a second approval to merge it. @andyhhp @dpsmith @rossphilipson care to take a look?

@BeataZdunczyk BeataZdunczyk added W: done Workflow: done. This issue is done/close. and removed W: todo Workflow: todo. The issue is in the initial to do state. labels Feb 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. T: feature request Type: feature reguest. A new feature for the project. W: done Workflow: done. This issue is done/close.
Projects
None yet
Development

No branches or pull requests

4 participants