This is the mail archive of the libc-ports@sources.redhat.com mailing list for the libc-ports project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Update ARM and MIPS ports for libc changes


On 9/11/07, Joseph S. Myers <joseph@codesourcery.com> wrote:
> This patch updates the ARM and MIPS ports for changes to libc over the
> past several months.  (Other ports may well also need updates - HPPA for
> the private futex changes, various for new defines in headers and extern
> inline fixes.)

I finished all the HPPA patches on the weekend, and haven't had time
to verify that the testsuite run shows no regressions. I will probably
do that this coming weekend.

> There are some new header defines and extern inline fixes, but the bulk is
> private futex updates, where I've tried to follow various the examples of
> various targets in libc as guidance.  Two points of particular note:
> <sysdep.h> needs to include <tls.h> for consistency with libc targets to
> get the <tls.h> definitions in some source files; <lowlevellock.h> gets
> indirectly included from the middle of headers such as <atomic.h> and
> <tls.h> which means that inline functions in <lowlevellock.h> can't use
> all the macros from those two headers and some such inline functions have
> had to change to macros to avoid this problem (as has been done for
> various targets in libc).

I tracked down the header dependency issue to the following apparently
unneeded inclusion of lowlevellock.h. Removing the include of
lowlevelock.h from nptl/descr.h fixes the problem for hppa.

--- descr.h     1 Aug 2007 04:09:32 -0000       1.42
+++ descr.h     12 Sep 2007 14:33:25 -0000
@@ -27,7 +27,7 @@
 #include <sys/types.h>
 #include <hp-timing.h>
 #include <list.h>
-#include <lowlevellock.h>
 #include <pthreaddef.h>
 #include <dl-sysdep.h>
 #include "../nptl_db/thread_db.h"

Why is descr.h including lowlevellock.h? It doesn't use any of the
definitions therein.

Cheers,
Carlos.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]