[PATCH] Avoid redefinitions in libc/include/sys/types.h in case __rtems__ is defined

Sebastian Huber sebastian.huber@embedded-brains.de
Mon Feb 28 13:01:00 GMT 2011

On 02/28/2011 12:40 PM, Corinna Vinschen wrote:
> On Feb 24 13:44, Sebastian Huber wrote:
>> Hello,
>> currently we have in Newlib for RTEMS:
>> sys/types.h:393:0: warning: "PTHREAD_MUTEX_NORMAL" redefined
>> sys/types.h:310:0: note: this is the location of the previous definition
>> sys/types.h:394:0: warning: "PTHREAD_MUTEX_RECURSIVE" redefined
>> sys/types.h:325:0: note: this is the location of the previous definition
>> sys/types.h:396:0: warning: "PTHREAD_MUTEX_DEFAULT" redefined
>> sys/types.h:332:0: note: this is the location of the previous definition
>> In <sys/features.h> we have
>> #ifdef __rtems__
>> ...
>>  #define _UNIX98_THREAD_MUTEX_ATTRIBUTES         1
>> ...
>> #endif
>> We include <sys/features.h> in <sys/types.h> before the PTHREAD_MUTEX_*
>> defines.  The _UNIX98_THREAD_MUTEX_ATTRIBUTES is also used in the RTEMS sources
>> to enable the attribute specific code.
>> Why do we have different numeric values for the PTHREAD_MUTEX_* defines?
> I'm not sure whom you're asking.
> To me, this change looks a bit suspicious.  If _UNIX98_THREAD_MUTEX_ATTRIBUTES
> was defined for RTEMS and XMK for a while, the second set of definition of
> the PTHREAD_MUTEX values have ruled for a long time, right?  

Yes, it is there since:

2008-09-25  Raphael Derossa Pereira  <raphaelpereira@gmail.com>

        * libc/include/sys/types.h[_UNIX98_THREAD_MUTEX_ATTRIBUTES]: Add
        new mutex type values.
        (pthread_mutexattr_t)[_UNIX98_THREAD_MUTEX_ATTRIBUTES]: Add type


> So what your
> patch is doing is to replace the so far used values with the overruled values
> from earlier in the file.  Doesn't that break backward compatibility?

You are right.  The earlier defines are from:

2009-06-17  Michael Eager <eager@eagercon.com>

        * libc/include/pthread.h: Support XMK (Xilinx) BSP, add RTEMS to
        * libc/include/sys/types.h: Same.



RTEMS defines _UNIX98_THREAD_MUTEX_ATTRIBUTES in <sys/features.h> and XMK
defines it in <sys/config.h>.  Both header files will be included in <sys/types.h>.

So hopefully the attached patch is now correct.

Sebastian Huber, embedded brains GmbH

Address : Obere Lagerstr. 30, D-82178 Puchheim, Germany
Phone   : +49 89 18 90 80 79-6
Fax     : +49 89 18 90 80 79-9
E-Mail  : sebastian.huber@embedded-brains.de
PGP     : Public key available on request.

Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: newlib-sys-types.patch
Type: text/x-patch
Size: 2424 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/newlib/attachments/20110228/53e267d0/attachment.bin>

More information about the Newlib mailing list