From a15933d86161a17ed17d531d83c09e5fafbf6161 Mon Sep 17 00:00:00 2001 From: Stefan Reinauer Date: Thu, 11 Oct 2007 16:13:12 +0000 Subject: [PATCH] Create shared symbols for stage0 functions. This fixes for example printk calls from initram code. Signed-off-by: Stefan Reinauer Acked-by: Jordan Crouse git-svn-id: svn://coreboot.org/repository/LinuxBIOSv3@505 f3766cd6-281f-0410-b1cd-43a5c92072e9 --- arch/x86/Makefile | 5 ++ arch/x86/pci_ops_conf2.c | 6 +-- include/console.h | 6 +-- include/post_code.h | 4 +- include/shared.h | 73 +++++++++++++++++++++++++++ mainboard/adl/msm800sev/Makefile | 17 +++++-- mainboard/amd/norwich/Makefile | 17 +++++-- mainboard/artecgroup/dbe61/Makefile | 17 +++++-- mainboard/emulation/qemu-x86/Makefile | 11 ++-- 9 files changed, 129 insertions(+), 27 deletions(-) create mode 100644 include/shared.h diff --git a/arch/x86/Makefile b/arch/x86/Makefile index f8783a19e4..b1c626c656 100644 --- a/arch/x86/Makefile +++ b/arch/x86/Makefile @@ -148,6 +148,11 @@ $(obj)/stage0.o $(obj)/stage0.init: $(STAGE0_OBJ) $(Q)printf " OBJCOPY $(subst $(shell pwd)/,,$(@))\n" $(Q)$(OBJCOPY) -O binary $(obj)/stage0.o $(obj)/stage0.init + $(Q)# Do another OBJCOPY to get a copy with renamed symbols + $(Q)# for XIP code. + $(Q)printf " OBJCOPY $(subst $(shell pwd)/,,$(@)) (prefixing stage0)\n" + $(Q)$(OBJCOPY) --prefix-symbols=stage0_ $(obj)/stage0.o $(obj)/stage0-prefixed.o + $(Q)printf " TEST $(subst $(shell pwd)/,,$(@))\n" $(Q)test `wc -c < $(obj)/stage0.init` -gt 16128 && \ printf "Error. Bootblock got too big.\n" || true diff --git a/arch/x86/pci_ops_conf2.c b/arch/x86/pci_ops_conf2.c index 668af52e49..278a1d0d18 100644 --- a/arch/x86/pci_ops_conf2.c +++ b/arch/x86/pci_ops_conf2.c @@ -12,8 +12,8 @@ */ #define IOADDR(devfn, where) ((0xC000 | ((devfn & 0x78) << 5)) + where) -#define FUNC(devfn) (((devfn & 7) << 1) | 0xf0) -#define SET(bus,devfn) outb(FUNC(devfn), 0xCF8); outb(bus, 0xCFA); +#define DEVFUNC(devfn) (((devfn & 7) << 1) | 0xf0) +#define SET(bus,devfn) outb(DEVFUNC(devfn), 0xCF8); outb(bus, 0xCFA); static u8 pci_conf2_read_config8(struct bus *pbus, int bus, int devfn, int where) { @@ -65,7 +65,7 @@ static void pci_conf2_write_config32(struct bus *pbus, int bus, int devfn, int w #undef SET #undef IOADDR -#undef FUNC +#undef DEVFUNC struct pci_bus_operations pci_cf8_conf2 = { diff --git a/include/console.h b/include/console.h index a4384c493e..a62c30c3cc 100644 --- a/include/console.h +++ b/include/console.h @@ -18,6 +18,7 @@ #define CONSOLE_H #include +#include /* We share symbols from stage 0 */ #define BIOS_EMERG 0 /* system is unusable */ #define BIOS_ALERT 1 /* action must be taken immediately */ @@ -45,8 +46,7 @@ struct console_driver { int (*tst_byte)(void); }; -// -int printk(int msg_level, const char *fmt, ...) - __attribute__((format (printf, 2, 3))); +SHARED_WITH_ATTRIBUTES(printk, int, __attribute__((format (printf, 2, 3))), + int msg_level, const char *fmt, ...); #endif /* CONSOLE_H */ diff --git a/include/post_code.h b/include/post_code.h index 17edef470f..700fe3a4c7 100644 --- a/include/post_code.h +++ b/include/post_code.h @@ -18,7 +18,9 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA */ -void post_code(u8 value); +#include + +SHARED(post_code, void, u8 value); #define POST_START_OF_MAIN 0x01 diff --git a/include/shared.h b/include/shared.h new file mode 100644 index 0000000000..9af9fc1168 --- /dev/null +++ b/include/shared.h @@ -0,0 +1,73 @@ +/* + * This file is part of the LinuxBIOS project + * + * Copyright(C) 2007 coresystems GmbH + * Written by Stefan Reinauer + * + * 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 + */ + +#ifndef SHARED_H +#define SHARED_H + +#ifdef _SHARED +/* _SHARED mode enforces some functions to be called with an + * absolute address, even in PIE mode. This is required since + * the relative distance between XIP code and stage 0 is not known + */ +#define FUNC(func, ret, attr, args...) \ + ret stage0_##func(args) attr +#define EXTERN(func, ret, attr, args...) \ + ret (*func)(args) attr= stage0_##func +#else +#define FUNC(func, ret, attr, args...) \ + ret func(args) attr +#define EXTERN(func, ret, attr, args...) +#endif + +/** + * Use the SHARED macro to create a universally usable function + * prototype. This will create a function prototype in stage 0 and + * a function prototype plus a function pointer for all code compiled + * with _SHARED defined (required for XIP code) + * + * @func function name + * @ret return value + * @args function arguments + */ + +#define SHARED(func,ret,args...) \ + FUNC(func,ret,,##args); \ + EXTERN(func,ret,,##args) + + +/** + * Use the SHARED_WITH_ATTRIBUTES macro to create a universally usable function + * prototype for a function using GCC attributes. + * This macro works identically to SHARED(), but it adds a GCC attribute to the + * function. So far we use this to have printk parameters tested with a + * "format" attribute. + * + * @func function name + * @ret return value + * @attr function attributes + * @args function arguments + */ + +#define SHARED_WITH_ATTRIBUTES(func,ret,attr,args...) \ + FUNC(func,ret,attr,##args); \ + EXTERN(func,ret,attr,##args) + +#endif /* SHARED_H */ diff --git a/mainboard/adl/msm800sev/Makefile b/mainboard/adl/msm800sev/Makefile index 22c2f0e444..2202d276a0 100644 --- a/mainboard/adl/msm800sev/Makefile +++ b/mainboard/adl/msm800sev/Makefile @@ -32,14 +32,21 @@ INITRAM_OBJ = $(obj)/mainboard/$(MAINBOARDDIR)/initram.o \ $(obj)/southbridge/amd/cs5536/smbus_initram.o \ $(obj)/arch/x86/geodelx/geodelx.o -# These are possibly not permanent -INITRAM_OBJ += $(obj)/lib/console.o $(obj)/lib/vtxprintf.o $(obj)/lib/uart8250.o $(obj)/arch/x86/post_code.o +# Next Quest: Make a single rule out of those: +$(obj)/mainboard/$(MAINBOARDDIR)/initram.o: $(src)/mainboard/$(MAINBOARDDIR)/initram.c + $(Q)$(CC) $(INITCFLAGS) -D_SHARED -fPIE -c $< -o $@ +$(obj)/northbridge/amd/geodelx/raminit.o: $(src)/northbridge/amd/geodelx/raminit.c + $(Q)$(CC) $(INITCFLAGS) -D_SHARED -fPIE -c $< -o $@ +$(obj)/southbridge/amd/cs5536/smbus_initram.o: $(src)/southbridge/amd/cs5536/smbus_initram.c + $(Q)$(CC) $(INITCFLAGS) -D_SHARED -fPIE -c $< -o $@ +$(obj)/arch/x86/geodelx/geodelx.o: $(src)/arch/x86/geodelx/geodelx.c + $(Q)$(CC) $(INITCFLAGS) -D_SHARED -fPIE -c $< -o $@ -$(obj)/linuxbios.initram: $(obj)/stage0.init $(obj)/stage0.o $(INITRAM_OBJ) +$(obj)/linuxbios.initram $(obj)/linuxbios.initram.map: $(obj)/stage0.init $(obj)/stage0-prefixed.o $(INITRAM_OBJ) $(Q)# initram links against stage0 $(Q)printf " LD $(subst $(shell pwd)/,,$(@))\n" - $(Q)$(LD) -Ttext 0x80000 $(INITRAM_OBJ) \ - --entry=main -o $(obj)/linuxbios.initram.o + $(Q)$(LD) --entry main -N -R $(obj)/stage0-prefixed.o \ + $(INITRAM_OBJ) -o $(obj)/linuxbios.initram.o $(Q)printf " OBJCOPY $(subst $(shell pwd)/,,$(@))\n" $(Q)$(OBJCOPY) -O binary $(obj)/linuxbios.initram.o \ $(obj)/linuxbios.initram diff --git a/mainboard/amd/norwich/Makefile b/mainboard/amd/norwich/Makefile index 97986e1984..b874e8f759 100644 --- a/mainboard/amd/norwich/Makefile +++ b/mainboard/amd/norwich/Makefile @@ -26,8 +26,15 @@ INITRAM_OBJ = $(obj)/mainboard/$(MAINBOARDDIR)/initram.o \ $(obj)/southbridge/amd/cs5536/smbus_initram.o \ $(obj)/arch/x86/geodelx/geodelx.o -# These are possibly not permanent -INITRAM_OBJ += $(obj)/lib/console.o $(obj)/lib/vtxprintf.o $(obj)/lib/uart8250.o $(obj)/arch/x86/post_code.o +# Next Quest: Make a single rule out of those: +$(obj)/mainboard/$(MAINBOARDDIR)/initram.o: $(src)/mainboard/$(MAINBOARDDIR)/initram.c + $(Q)$(CC) $(INITCFLAGS) -D_SHARED -fPIE -c $< -o $@ +$(obj)/northbridge/amd/geodelx/raminit.o: $(src)/northbridge/amd/geodelx/raminit.c + $(Q)$(CC) $(INITCFLAGS) -D_SHARED -fPIE -c $< -o $@ +$(obj)/southbridge/amd/cs5536/smbus_initram.o: $(src)/southbridge/amd/cs5536/smbus_initram.c + $(Q)$(CC) $(INITCFLAGS) -D_SHARED -fPIE -c $< -o $@ +$(obj)/arch/x86/geodelx/geodelx.o: $(src)/arch/x86/geodelx/geodelx.c + $(Q)$(CC) $(INITCFLAGS) -D_SHARED -fPIE -c $< -o $@ STAGE2_MAINBOARD_OBJ = @@ -35,11 +42,11 @@ $(obj)/linuxbios.vpd: $(Q)printf " BUILD DUMMY VPD\n" $(Q)dd if=/dev/zero of=$(obj)/linuxbios.vpd bs=256 count=1 $(SILENT) -$(obj)/linuxbios.initram $(obj)/linuxbios.initram.map: $(obj)/stage0.init $(obj)/stage0.o $(INITRAM_OBJ) +$(obj)/linuxbios.initram $(obj)/linuxbios.initram.map: $(obj)/stage0.init $(obj)/stage0-prefixed.o $(INITRAM_OBJ) $(Q)# initram links against stage0 $(Q)printf " LD $(subst $(shell pwd)/,,$(@))\n" - $(Q)$(LD) -Ttext 0x80000 $(INITRAM_OBJ) \ - --entry=main -o $(obj)/linuxbios.initram.o + $(Q)$(LD) --entry main -N -R $(obj)/stage0-prefixed.o \ + $(INITRAM_OBJ) -o $(obj)/linuxbios.initram.o $(Q)printf " OBJCOPY $(subst $(shell pwd)/,,$(@))\n" $(Q)$(OBJCOPY) -O binary $(obj)/linuxbios.initram.o \ $(obj)/linuxbios.initram diff --git a/mainboard/artecgroup/dbe61/Makefile b/mainboard/artecgroup/dbe61/Makefile index 5c8cb7c4b8..f7e04a205d 100644 --- a/mainboard/artecgroup/dbe61/Makefile +++ b/mainboard/artecgroup/dbe61/Makefile @@ -30,14 +30,21 @@ $(obj)/linuxbios.vpd: INITRAM_OBJ = $(obj)/mainboard/$(MAINBOARDDIR)/initram.o \ $(obj)/arch/x86/geodelx/geodelx.o -# These are possibly not permanent -INITRAM_OBJ += $(obj)/lib/console.o $(obj)/lib/vtxprintf.o $(obj)/lib/uart8250.o $(obj)/arch/x86/post_code.o +# Next Quest: Make a single rule out of those: +$(obj)/mainboard/$(MAINBOARDDIR)/initram.o: $(src)/mainboard/$(MAINBOARDDIR)/initram.c + $(Q)$(CC) $(INITCFLAGS) -D_SHARED -fPIE -c $< -o $@ +$(obj)/northbridge/amd/geodelx/raminit.o: $(src)/northbridge/amd/geodelx/raminit.c + $(Q)$(CC) $(INITCFLAGS) -D_SHARED -fPIE -c $< -o $@ +$(obj)/southbridge/amd/cs5536/smbus_initram.o: $(src)/southbridge/amd/cs5536/smbus_initram.c + $(Q)$(CC) $(INITCFLAGS) -D_SHARED -fPIE -c $< -o $@ +$(obj)/arch/x86/geodelx/geodelx.o: $(src)/arch/x86/geodelx/geodelx.c + $(Q)$(CC) $(INITCFLAGS) -D_SHARED -fPIE -c $< -o $@ -$(obj)/linuxbios.initram: $(obj)/stage0.init $(obj)/stage0.o $(INITRAM_OBJ) +$(obj)/linuxbios.initram $(obj)/linuxbios.initram.map: $(obj)/stage0.init $(obj)/stage0-prefixed.o $(INITRAM_OBJ) $(Q)# initram links against stage0 $(Q)printf " LD $(subst $(shell pwd)/,,$(@))\n" - $(Q)$(LD) -Ttext 0x80000 $(INITRAM_OBJ) \ - --entry=main -o $(obj)/linuxbios.initram.o + $(Q)$(LD) --entry main -N -R $(obj)/stage0-prefixed.o \ + $(INITRAM_OBJ) -o $(obj)/linuxbios.initram.o $(Q)printf " OBJCOPY $(subst $(shell pwd)/,,$(@))\n" $(Q)$(OBJCOPY) -O binary $(obj)/linuxbios.initram.o \ $(obj)/linuxbios.initram diff --git a/mainboard/emulation/qemu-x86/Makefile b/mainboard/emulation/qemu-x86/Makefile index 838e68998d..e8e1f3e558 100644 --- a/mainboard/emulation/qemu-x86/Makefile +++ b/mainboard/emulation/qemu-x86/Makefile @@ -41,14 +41,15 @@ $(obj)/linuxbios.vpd: # INITRAM_OBJ = $(obj)/mainboard/$(MAINBOARDDIR)/initram.o -# These are possibly not permanent -INITRAM_OBJ += $(obj)/lib/console.o $(obj)/lib/vtxprintf.o $(obj)/lib/uart8250.o $(obj)/arch/x86/post_code.o -$(obj)/linuxbios.initram: $(obj)/stage0.init $(obj)/stage0.o $(INITRAM_OBJ) +$(obj)/mainboard/$(MAINBOARDDIR)/initram.o: $(src)/mainboard/$(MAINBOARDDIR)/initram.c + $(Q)$(CC) $(INITCFLAGS) -D_SHARED -fPIE -c $< -o $@ + +$(obj)/linuxbios.initram $(obj)/linuxbios.initram.map: $(obj)/stage0.init $(obj)/stage0-prefixed.o $(INITRAM_OBJ) $(Q)# initram links against stage0 $(Q)printf " LD $(subst $(shell pwd)/,,$(@))\n" - $(Q)$(LD) -Ttext 0x80000 $(INITRAM_OBJ) \ - --entry=main -o $(obj)/linuxbios.initram.o + $(Q)$(LD) --entry main -N -R $(obj)/stage0-prefixed.o \ + $(INITRAM_OBJ) -o $(obj)/linuxbios.initram.o $(Q)printf " OBJCOPY $(subst $(shell pwd)/,,$(@))\n" $(Q)$(OBJCOPY) -O binary $(obj)/linuxbios.initram.o \ $(obj)/linuxbios.initram