Sometimes it is useful to be able to dump the efi boot-services code and
data. This commit adds these as debugfs-blobs to /sys/kernel/debug/efi,
but only if efi=debug is passed on the kernel-commandline as this requires
not freeing those memory-regions, which costs 20+ MB of RAM.

Signed-off-by: Hans de Goede <hdego...@redhat.com>
---
Changes in v2:
-Do not call pr_err on debugfs call failures
---
 arch/x86/platform/efi/quirks.c |  4 +++
 drivers/firmware/efi/efi.c     | 53 ++++++++++++++++++++++++++++++++++
 2 files changed, 57 insertions(+)

diff --git a/arch/x86/platform/efi/quirks.c b/arch/x86/platform/efi/quirks.c
index 5b513ccffde4..0f968c7bcfec 100644
--- a/arch/x86/platform/efi/quirks.c
+++ b/arch/x86/platform/efi/quirks.c
@@ -374,6 +374,10 @@ void __init efi_free_boot_services(void)
        int num_entries = 0;
        void *new, *new_md;
 
+       /* Keep all regions for /sys/kernel/debug/efi */
+       if (efi_enabled(EFI_DBG))
+               return;
+
        for_each_efi_memory_desc(md) {
                unsigned long long start = md->phys_addr;
                unsigned long long size = md->num_pages << EFI_PAGE_SHIFT;
diff --git a/drivers/firmware/efi/efi.c b/drivers/firmware/efi/efi.c
index cd42f66a7c85..10c896e8b82b 100644
--- a/drivers/firmware/efi/efi.c
+++ b/drivers/firmware/efi/efi.c
@@ -18,6 +18,7 @@
 #include <linux/kobject.h>
 #include <linux/module.h>
 #include <linux/init.h>
+#include <linux/debugfs.h>
 #include <linux/device.h>
 #include <linux/efi.h>
 #include <linux/of.h>
@@ -316,6 +317,55 @@ static __init int efivar_ssdt_load(void)
 static inline int efivar_ssdt_load(void) { return 0; }
 #endif
 
+#ifdef CONFIG_DEBUG_FS
+
+#define EFI_DEBUGFS_MAX_BLOBS 32
+
+static struct debugfs_blob_wrapper debugfs_blob[EFI_DEBUGFS_MAX_BLOBS];
+
+static void __init efi_debugfs_init(void)
+{
+       struct dentry *efi_debugfs;
+       efi_memory_desc_t *md;
+       char name[32];
+       int type_count[EFI_BOOT_SERVICES_DATA + 1] = {};
+       int i = 0;
+
+       efi_debugfs = debugfs_create_dir("efi", NULL);
+       if (IS_ERR_OR_NULL(efi_debugfs))
+               return;
+
+       for_each_efi_memory_desc(md) {
+               switch (md->type) {
+               case EFI_BOOT_SERVICES_CODE:
+                       snprintf(name, sizeof(name), "boot_services_code%d",
+                                type_count[md->type]++);
+                       break;
+               case EFI_BOOT_SERVICES_DATA:
+                       snprintf(name, sizeof(name), "boot_services_data%d",
+                                type_count[md->type]++);
+                       break;
+               default:
+                       continue;
+               }
+
+               debugfs_blob[i].size = md->num_pages << EFI_PAGE_SHIFT;
+               debugfs_blob[i].data = memremap(md->phys_addr,
+                                               debugfs_blob[i].size,
+                                               MEMREMAP_WB);
+               if (!debugfs_blob[i].data)
+                       continue;
+
+               debugfs_create_blob(name, 0400, efi_debugfs, &debugfs_blob[i]);
+               i++;
+               if (i == EFI_DEBUGFS_MAX_BLOBS)
+                       break;
+       }
+}
+#else
+static inline void efi_debugfs_init(void) {}
+#endif
+
 /*
  * We register the efi subsystem with the firmware subsystem and the
  * efivars subsystem with the efi subsystem, if the system was booted with
@@ -360,6 +410,9 @@ static int __init efisubsys_init(void)
                goto err_remove_group;
        }
 
+       if (efi_enabled(EFI_DBG))
+               efi_debugfs_init();
+
        return 0;
 
 err_remove_group:
-- 
2.17.0

--
To unsubscribe from this list: send the line "unsubscribe linux-efi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to