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 for Android: free password hash cracker in your pocket

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

Date:	Thu, 10 May 2007 09:33:29 +0100
From:	Andy Whitcroft <apw@...dowen.org>
To:	Andi Kleen <ak@...e.de>
CC:	Andy Whitcroft <apw@...dowen.org>,
	Andi Kleen <andi@...stfloor.org>,
	Andrew Morton <akpm@...ux-foundation.org>,
	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

Andy Whitcroft wrote:
> 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>

I note that this problem is also up in mainline, since 2.6.21-git6.
Andi can you push this fix up too?

-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/