linux-kernel - Re: 2.6.23-rc3-mm1

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:	Wed, 22 Aug 2007 22:53:11 +0200
From:	Andi Kleen <ak@...e.de>
To:	Randy Dunlap <randy.dunlap@...cle.com>
Cc:	Andi Kleen <ak@...e.de>, Andrew Morton <akpm@...ux-foundation.org>,
	linux-kernel@...r.kernel.org, Jan Beulich <jbeulich@...ell.com>
Subject: Re: 2.6.23-rc3-mm1

On Wed, Aug 22, 2007 at 12:17:31PM -0700, Randy Dunlap wrote:
> > Why does that compiler not know __builtin_abs?
> 
> I dunno:
> 
> > gcc --version
> gcc (GCC) 4.1.0 (SUSE Linux)

Hmm I use the same compiler from SUSE10.2 and it works for me (with both
mm and only my tree applied) 

Ok mm fails with some errors in the wireless drivers but with 
wireless disabled it compiles.

When you compile a simple test program like

main() { printf("%lu\n", __builtin_labs(-1)); }

does it work?


> > > One wonders why x86_64-mm-unwinder.patch has an open-coded call to
> > > __builtin_labs(), when include/linux/kernel.h:abs() should do a fine job.

Andrew, I actually checked that and the abs() there is just abs()
not a labs(). So it wouldn't work on 64bit platform.

We could opencode it of course, but __builtin_labs should be really 
there.

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