[coreboot-gerrit] Patch set updated for coreboot: 7b39409 Multiboot: remove multiboot tables generation.

Vladimir Serbinenko (phcoder@gmail.com) gerrit at coreboot.org
Mon Jan 6 20:57:27 CET 2014


Vladimir Serbinenko (phcoder at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/4628

-gerrit

commit 7b394098324f1d2b9f6988ac76fffed32423ae1f
Author: Vladimir Serbinenko <phcoder at gmail.com>
Date:   Mon Jan 6 20:40:27 2014 +0100

    Multiboot: remove multiboot tables generation.
    
    GRUB2-as-payload doesn't use them. Libpayload can live with just coreboot tables
    if loaded as payload. memtest86+ can use them but is buggy with them. Solaris
    needs a huge boot archive not supported by coreboot and too big to fit in
    flash (dozens of megabytes). All-in-all looks like no users are left for this.
    
    Change-Id: Id92f73be5a397db80f5b0132ee57c37ee6eeb563
    Signed-off-by: Vladimir Serbinenko <phcoder at gmail.com>
---
 src/Kconfig                     |   5 --
 src/arch/x86/boot/Makefile.inc  |   1 -
 src/arch/x86/boot/boot.c        |   8 --
 src/arch/x86/boot/multiboot.c   |  76 -----------------
 src/arch/x86/boot/tables.c      |  10 +--
 src/include/cpu/x86/multiboot.h | 184 ----------------------------------------
 6 files changed, 1 insertion(+), 283 deletions(-)

diff --git a/src/Kconfig b/src/Kconfig
index 88df9ae..d427e15 100644
--- a/src/Kconfig
+++ b/src/Kconfig
@@ -469,11 +469,6 @@ config MAX_PIRQ_LINKS
 
 menu "System tables"
 
-config MULTIBOOT
-	bool "Generate Multiboot tables (for GRUB2)"
-	default y
-	depends on ARCH_X86
-
 config GENERATE_ACPI_TABLES
 	prompt "Generate ACPI tables" if HAVE_ACPI_TABLES
 	bool
diff --git a/src/arch/x86/boot/Makefile.inc b/src/arch/x86/boot/Makefile.inc
index 3e53d79..629c644 100644
--- a/src/arch/x86/boot/Makefile.inc
+++ b/src/arch/x86/boot/Makefile.inc
@@ -2,7 +2,6 @@ romstage-$(CONFIG_EARLY_CBMEM_INIT) += cbmem.c
 romstage-$(CONFIG_HAVE_ACPI_RESUME) += cbmem.c
 
 ramstage-y += boot.c
-ramstage-$(CONFIG_MULTIBOOT) += multiboot.c
 ramstage-y += gdt.c
 ramstage-y += tables.c
 ramstage-y += cbmem.c
diff --git a/src/arch/x86/boot/boot.c b/src/arch/x86/boot/boot.c
index 1b28a4c..3ef46a5 100644
--- a/src/arch/x86/boot/boot.c
+++ b/src/arch/x86/boot/boot.c
@@ -2,7 +2,6 @@
 #include <arch/stages.h>
 #include <ip_checksum.h>
 #include <string.h>
-#include <cpu/x86/multiboot.h>
 
 #if CONFIG_RELOCATABLE_RAMSTAGE
 /* When the ramstage is relocatable the elf loading ensures an elf image cannot
@@ -21,9 +20,6 @@ void jmp_to_elf_entry(void *entry, unsigned long unused1, unsigned long unused2)
 
 		::
 		"r" (entry)
-#if CONFIG_MULTIBOOT
-		, "b"(mbi), "a" (MB_MAGIC2)
-#endif
 		);
 }
 #else
@@ -123,11 +119,7 @@ void jmp_to_elf_entry(void *entry, unsigned long buffer, unsigned long size)
 		::
 		"ri" (lb_start), "ri" (buffer), "ri" (lb_size),
 		"ri" (entry),
-#if CONFIG_MULTIBOOT
-		"ri"(mbi), "ri" (MB_MAGIC2)
-#else
 		"ri"(0), "ri" (0)
-#endif
 		);
 }
 #endif /* CONFIG_RELOCATABLE_RAMSTAGE */
diff --git a/src/arch/x86/boot/multiboot.c b/src/arch/x86/boot/multiboot.c
deleted file mode 100644
index a043e60..0000000
--- a/src/arch/x86/boot/multiboot.c
+++ /dev/null
@@ -1,76 +0,0 @@
-/*
- * support for Multiboot payloads
- *
- * Copyright (C) 2008 Robert Millan
- *
- * 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; either version 2 of the License, or
- * (at your option) any later version.
- *
- * 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, see <http://www.gnu.org/licenses/>.
- *
- */
-
-#include <cpu/x86/multiboot.h>
-#include <string.h>
-#include <device/resource.h>
-#include <console/console.h>
-#include <boot/coreboot_tables.h>
-
-struct multiboot_info *mbi = NULL;
-
-unsigned long write_multiboot_info(unsigned long rom_table_end)
-{
-	static struct multiboot_mmap_entry *mb_mem;
-	struct lb_memory* coreboot_table;
-	int entries;
-	int i;
-
-	mbi = (struct multiboot_info *)rom_table_end;
-
-	memset(mbi, 0, sizeof(*mbi));
-	rom_table_end += sizeof(*mbi);
-
-	mbi->mmap_addr = (u32) rom_table_end;
-	mb_mem = (struct multiboot_mmap_entry *)rom_table_end;
-
-	/* copy regions from coreboot tables */
-	coreboot_table = get_lb_mem();
-	entries = (coreboot_table->size - sizeof(*coreboot_table))/sizeof(coreboot_table->map[0]);
-
-	if (coreboot_table == NULL || entries < 1) {
-	    printk(BIOS_INFO, "%s: Cannot find coreboot table.\n", __func__);
-	    return (unsigned long) mb_mem;
-	}
-
-	for (i = 0; i < entries; i++) {
-	  uint64_t entry_start = unpack_lb64(coreboot_table->map[i].start);
-	  uint64_t entry_size = unpack_lb64(coreboot_table->map[i].size);
-	  mb_mem->addr = entry_start;
-	  mb_mem->len = entry_size;
-	  switch (coreboot_table->map[i].type) {
-	    case LB_MEM_RAM:
-	      mb_mem->type = MULTIBOOT_MEMORY_AVAILABLE;
-	      break;
-	    default: // anything other than usable RAM
-	      mb_mem->type = MULTIBOOT_MEMORY_RESERVED;
-	      break;
-	  }
-	  mb_mem->size = sizeof(*mb_mem) - sizeof(mb_mem->size);
-	  mb_mem++;
-	}
-
-	mbi->mmap_length = ((u32) mb_mem) - mbi->mmap_addr;
-	mbi->flags |= MB_INFO_MEM_MAP;
-
-	printk(BIOS_INFO, "Multiboot Information structure has been written.\n");
-
-	return (unsigned long)mb_mem;
-}
diff --git a/src/arch/x86/boot/tables.c b/src/arch/x86/boot/tables.c
index eea9bf1..b420a7c 100644
--- a/src/arch/x86/boot/tables.c
+++ b/src/arch/x86/boot/tables.c
@@ -27,7 +27,6 @@
 #include <arch/smp/mpspec.h>
 #include <arch/acpi.h>
 #include <string.h>
-#include <cpu/x86/multiboot.h>
 #include <cbmem.h>
 #include <lib.h>
 #include <smbios.h>
@@ -58,7 +57,7 @@ struct lb_memory *write_tables(void)
 	rom_table_end =   0xf0000;
 
 	/* Start low addr at 0x500, so we don't run into conflicts with the BDA
-	 * in case our data structures grow beyond 0x400. Only multiboot, GDT
+	 * in case our data structures grow beyond 0x400. Only GDT
 	 * and the coreboot table use low_tables.
 	 */
 	low_table_start = 0;
@@ -247,13 +246,6 @@ struct lb_memory *write_tables(void)
 				     rom_table_start, rom_table_end);
 	}
 
-#if CONFIG_MULTIBOOT
-	post_code(0x9d);
-
-	/* The Multiboot information structure */
-	write_multiboot_info(rom_table_end);
-#endif
-
 	/* Print CBMEM sections */
 	cbmem_list();
 
diff --git a/src/include/cpu/x86/multiboot.h b/src/include/cpu/x86/multiboot.h
deleted file mode 100644
index 7bc25a9..0000000
--- a/src/include/cpu/x86/multiboot.h
+++ /dev/null
@@ -1,184 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2003 Free Software Foundation, Inc.
- * Copyright (C) 2008 Robert Millan
- *
- * 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; either version 2 of the License, or
- * (at your option) any later version.
- *
- * 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
- */
-
-/*
- * multiboot.h - multiboot header file.
- */
-
-#ifndef MULTIBOOT_H
-#define MULTIBOOT_H
-
-#include <stdint.h>
-
-/* How many bytes from the start of the file we search for the header.  */
-#define MB_SEARCH                 8192
-
-/* The magic field should contain this.  */
-#define MB_MAGIC                  0x1BADB002
-
-/* This should be in %eax.  */
-#define MB_MAGIC2                 0x2BADB002
-
-/* The bits in the required part of flags field we don't support.  */
-#define MB_UNSUPPORTED            0x0000fffc
-
-/* Alignment of multiboot modules.  */
-#define MB_MOD_ALIGN              0x00001000
-
-/*
- * Flags set in the 'flags' member of the multiboot header.
- */
-
-/* Align all boot modules on i386 page (4KB) boundaries.  */
-#define MB_PAGE_ALIGN		0x00000001
-
-/* Must pass memory information to OS.  */
-#define MB_MEMORY_INFO		0x00000002
-
-/* Must pass video information to OS.  */
-#define MB_VIDEO_MODE		0x00000004
-
-/* This flag indicates the use of the address fields in the header.  */
-#define MB_AOUT_KLUDGE		0x00010000
-
-/*
- *  Flags to be set in the 'flags' member of the multiboot info structure.
- */
-
-/* is there basic lower/upper memory information? */
-#define MB_INFO_MEMORY		0x00000001
-/* is there a boot device set? */
-#define MB_INFO_BOOTDEV		0x00000002
-/* is the command-line defined? */
-#define MB_INFO_CMDLINE		0x00000004
-/* are there modules to do something with? */
-#define MB_INFO_MODS		0x00000008
-
-/* These next two are mutually exclusive */
-
-/* is there a symbol table loaded? */
-#define MB_INFO_AOUT_SYMS		0x00000010
-/* is there an ELF section header table? */
-#define MB_INFO_ELF_SHDR		0x00000020
-
-/* is there a full memory map? */
-#define MB_INFO_MEM_MAP		0x00000040
-
-/* Is there drive info?  */
-#define MB_INFO_DRIVE_INFO		0x00000080
-
-/* Is there a config table?  */
-#define MB_INFO_CONFIG_TABLE	0x00000100
-
-/* Is there a boot loader name?  */
-#define MB_INFO_BOOT_LOADER_NAME	0x00000200
-
-/* Is there a APM table?  */
-#define MB_INFO_APM_TABLE		0x00000400
-
-/* Is there video information?  */
-#define MB_INFO_VIDEO_INFO		0x00000800
-
-struct multiboot_header {
-	/* Must be MB_MAGIC - see above.  */
-	uint32_t magic;
-
-	/* Feature flags.  */
-	uint32_t flags;
-
-	/* The above fields plus this one must equal 0 mod 2^32. */
-	uint32_t checksum;
-
-	/* These are only valid if MB_AOUT_KLUDGE is set.  */
-	uint32_t header_addr;
-	uint32_t load_addr;
-	uint32_t load_end_addr;
-	uint32_t bss_end_addr;
-	uint32_t entry_addr;
-
-	/* These are only valid if MB_VIDEO_MODE is set.  */
-	uint32_t mode_type;
-	uint32_t width;
-	uint32_t height;
-	uint32_t depth;
-};
-
-struct multiboot_info {
-	/* Multiboot info version number */
-	uint32_t flags;
-
-	/* Available memory from BIOS */
-	uint32_t mem_lower;
-	uint32_t mem_upper;
-
-	/* "root" partition */
-	uint32_t boot_device;
-
-	/* Kernel command line */
-	uint32_t cmdline;
-
-	/* Boot-Module list */
-	uint32_t mods_count;
-	uint32_t mods_addr;
-
-	uint32_t syms[4];
-
-	/* Memory Mapping buffer */
-	uint32_t mmap_length;
-	uint32_t mmap_addr;
-
-	/* Drive Info buffer */
-	uint32_t drives_length;
-	uint32_t drives_addr;
-
-	/* ROM configuration table */
-	uint32_t config_table;
-
-	/* Boot Loader Name */
-	uint32_t boot_loader_name;
-
-	/* APM table */
-	uint32_t apm_table;
-
-	/* Video */
-	uint32_t vbe_control_info;
-	uint32_t vbe_mode_info;
-	uint16_t vbe_mode;
-	uint16_t vbe_interface_seg;
-	uint16_t vbe_interface_off;
-	uint16_t vbe_interface_len;
-};
-
-#define MULTIBOOT_MEMORY_AVAILABLE              1
-#define MULTIBOOT_MEMORY_RESERVED               2
-
-struct multiboot_mmap_entry {
-	uint32_t size;
-	uint64_t addr;
-	uint64_t len;
-	uint32_t type;
-} __attribute__ ((packed));
-
-extern struct multiboot_info *mbi;
-
-unsigned long  write_multiboot_info(unsigned long rom_table_end);
-
-#endif



More information about the coreboot-gerrit mailing list