linux-kernel - Re: 2.6.21-mm1 -- x86 verify

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <4641F19F.605@shadowen.org>
Date:	Wed, 09 May 2007 17:06:55 +0100
From:	Andy Whitcroft <apw@...dowen.org>
To:	Andi Kleen <andi@...stfloor.org>
CC:	Andrew Morton <akpm@...ux-foundation.org>, Andi Kleen <ak@...e.de>,
	linux-kernel@...r.kernel.org, Steve Fox <drfickle@...ibm.com>,
	Mel Gorman <mel@....ul.ie>
Subject: Re: 2.6.21-mm1 -- x86 verify_cpu.S compile failure

Andi Kleen wrote:
> On Tue, May 08, 2007 at 10:46:20AM +0100, Andy Whitcroft wrote:
>> We are seeing the following compile error on older x86 installs:
>>
>>   arch/i386/kernel/verify_cpu.S: Assembler messages:
>>   arch/i386/kernel/verify_cpu.S:13: Error: `(%esp)' is
>> 			 not a valid 16 bit base/index expression
>>
>> Seems to come from:
>>
>>   x86_64-mm-i386-verify-cpu
>>
>> Compiler:
>>
>>   gcc version 3.3.4 (Debian 1:3.3.4-3)
> 
> Your compiler must be a brother in spirit of Andrew's vaio.
> 
> Does this patch help? 
> 
> -Andi
> 
> 
> Index: linux/arch/i386/kernel/verify_cpu.S
> ===================================================================
> --- linux.orig/arch/i386/kernel/verify_cpu.S
> +++ linux/arch/i386/kernel/verify_cpu.S
> @@ -10,7 +10,9 @@ verify_cpu:
>  
>  #if CONFIG_X86_MINIMUM_CPU_MODEL >= 4
>  	pushfl
> -	orl	$(1<<18),(%esp)		# try setting AC
> +	pop	%eax
> +	orl	$(1<<18),%eax		# try setting AC
> +	push	%eax
>  	popfl
>  	pushfl
>  	popl    %eax
> 
> 

Yep that gets us past the compile problems.  Of course she then blows up
cause SLUB does not work on x86 in that version ... but ...

Acked-by: Andy Whitcroft <apw@...dowen.org>

-apw
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/