Commit 2136220d authored by Andi Kleen's avatar Andi Kleen Committed by Andi Kleen
Browse files

[PATCH] x86-64: Remove CONFIG_REORDER 



The option never worked well and functionlist wasn't well maintained.
Also it made the build very slow on many binutils version.

So just remove it.

Cc: arjan@linux.intel.com
Signed-off-by: default avatarAndi Kleen <ak@suse.de>
parent a106009b

arch/x86_64/Kconfig

+0 −8
Original line number Diff line number Diff line
@@ -660,14 +660,6 @@ config CC_STACKPROTECTOR_ALL 


source kernel/Kconfig.hz



config REORDER

	bool "Function reordering"

	default n

	help

         This option enables the toolchain to reorder functions for a more 

         optimal TLB usage. If you have pretty much any version of binutils, 

	 this can increase your kernel build time by roughly one minute.



config K8_NB

	def_bool y

	depends on AGP_AMD64 || IOMMU || (PCI && NUMA)

arch/x86_64/Makefile

+0 −1
Original line number Diff line number Diff line
@@ -40,7 +40,6 @@ cflags-y += -m64 
cflags-y += -mno-red-zone

cflags-y += -mcmodel=kernel

cflags-y += -pipe

cflags-kernel-$(CONFIG_REORDER) += -ffunction-sections

cflags-y += -Wno-sign-compare

cflags-y += -fno-asynchronous-unwind-tables

ifneq ($(CONFIG_DEBUG_INFO),y)

arch/x86_64/kernel/functionlist

deleted100644 → 0
+0 −1284

File deleted.

Preview size limit exceeded, changes collapsed.

arch/x86_64/kernel/vmlinux.lds.S

+0 −3
Original line number Diff line number Diff line
@@ -30,9 +30,6 @@ SECTIONS 
	/* First the code that has to be first for bootstrapping */

	*(.bootstrap.text)

	_stext = .;

	/* Then all the functions that are "hot" in profiles, to group them

           onto the same hugetlb entry */

	#include "functionlist"

	/* Then the rest */

	*(.text)

	SCHED_TEXT

CRA Git | Maintained and supported by SUSTech CRA and CCSE