public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* Tiny cleanup of entry.S
@ 2001-04-20 18:17 Pavel Machek
  0 siblings, 0 replies; only message in thread
From: Pavel Machek @ 2001-04-20 18:17 UTC (permalink / raw)
  To: torvalds, kernel list

Hi!

I believe that no ifdefs are worth several bytes of code expansion...

Index: arch/i386/kernel/head.S
===================================================================
RCS file: /home/cvs/Repository/linux/arch/i386/kernel/head.S,v
retrieving revision 1.1.1.1
diff -u -u -r1.1.1.1 head.S
--- arch/i386/kernel/head.S	2000/09/04 16:54:43	1.1.1.1
+++ arch/i386/kernel/head.S	2001/02/13 13:44:48
@@ -169,9 +169,7 @@
 	rep
 	movsl
 1:
-#ifdef CONFIG_SMP
 checkCPUtype:
-#endif
 
 	movl $-1,X86_CPUID		#  -1 for no CPUID initially
 
@@ -244,9 +242,7 @@
 	orl $2,%eax		# set MP
 2:	movl %eax,%cr0
 	call check_x87
-#ifdef CONFIG_SMP
 	incb ready
-#endif
 	lgdt gdt_descr
 	lidt idt_descr
 	ljmp $(__KERNEL_CS),$1f
@@ -278,9 +274,7 @@
 	jmp L6			# main should never return here, but
 				# just in case, we know what happens.
 
-#ifdef CONFIG_SMP
 ready:	.byte 0
-#endif
 
 /*
  * We depend on ET to be correct. This checks for 287/387.

-- 
I'm pavel@ucw.cz. "In my country we have almost anarchy and I don't care."
Panos Katsaloulis describing me w.r.t. patents at discuss@linmodems.org

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2001-04-20 18:23 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2001-04-20 18:17 Tiny cleanup of entry.S Pavel Machek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox