-
Notifications
You must be signed in to change notification settings - Fork 7
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
meta-dts-distro/recipes-bsp/txe-secure-boot: Add smmstore recipe
Signed-off-by: Pawel Langowski <pawel.langowski@3mdeb.com>
- Loading branch information
1 parent
8f766cd
commit f1ef77f
Showing
2 changed files
with
62 additions
and
0 deletions.
There are no files selected for viewing
41 changes: 41 additions & 0 deletions
41
meta-dts-distro/recipes-bsp/txe-secure-boot/files/a5df001.diff
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,41 @@ | ||
From a5df001f258be5536b7ad9f234546330d5f3ca66 Mon Sep 17 00:00:00 2001 | ||
From: Maciej Pijanowski <maciej.pijanowski@3mdeb.com> | ||
Date: Wed, 29 May 2024 13:28:14 +0200 | ||
Subject: [PATCH] util/smmstoretool: explain what happens when no store is found | ||
|
||
We are going to expose ths tool to end users, and want to take | ||
care that the presented information can be consumed by them. | ||
|
||
The current code simply prints below warnings if we use release | ||
binary available for end-user to download: | ||
|
||
No firmware volume header present | ||
No valid firmware volume was found | ||
|
||
It will be concerning and not clear to end users, they might not | ||
understant why it happens, what are the implications, and whether | ||
it is something that they should worry about. | ||
|
||
This commit tries to explain what actually happens here. | ||
|
||
Change-Id: Iaa2678f5ae7c243811484c0567ced97ae0b3fc0a | ||
Signed-off-by: Maciej Pijanowski <maciej.pijanowski@3mdeb.com> | ||
--- | ||
|
||
diff --git a/util/smmstoretool/storage.c b/util/smmstoretool/storage.c | ||
index 8a02384..87284f7 100644 | ||
--- a/util/smmstoretool/storage.c | ||
+++ b/util/smmstoretool/storage.c | ||
@@ -50,6 +50,12 @@ | ||
goto error; | ||
} | ||
|
||
+ fprintf(stderr, | ||
+ "\nThe variable store has not been found in the ROM image\n" | ||
+ "and is about to be initialized. This situation is normal\n" | ||
+ "for a release image, as the variable store is usually\n" | ||
+ "initialized on the first boot of the platform.\n\n"); | ||
+ | ||
if (!fv_init(storage->region)) { | ||
fprintf(stderr, | ||
"Failed to create variable store in \"%s\"\n", |
21 changes: 21 additions & 0 deletions
21
meta-dts-distro/recipes-bsp/txe-secure-boot/smmstore_git.bb
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,21 @@ | ||
DESCRIPTION = "Utility for generating TXE Secure Boot manifests" | ||
LICENSE = "GPL-2.0-or-later" | ||
|
||
SRC_URI = " \ | ||
git://github.com/coreboot/coreboot.git;branch=main;protocol=https \ | ||
file://a5df001.diff \ | ||
" | ||
|
||
PV = "1.0+git${SRCPV}" | ||
SRCREV = "${AUTOREV}" | ||
|
||
LIC_FILES_CHKSUM = " \ | ||
file://${WORKDIR}/git/LICENSES/GPL-2.0-or-later.txt;md5=261bea1168c0bdfa73232ee90df11eb6 \ | ||
" | ||
|
||
S = "${WORKDIR}/git" | ||
|
||
# Change working directory after patching | ||
do_patch:append() { | ||
d.setVar("S", "${WORKDIR}/git/util/txe_sb_tool") | ||
} |