Re: [PATCH v8 5/5] efi: selftest: add selftest for EFI_DEBUG_SUPPORT

[Heinrich Schuchardt]

On 03.07.25 08:28, Ying-Chun Liu (PaulLiu) wrote:

From: "Ying-Chun Liu (PaulLiu)" <paul....@linaro.org>

Add selftest to check the installed configuration table that has
the correct GUID.

Signed-off-by: Ying-Chun Liu (PaulLiu) <paul....@linaro.org>
Cc: Heinrich Schuchardt <xypron.g...@gmx.de>
Cc: Ilias Apalodimas <ilias.apalodi...@linaro.org>

Reviewed-by: Heinrich Schuchardt <xypron.g...@gmx.de>

---
V7: add selftest for EFI_DEBUG_SUPPORT
V8: refine the selftest.
---
  lib/efi_selftest/Makefile                     |  2 +
  lib/efi_selftest/efi_selftest_debug_support.c | 40 +++++++++++++++++++
  2 files changed, 42 insertions(+)
  create mode 100644 lib/efi_selftest/efi_selftest_debug_support.c

diff --git a/lib/efi_selftest/Makefile b/lib/efi_selftest/Makefile
index d78bf7d6191..842433f68aa 100644
--- a/lib/efi_selftest/Makefile
+++ b/lib/efi_selftest/Makefile
@@ -78,6 +78,8 @@ endif

obj-$(CONFIG_EFI_ESRT) += efi_selftest_esrt.o +obj-$(CONFIG_EFI_DEBUG_SUPPORT) += efi_selftest_debug_support.o

+
  targets += \
  efi_miniapp_file_image_exception.h \
  efi_miniapp_file_image_exit.h \
diff --git a/lib/efi_selftest/efi_selftest_debug_support.c 
b/lib/efi_selftest/efi_selftest_debug_support.c
new file mode 100644
index 00000000000..9ca8b3f82f5
--- /dev/null
+++ b/lib/efi_selftest/efi_selftest_debug_support.c
@@ -0,0 +1,40 @@
+// SPDX-License-Identifier: GPL-2.0-or-later
+/*
+ * efi_selftest_debug_support
+ *
+ * Copyright (c) 2025 Ying-Chun Liu, Linaro Ltd. <paul....@linaro.org>
+ *
+ * Test the EFI_DEBUG_SUPPORT
+ */
+
+#include <efi_loader.h>
+#include <efi_selftest.h>
+
+/**
+ * efi_st_debug_support_execute() - execute test
+ *
+ * Test EFI_DEBUG_SUPPORT tables.
+ *
+ * Return:     status code
+ */
+static int efi_st_debug_support_execute(void)
+{
+       struct efi_debug_image_info_table_header 
*efi_st_debug_info_table_header = NULL;
+       efi_guid_t efi_debug_image_info_table_guid = 
EFI_DEBUG_IMAGE_INFO_TABLE_GUID;
+
+       /* get EFI_DEBUG_IMAGE_INFO_TABLE */
+       efi_st_debug_info_table_header = 
efi_st_get_config_table(&efi_debug_image_info_table_guid);
+
+       if (!efi_st_debug_info_table_header) {
+               efi_st_error("Missing EFI_DEBUG_IMAGE_INFO_TABLE\n");
+               return EFI_ST_FAILURE;
+       }
+
+       return EFI_ST_SUCCESS;
+}
+
+EFI_UNIT_TEST(debug_support) = {
+       .name = "debug_support",
+       .phase = EFI_EXECUTE_BEFORE_BOOTTIME_EXIT,
+       .execute = efi_st_debug_support_execute,
+};