[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20070822121731.011cd263.randy.dunlap@oracle.com>
Date: Wed, 22 Aug 2007 12:17:31 -0700
From: Randy Dunlap <randy.dunlap@...cle.com>
To: Andi Kleen <ak@...e.de>
Cc: 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, 22 Aug 2007 21:38:58 +0200 Andi Kleen wrote:
> On Wed, Aug 22, 2007 at 11:32:11AM -0700, Andrew Morton wrote:
> > > On Wed, 22 Aug 2007 02:06:48 -0700 Andrew Morton wrote:
> > >
> > > >
> > > > ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/patches/2.6/2.6.23-rc3/2.6.23-rc3-mm1/
> > >
> > > allyesconfig on x86_64 says:
> > >
> > > kernel/unwind.c:1016:31: error: undefined identifier '__builtin_labs'
> > > kernel/unwind.c:1232:25: error: undefined identifier '__builtin_labs'
> > >
> >
>
> Why does that compiler not know __builtin_abs?
I dunno:
> gcc --version
gcc (GCC) 4.1.0 (SUSE Linux)
> > 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.
>
> I'll fix.
>
> >
> > And what's this stuff, anyway?
> >
> > +typedef unsigned long uleb128_t;
> > +typedef signed long sleb128_t;
> > +#define sleb128abs __builtin_labs
> >
> > unsigned and signed little-endian 128-bit types? Nope, they're 32-bit or
> > 64-bit. All very mysterious.
>
> dwarf2 uses a magic compressing encoding for numbers that uses less bytes
> for small numbers and more bytes for larger numbers. These are the base
> types for this.
>
> It's similar to fs/reiser4/dscale.h in your tree.
>
> -Andi
---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***
-
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/
Powered by blists - more mailing lists