diff options
| author | Theo de Raadt <deraadt@cvs.openbsd.org> | 2010-12-02 04:35:04 +0000 |
|---|---|---|
| committer | Theo de Raadt <deraadt@cvs.openbsd.org> | 2010-12-02 04:35:04 +0000 |
| commit | 0932b10f5dc8843553000e2951f2f640740a3e10 (patch) | |
| tree | 0ca7e33ebb1d1eb6e026fd77cd56b784d2f61441 /sys/arch/i386 | |
| parent | 80591623fee343785ca049e555271283cce441f6 (diff) | |
move vers.o to before the other objects, so that it is not linked last.
having it linked last is bad (on at least i386 and amd64) because the lapic
is mapped over the start of the data segment -- savecore(8) then reads the
version string for a fixed buffer space, and reads into the lapic area
causing unintended side-effects (at least on Intel X5570 and X5680)
found by pedro, discussed with kettenis and mpf and miod
Diffstat (limited to 'sys/arch/i386')
| -rw-r--r-- | sys/arch/i386/conf/Makefile.i386 | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/sys/arch/i386/conf/Makefile.i386 b/sys/arch/i386/conf/Makefile.i386 index c9b2c5525e1..c85f5207953 100644 --- a/sys/arch/i386/conf/Makefile.i386 +++ b/sys/arch/i386/conf/Makefile.i386 @@ -1,4 +1,4 @@ -# $OpenBSD: Makefile.i386,v 1.76 2010/06/01 23:49:22 deraadt Exp $ +# $OpenBSD: Makefile.i386,v 1.77 2010/12/02 04:35:02 deraadt Exp $ # For instructions on building kernels consult the config(8) and options(4) # manual pages. @@ -73,11 +73,11 @@ NORMAL_S= ${CC} ${AFLAGS} ${CPPFLAGS} -c $< # ${SYSTEM_LD_HEAD} # ${SYSTEM_LD} swapxxx.o # ${SYSTEM_LD_TAIL} -SYSTEM_OBJ= locore.o param.o ioconf.o ${OBJS} +SYSTEM_OBJ= locore.o param.o ioconf.o vers.o ${OBJS} SYSTEM_DEP= Makefile ${SYSTEM_OBJ} SYSTEM_LD_HEAD= @rm -f $@ -SYSTEM_LD= @echo ${LD} ${LINKFLAGS} -o $@ '$${SYSTEM_OBJ}' vers.o; \ - ${LD} ${LINKFLAGS} -o $@ ${SYSTEM_OBJ} vers.o +SYSTEM_LD= @echo ${LD} ${LINKFLAGS} -o $@ '$${SYSTEM_OBJ}'; \ + ${LD} ${LINKFLAGS} -o $@ ${SYSTEM_OBJ} SYSTEM_LD_TAIL= @${SIZE} $@; chmod 755 $@ DEBUG?= |