summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKyösti Mälkki <kyosti.malkki@gmail.com>2014-12-18 10:36:33 +0200
committerKyösti Mälkki <kyosti.malkki@gmail.com>2014-12-31 09:51:50 +0100
commitbae775a4f43449e79622ed144118554ff780744a (patch)
treede6094464d801f18c33bb6e929ee9afef0cfce82
parenta91e1e6cc18b3d510a8dd5e3b2c7bdbba3299369 (diff)
downloadcoreboot-bae775a4f43449e79622ed144118554ff780744a.tar.xz
arch/x86: Declare GDT symbols and move_gdt()
We relocate GDT to CBMEM, this can be done late in ramstage. Note: We currently do this for BSP CPU only. Change-Id: I626faaf22f846433f25ca2253d6a2a5230f50b6b Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com> Reviewed-on: http://review.coreboot.org/7858 Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org> Tested-by: build bot (Jenkins) Reviewed-by: Edward O'Callaghan <eocallaghan@alterapraxis.com>
-rw-r--r--src/arch/x86/boot/gdt.c12
-rw-r--r--src/arch/x86/boot/tables.c2
-rw-r--r--src/cpu/x86/lapic/lapic_cpu_init.c7
-rw-r--r--src/cpu/x86/mp_init.c5
-rw-r--r--src/include/cpu/x86/gdt.h34
-rw-r--r--src/include/lib.h3
6 files changed, 42 insertions, 21 deletions
diff --git a/src/arch/x86/boot/gdt.c b/src/arch/x86/boot/gdt.c
index 7ebf92ae85..7b64af7c2e 100644
--- a/src/arch/x86/boot/gdt.c
+++ b/src/arch/x86/boot/gdt.c
@@ -20,12 +20,8 @@
#include <types.h>
#include <string.h>
#include <cbmem.h>
-#include <lib.h>
#include <console/console.h>
-
-// Global Descriptor Table, defined in c_start.S
-extern char gdt;
-extern char gdt_end;
+#include <cpu/x86/gdt.h>
/* i386 lgdt argument */
struct gdtarg {
@@ -33,11 +29,13 @@ struct gdtarg {
u32 base;
} __attribute__((packed));
-// Copy GDT to new location and reload it
+/* Copy GDT to new location and reload it.
+ * FIXME: We only do this for BSP CPU.
+ */
void move_gdt(void)
{
void *newgdt;
- u16 num_gdt_bytes = &gdt_end - &gdt;
+ u16 num_gdt_bytes = (u32)&gdt_end - (u32)&gdt;
struct gdtarg gdtarg;
newgdt = cbmem_find(CBMEM_ID_GDT);
diff --git a/src/arch/x86/boot/tables.c b/src/arch/x86/boot/tables.c
index 9f2afd4a2f..8685ed48df 100644
--- a/src/arch/x86/boot/tables.c
+++ b/src/arch/x86/boot/tables.c
@@ -21,6 +21,7 @@
#include <console/console.h>
#include <cpu/cpu.h>
+#include <cpu/x86/gdt.h>
#include <boot/tables.h>
#include <boot/coreboot_tables.h>
#include <arch/pirq_routing.h>
@@ -28,7 +29,6 @@
#include <arch/acpi.h>
#include <string.h>
#include <cbmem.h>
-#include <lib.h>
#include <smbios.h>
diff --git a/src/cpu/x86/lapic/lapic_cpu_init.c b/src/cpu/x86/lapic/lapic_cpu_init.c
index 61b6bd72f7..472d6edf60 100644
--- a/src/cpu/x86/lapic/lapic_cpu_init.c
+++ b/src/cpu/x86/lapic/lapic_cpu_init.c
@@ -21,6 +21,7 @@
*/
#include <cpu/x86/cr.h>
+#include <cpu/x86/gdt.h>
#include <cpu/x86/lapic.h>
#include <delay.h>
#include <halt.h>
@@ -53,11 +54,6 @@ char *lowmem_backup_ptr;
int lowmem_backup_size;
#endif
-extern char _secondary_start[];
-extern char _secondary_gdt_addr[];
-extern char gdt[];
-extern char gdt_end[];
-
static inline void setup_secondary_gdt(void)
{
u16 *gdt_limit;
@@ -72,7 +68,6 @@ static inline void setup_secondary_gdt(void)
static void copy_secondary_start_to_lowest_1M(void)
{
- extern char _secondary_start_end[];
unsigned long code_size;
/* Fill in secondary_start's local gdt. */
diff --git a/src/cpu/x86/mp_init.c b/src/cpu/x86/mp_init.c
index a97a4c532b..3e293f60c4 100644
--- a/src/cpu/x86/mp_init.c
+++ b/src/cpu/x86/mp_init.c
@@ -26,6 +26,7 @@
#include <cpu/cpu.h>
#include <cpu/intel/microcode.h>
#include <cpu/x86/cache.h>
+#include <cpu/x86/gdt.h>
#include <cpu/x86/lapic.h>
#include <cpu/x86/name.h>
#include <cpu/x86/msr.h>
@@ -67,10 +68,6 @@ struct saved_msr {
/* The sipi vector rmodule is included in the ramstage using 'objdump -B'. */
extern char _binary_sipi_vector_start[];
-/* These symbols are defined in c_start.S. */
-extern char gdt[];
-extern char gdt_end[];
-extern char idtarg[];
/* The SIPI vector is loaded at the SMM_DEFAULT_BASE. The reason is at the
* memory range is already reserved so the OS cannot use it. That region is
diff --git a/src/include/cpu/x86/gdt.h b/src/include/cpu/x86/gdt.h
new file mode 100644
index 0000000000..260b158d1f
--- /dev/null
+++ b/src/include/cpu/x86/gdt.h
@@ -0,0 +1,34 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * 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
+ */
+
+#ifndef CPU_X86_GDT
+#define CPU_X86_GDT
+
+/* These symbols are defined in c_start.S. */
+extern char gdt[];
+extern char gdt_end[];
+extern char idtarg[];
+
+/* These symbols are defined in secondary.S. */
+extern char _secondary_gdt_addr[];
+extern char _secondary_start[];
+extern char _secondary_start_end[];
+
+/* Defined in src/arch/x86/boot/gdt.c */
+void move_gdt(void);
+
+#endif /* CPU_X86_GDT */
diff --git a/src/include/lib.h b/src/include/lib.h
index 8f02004bff..772a66f317 100644
--- a/src/include/lib.h
+++ b/src/include/lib.h
@@ -33,9 +33,6 @@ unsigned long log2_ceil(unsigned long x);
/* Defined in src/lib/lzma.c */
unsigned long ulzma(unsigned char *src, unsigned char *dst);
-/* Defined in src/arch/x86/boot/gdt.c */
-void move_gdt(void);
-
/* Defined in src/lib/ramtest.c */
void ram_check(unsigned long start, unsigned long stop);
int ram_check_nodie(unsigned long start, unsigned long stop);