[coreboot] r3744 - trunk/payloads/coreinfo

svn at coreboot.org svn at coreboot.org
Tue Nov 11 20:53:42 CET 2008


Author: jcrouse
Date: 2008-11-11 20:53:42 +0100 (Tue, 11 Nov 2008)
New Revision: 3744

Added:
   trunk/payloads/coreinfo/multiboot_module.c
Modified:
   trunk/payloads/coreinfo/Kconfig
   trunk/payloads/coreinfo/Makefile
   trunk/payloads/coreinfo/coreinfo.c
Log:
[PATCH] coreinfo:  Add multiboot parsing support

Rename the "coreboot" menu "firmware", and add a module to parse
the multiboot table.  For now, just parse memory, but it can be
expanded as needed.

Signed-off-by: Jordan Crouse <jordan at cosmicpenguin.net>
Acked-by: Peter Stuge <peter at stuge.se>


Modified: trunk/payloads/coreinfo/Kconfig
===================================================================
--- trunk/payloads/coreinfo/Kconfig	2008-11-11 19:51:14 UTC (rev 3743)
+++ trunk/payloads/coreinfo/Kconfig	2008-11-11 19:53:42 UTC (rev 3744)
@@ -70,6 +70,12 @@
 	help
 	  This option will increase the ELF file size by ca. 2250 bytes.
 
+config MODULE_MULTIBOOT
+	bool "Enable the multiboot module"
+	default y
+	help
+	  This option will increase the ELF file size by ca. 1792 bytes.
+
 config MODULE_CPUINFO
 	bool "Enable the cpuinfo module"
 	default y

Modified: trunk/payloads/coreinfo/Makefile
===================================================================
--- trunk/payloads/coreinfo/Makefile	2008-11-11 19:51:14 UTC (rev 3743)
+++ trunk/payloads/coreinfo/Makefile	2008-11-11 19:53:42 UTC (rev 3744)
@@ -53,7 +53,7 @@
 CFLAGS := -Wall -Werror -Os $(INCLUDES)
 OBJECTS = cpuinfo_module.o cpuid.S.o pci_module.o coreboot_module.o \
 	  nvram_module.o bootlog_module.o ramdump_module.o lar_module.o \
-	  coreinfo.o
+	  multiboot_module.o coreinfo.o
 OBJS    = $(patsubst %,$(obj)/%,$(OBJECTS))
 TARGET  = $(obj)/coreinfo.elf
 

Modified: trunk/payloads/coreinfo/coreinfo.c
===================================================================
--- trunk/payloads/coreinfo/coreinfo.c	2008-11-11 19:51:14 UTC (rev 3743)
+++ trunk/payloads/coreinfo/coreinfo.c	2008-11-11 19:53:42 UTC (rev 3744)
@@ -27,6 +27,7 @@
 extern struct coreinfo_module cpuinfo_module;
 extern struct coreinfo_module pci_module;
 extern struct coreinfo_module coreboot_module;
+extern struct coreinfo_module multiboot_module;
 extern struct coreinfo_module nvram_module;
 extern struct coreinfo_module bootlog_module;
 extern struct coreinfo_module ramdump_module;
@@ -47,10 +48,13 @@
 #endif
 };
 
-struct coreinfo_module *coreboot_modules[] = {
+struct coreinfo_module *firmware_modules[] = {
 #ifdef CONFIG_MODULE_COREBOOT
 	&coreboot_module,
 #endif
+#ifdef CONFIG_MODULE_MULTIBOOT
+	&multiboot_module,
+#endif
 #ifdef CONFIG_MODULE_BOOTLOG
 	&bootlog_module,
 #endif
@@ -71,9 +75,9 @@
 		.count = ARRAY_SIZE(system_modules),
 	},
 	{
-		.name = "Coreboot",
-		.modules = coreboot_modules,
-		.count = ARRAY_SIZE(coreboot_modules),
+		.name = "Firmware",
+		.modules = firmware_modules,
+		.count = ARRAY_SIZE(firmware_modules),
 	}
 };
 

Added: trunk/payloads/coreinfo/multiboot_module.c
===================================================================
--- trunk/payloads/coreinfo/multiboot_module.c	                        (rev 0)
+++ trunk/payloads/coreinfo/multiboot_module.c	2008-11-11 19:53:42 UTC (rev 3744)
@@ -0,0 +1,121 @@
+/*
+ * This file is part of the coreinfo project.
+ *
+ * Copyright (C) 2008 Jordan Crouse <jordan at cosmicpenguin.net>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 2 of the License.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301 USA
+ */
+
+#include <multiboot_tables.h>
+#include "coreinfo.h"
+
+#ifdef CONFIG_MODULE_MULTIBOOT
+
+#define MAX_MEMORY_COUNT  10
+
+static struct {
+	int mem_count;
+
+	struct {
+		u64 start;
+		u64 size;
+		int type;
+	} range[MAX_MEMORY_COUNT];
+} cb_info;
+
+static int tables_good = 0;
+
+int multiboot_module_redraw(WINDOW *win)
+{
+	int row = 2;
+	int i;
+
+	print_module_title(win, "Multiboot Tables");
+
+	if (tables_good == 0) {
+		mvwprintw(win, row++, 1, "No multiboot tables were found");
+		return 0;
+	}
+
+	row++;
+	mvwprintw(win, row++, 1, "-- Memory Map --");
+
+	for (i = 0; i < cb_info.mem_count; i++) {
+
+		if (cb_info.range[i].type == 1)
+			mvwprintw(win, row++, 3, "     RAM: ");
+		else
+			mvwprintw(win, row++, 3, "Reserved: ");
+
+		wprintw(win, "%16.16llx - %16.16llx",
+			cb_info.range[i].start,
+			cb_info.range[i].start + cb_info.range[i].size - 1);
+	}
+
+	return 0;
+}
+
+static void parse_memory(struct multiboot_header *table)
+{
+        u8 *start = (u8 *) phys_to_virt(table->mmap_addr);
+        u8 *ptr = start;
+	int i = 0;
+
+	cb_info.mem_count = 0;
+
+        while(ptr < (start + table->mmap_length)) {
+                struct multiboot_mmap *mmap = (struct multiboot_mmap *) ptr;
+
+		cb_info.range[i].start = mmap->addr;
+		cb_info.range[i].size = mmap->length;
+		cb_info.range[i].type = mmap->type;
+
+		if (++cb_info.mem_count == MAX_MEMORY_COUNT)
+			return;
+
+                ptr += (mmap->size + sizeof(mmap->size));
+		i++;
+        }
+}
+
+static void parse_header(unsigned long addr)
+{
+	struct multiboot_header *table = (struct multiboot_header *) addr;
+
+	if (table->flags & MULTIBOOT_FLAGS_MMAP)
+		parse_memory(table);
+}
+
+static int multiboot_module_init(void)
+{
+	unsigned long mbaddr;
+	tables_good = sysinfo_have_multiboot(&mbaddr);
+
+	parse_header(mbaddr);
+
+	return tables_good ? 0 : -1;
+}
+
+struct coreinfo_module multiboot_module = {
+	.name = "Multiboot",
+	.init = multiboot_module_init,
+	.redraw = multiboot_module_redraw,
+};
+
+#else
+
+struct coreinfo_module multiboot_module = {
+};
+
+#endif





More information about the coreboot mailing list