NPTL not building?

[Carmelo Amoroso]

On 29/01/11 02:08, Rob Landley wrote:
> On 01/27/2011 08:37 AM, Carmelo AMOROSO wrote:
>> -----BEGIN PGP SIGNED MESSAGE-----
>> Hash: SHA1
>>
>> On 1/27/2011 3:34 PM, Rob Landley wrote:
>>> AS
>>> libpthread/nptl/sysdeps/unix/sysv/linux/i386/i686/pthread_cond_timedwait.oS
>>> libpthread/nptl/sysdeps/unix/sysv/linux/i386/i686/../i486/pthread_cond_timedwait.S:
>>> Assembler messages:
>>> libpthread/nptl/sysdeps/unix/sysv/linux/i386/i686/../i486/pthread_cond_timedwait.S:40:
>>> Error: unknown pseudo-op: `.cfi_personality'
>>> libpthread/nptl/sysdeps/unix/sysv/linux/i386/i686/../i486/pthread_cond_timedwait.S:42:
>>> Error: unknown pseudo-op: `.cfi_lsda'
>>> make: ***
>>> [libpthread/nptl/sysdeps/unix/sysv/linux/i386/i686/pthread_cond_timedwait.oS]
>>> Error 1
>>>
>>> Did I hork something in my .config? I replaced:
>>>
>>> LINUXTHREADS_OLD=y
>>>
>>> With
>>>
>>> UCLIBC_HAS_THREADS_NATIVE=y
>>>
>>> And it stopped building. Switched it back and it built fine.
>>>
>>> Rob
>>
>> Rob,
>> it seems that your binutils is too old and does not support CFI pseudo
>> ops (or part of them).
> 
> It's the last GPLv2 release, binutils 2.17.  So you're saying that NPTL
> support won't build under non-GPLv3 toolchain.
> 
> That's a show-stopper bug for me.
> 
> Rob

In understand.
An option could be to transform all .cfi_xxx pseudo ops in a corresponding macro cfi_xxx
that expands to nothing if the binutils or arch does not support (or the user does not want) CFI.

A lot of CFI ops are already managed in this way. There is a define under uClibc_arch_feature that could be used for this

 /* define if target supports CFI pseudo ops */
 #undef __UCLIBC_HAVE_ASM_CFI_DIRECTIVES__

Does it sound reasonable ?

We should review all the code and transform the missing one.
I had a similar problem with ARM/NPTL where my binutils 2.10 does not support .cfi_sections that is used
in some the unwdiner asm code. I have a my own patch for this, not yet fully complete.

I'll see to find some time to come with a proposal patch.

Cheers,
Carmelo

> _______________________________________________
> uClibc mailing list
> uClibc at uclibc.org
> http://lists.busybox.net/mailman/listinfo/uclibc
>