Oops.

From: uclibc-bounces@uclibc.org on behalf of Peter Kjellerstedt
[peter.kjellerstedt@axis.com]
Peter Kellerstedt writes:

May I suggest the attached patch instead?
It handles PICFLAG the same way as it was already done
for other architectures (e.g., CRIS and frv).

This should fix the dlsym problem Peter van Hoyweghen reported.
However RTLD_LOCAL still doesn't work. Everything is RTLD_GLOBAL.

Joakim Tjernlund writes:

PPC32, SPARC32/64 and S390 includes the PLT in its RELA size. This caused ldso
to always do unlazy relocation of the JMPRELs. This patch fixes it.

Joakim Tjernlund writes:

This patch makes -fpic work for PCC and optimzes the relcation by moving the cache
flushing stuff to JMP relocs only. Actually PPCs ldso can only handle small
GOT tables(<=8192 entries)anyhow, so it makes little sense to compile PPC with -fPIC.
libuClibc shrunk from 340724 to 330780 bytes with -fpic.

doh

Add missing header file needed to define PAGE_SIZE

As Joakim Tjernlund rightly notes, I should have used PAGE_SIZE
not the hard coded value of 4096.

cut-n-paste strikes again

Cleanup some dead wood in the header files

s/___brk_addr/__curbrk/g
Some utilities, such as valgrind, have a legitimate reason to know the address
of the current brk.  Since we know such utils will peek under our skirt, we
might as well give them what they expect and not use a gratuitously different
symbol name.
 -Erik

Some mips updates.  Add a couple of missing sys/*.h files and update a few others.

Teach libdl to use normal libc symbols whenever possible, i.e. malloc
rather than _dl_malloc, fprintf rather than _dl_printf, etc.
 -Erik

Whitespace cleanup

Fix screwed up indenting

Do not grovel about in the ELF header for the page size when
running on uClinux, which at runtime uses the FLAT file format.

Be certain to include sys/sysmacros.h when major() and minor()
are being used

Remove all reference to __GLIBC_HAVE_LONG_LONG

Support linux 2.6.x wide device major/minor numbers

Fix ftruncate64 and truncate64 for powerpc and mips
 -Erik

Update todo list

Update sigaction for arm

Add sigaction for mips

Follow glibc's lead and add a gcc-3.4.1 required symbol to crt0.S.

Hide the arm libfloat support for now.  No longer needed for gcc 3.4.1 and
won't be needed for 3.3.4 either when I get some time to clean up that
toolchain which also suffers from the g++ include dir search order bug.

Revert this stuff for now

Pete Popov writes:

Hi Erik,

I'm not sure why the NIOS support is not in uClibc -- perhaps the patch
was rejected or never submitted? In any case, I'm playing with some NIOS
stuff and created this patch against 0.9.26. The work was done by
Microtronix. I'm not sure who else contributed to it. It would be great
to have the NIOS support available in uClibc so developers don't have to
go searching for these bits.

Pete

Cleanup some cruft

update packaging a bit

Cleanup a few of the more egregiously broken sysconf values to
actually match reality.  In particular, _SC_GETPW_R_SIZE_MAX and
_SC_GETGR_R_SIZE_MAX were causing us problems with programs such
as libglib, since they were always returning -1, which is a bit
smaller than the actual passwd and group max buffer sizes.
 -Erik

_SC_PAGESIZE is standard.  Some ancient legacy unix variants used _SC_PAGE_SIZE
instead, but we are not such a system and should not propagate such things.

_SC_PAGESIZE is standard.  Some ancient legacy unix variants used _SC_PAGE_SIZE
instead, but we are not such a system and should not propagate such things.

Use MAP_PRIVATE whenever __ARCH_HAS_MMU__ is set.
Only use MAP_SHARED when mmu-less.

Atsushi Nemoto writes:

This is a patch for uClibc/libc/sysdeps/linux/mips/sys/procfs.h pulled
from glibc CVS.

This fixes gdb with threaded application (with glibc environment).
Although debugging threaded programs seems not working on uClibc/mips
yet, this patch will be required when it becomes working.

This patch can be applied to any version of uClibc (including current
CVS).

h8300 updates from Yoshinori Sato

Patch from Yoshinori Sato to add H8/300 relocation types.

Peter Kjellerstedt at axis.com writes:

Hello,

the attached patch should bring extra/config in line
with the Linux 2.6.7 sources.

The following are the commit messages for the respective
files from the Linux bk-repository:

checklist.c:
* fix menuconfig choice item help display

confdata.c:
* config: choice fix
* kconfig: don't rename target dir when saving config

expr.c, expr.h:
* config: disable debug prints

mconf.c:
* fix menuconfig choice item help display

menu.c:
* Kconfig: use select statements

symbol.c:
* config: choice fix
* Avoid bogus warning about recursive dependencies
* c99 struct initialiser conversions

textbox.c:
* janitor: don't init statics to 0

util.c:
* fix lxdialog behaviour

//Peter

Sigh.  I got things working this morning, then checked stuff in from the wrong
tree.  Bad boy, No doughnut.
 -Erik

Based on a patch from Alexandre Oliva, make sure _dl_malloc returns a nicely
aligned pointer that may be aligned up to page_size.  Also add _dl_free,

Update FAQ a bit

Joakim Tjernlund writes:

Hi yet again :)

in dl-startup.c when performing boot strap relocation the following test
exists to make sure that only "_dl_" symbols are relocated:
/* We only do a partial dynamic linking right now.  The user
 is not supposed to define any symbols that start with a
 '_dl', so we can do this with confidence. */
 if (!symname || !_dl_symbol(symname)) {
        continue;
 }

However on PPC(and the other archs as well I suspect) all symbols are
"_dl_" symbols so the test is never true. The test can be removed and the
whole loop simplified(smaller). This also makes it possible to
simplify elfinterp.c

This remove the scanning of ldso.so relocs, making relocation faster.

I have tested this on PPC and it works well.
Do you think this optimization will work for the other arches as well?
I can't see why not.

     Jocke

* Tested on x86, arm, mipsel, and powerpc by Erik and works nicely
 -Erik

Disable support for R_ARM_PC24, just like R_PPC_REL24 these should
only occur when shared libs are not compiled with -fPIC

Let dl-string.h include dl-sysdep.h, since it has proper include guards

Whine about R_PPC_REL24 relocations, and tell people to compile
shared libraries with -fPIC

Joakim Tjernlund writes:

Make old weak symbol handling work, if someone wants to
enable it.

Joakim Tjernlund writes:

Since MIPS don't have COPY relocs it seems apropiate to define
DL_NO_COPY_RELOCS. This will optimize dl_find_hash somewhat.

Joakim Tjernlund writes:

Glibc does not use COPY class for SHN_COMMOM in MIPS. Make uClibc ldso
the same.

Bradley D. LaRonde writes:

dl-string.h references do_rem, but do_rem is a #define in <arch>/dl-sysdep.h
which is not included by dl-sysdep.h.  This causes a problem in libdl:

    In file included from ../../ldso/include/ldso.h:27, from libdl.c:33:
    ../../ldso/include/dl-string.h: In function `_dl_simple_ltoa':
    ../../ldso/include/dl-string.h:216: warning: implicit declaration of
function `do_rem'

Include dl-sysdep.h in dl-string.h before using do_rem.

note new mailing list posting policy

Stephen Hemminger from osdl dot org writes:

Rather than copy more code back into the bridge-utilities, how about applying this
change to uClibc?  I hate when packages get cluttered for workarounds for other
incompatibilities.

The problem is that SIOCGIFCONF only lists interfaces that have IP addresses, so it
doesn't find the other interfaces that are being used for bridging. It could be fixed
in the kernel to return all interfaces, but then something else might break; and still
it mean a kernel update for the 2.4 users.

The whole use of ifindex in the bridge API is a bad idea. But we probably have to live
with it for compatibility.

Patch against uClibc 0.9.26

Jakub Bogusz from pld-linux dot org writes:

Hello,

I managed to get ldso (and thus shared linking to uClibc) to work on
sparc (actually sparc64 kernel with 32-bit userspace), at least on
simple "hello world" program (more complex ones not tested).

Some notes on attached patch (against 0.9.26, would require some work
to apply on current CVS - but I tested 0.9.26, not CVS):

- ELF magic cannot be examined by _dl_strncmp so early, probably because of
  string constant, like on ppc/mips/sh
  (note that early SEND_STDERR still crashes when trying to do _dl_strlen
   - I suppose that string constants require relocation; but adding
   load_addr didn't help, just ELF header was displayed instead of crash)
- mmap() is syscall6 like on ppc/mips/sh, not old i386 mmap()
- for generic sparc (i.e. not sparcv8/sparcv9) gcc produces .udiv/.urem
  calls for unsigned integer / and % operators - so these operations
  must be avoided. I copied do_rem definition from arm header.
  But / and % are used also in _dl_simple_ltoa() and
  _dl_simple_ltoahex(); in ltoahex gcc optimizes it to shifts (but
  I think it's safer to use shifts explicitly, not rely on
  optimization...).
  I changed % in ltoa to do_rem, but as there was no do_div definition,
  I changed all "%d" specifiers to "%x" to avoid crashes (this changes
  wouldn't be needed if _dl_simple_ltoa() were fixed to not use
  division on sparc).
- "#define SOLARIS_COMPATIBLE" in ld_sysdep.h broke ldso on Linux
  because of redefining _dl_linux_resolve only in some places (it was
  still referenced in INIT_GOT before redefinition). So
  _dl_linux_resolve redefinition should be moved before INIT_GOT
  definition or removed.
- sparc64 kernel requires mmap() addresses to be aligned to 8192, not
  4096, otherwise mmap() call failed
- reloc_entry must be shifted by 10, not 12 (I found similar operation
  in glibc sources)

Aside of sparc-specific fixes:
- I moved some _dl_dprintf()s inside if(_dl_debug_*) conditions (to avoid
  debugging messages when LD_DEBUG is not defined)
- it seems that there was possible off-by-one in ltoa and ltoahex?
  they are called with char[22] as 1st argument, and then '\0' is stored
  in local[22] (_before_ p decrementation)... or am I missing something?
  If not, fix is included in patch.

Philip Craig at snapgear dot com writes:

Strlen was counting the last 3 bytes incorrectly for big endian arm.

Per comments from Bernhard Rosenkraenzer, adjust gcc 3.3.x generated asm
with s/i686.get_pc_thunk.bx/get_pc_thunk_bx/g to make gcc 3.4 happy.

Oops.  Missed a spot.

scrap the goofy time + getpid() based "randomness" and use
the real thing.

Removed unused file.

Cope when the __NR_get_kernel_syms is undefined (i.e. 2.6.x arm-linux)

Fix changed link into the mailing list archive.

Joakim Tjernlund writes:

Hi Erik

It seems to me that __pthread_once and __pthread_initialize_minimal could be made
WEAKs with no stub. The code in rpc_thread.c and __uClibc_main.c appears to expect this.
Also, __pthread_return_0 __pthread_return_1 and __pthread_return_void can be static, not to
pollute the name space.

  Jocke

Alexandre Oliva writes:

Here are a number of minor changes to FR-V-specific bits of the uClibc
port:

- I've adjusted the definition of _dl_mmap to cope with the fact that
  there will be a definition for an mmap2-only system.

- We don't have COPY relocs, so optimize the copy reloc-related code
  away.

- Change the page size to 16KiB, to match the ABI spec, and not a
  stale value I'd copied from a linker config file.  Oops.

- Fix error handling in clone and vfork; parts of the changes by David
  Howells <dhowells@redhat.com>

- Rearrange includes in crtreloc.c.

- Change the API of __syscall_error, to reduce code size.

- Improve __syscall_return in terms of code size, so as to enable
  tail-calling of __syscall_error, at least within libc.  Ideally,
  __syscall_error should be hidden within libc.so, but this didn't
  work because of libdl.  I haven't looked into why, and figured I'd
  leave it visible for now.

- Rename enumerators and macros in sys/ucontext.h to reduce namespace
  pollution.

__data_start needs to be added to all crt0.S files that don't currently
have it.  It is used by the boehm gc, amoung other things.

Kill off the now obsolete __libc_vfork symbol

Alexandre Oliva writes:

This patch introduces optimized versions of memcpy and memset for
frv.

Patch from Alexandre Oliva:

On Mar 20, 2004, Erik Andersen <andersen@codepoet.org> wrote:

> If you supply a FR-V specific link.h header into
> libc/sysdeps/linux/frv/ then you can do the same sortof
> thing that mips does i.e. with sgidefs.h in the headers
> target in libc/sysdeps/linux/mips/Makefile

Thanks, this patch implements your suggestion.

Alexandre Oliva writes:

A program that requests __pthread_sig_debug to be blocked will
self-deadlock when it requests a thread to be created, because the
debugger (rda or gdb) will never get the signal, so it won't wake up
the pthread manager as expected.

This patch fixes it.

update libdl so it also groks the new _dl_errno

Add define for SHM_HUGETLB

Bradley D. LaRonde writes:

Oops, this adds the page alignment offset to the mprotected size.

Regards,
Brad

Patch from Bradley D. LaRonde:
ld.so mprotect rwx .dyanmic segment for mips DT_DEBUG
Allow writing debug_addr into the .dynamic segment.
Even though the program header is marked RWE, the kernel
gives it to us rx.

Add a local '_dl_errno' to be used by syscalls in ldso, allowing
useful syscall failure diagnostics.

As noticed by the sharp eyed Joakim Tjernlund, I stupidly
hard coded 4096 instead of PAGE_SIZE.  Because I'm an idiot.

Setup a _dl_pagesize value for use by staticly linked apps

make certain that getpagesize() returns correct the value for mips
by extracting the value from the ELF header.

Purge trailing white space

Bradley D. LaRonde, brad at laronde dot org writes:

If I change that ELF_RTYPE_CLASS_COPY to ELF_RTYPE_CLASS_PLT to tell
_dl_find_hash to ignore stubs when resolving undefined functions without
stubs, the dlopen tests all pass.  dlopen gets a pointer to the libc.so
malloc instead of a pointer to the libpthread malloc stub.  Yay!  :-)

Save r0 before modifying it.

Obligatory forgotten file...

Change clock() to allow wrapping.
Add timegm() function.
Make lookup_tzname() static (as it should have been).
Have strftime() get timezone information from the passed struct
  for the %z and %Z conversions when using struct tm extensions.

Peter Mazinger asked for this several months ago.

Patch from Yoshinori Sato <ysato@users.sourceforge.jp>.

"Fix" this so that at least things will link when threading is disabled.
Also fix the sizeof() issue since the change to a dynamicly allocated buf.
Note!  This is still broken wrt threading, but so is the glibc version.
I'm just commiting this for new until I can test my rewrite.

Fix a couple of issues in configurations I hadn't tested.

Add missing include.

UCLIBC_COMPLETELY_PIC does nothing, so kill it

Change the global 'buf' to 'servbuf' since some functions in here use
a local named 'buf' and we want to avoid shadowing that.

Alexandre Oliva writes:

The vfork() wrapper defined in libpthread, that's used to run
pthread_atfork()-registered handlers, is not only a very bad idea,
it's broken and useless.  Here's the rationale:

[---------snip----------]

Since the implementation as it stands is broken (linking a program
that vfork()s and exec()s on the child and wait()s on the parent works
unless you happen to link with libpthread), and I can't think of
any workable solution, I suggest that we simply remove the vfork()
overrider in the non-MMU case.  Yes, we might lose some small amount
of functionality here, but it's not like people running uClinux expect
anything resembling actual fork() to work.

Fix screwed up formatting

Jeroen Dobbelaere writes:

Because variables are linked to fixed registers, there is a problem in :
     (*__errno_location())=(-_r0);

As __errno_location() uses r0 to return the address of the errno location,
the negated address will be assigned instead of the error code.

Attached patch will resolve this.

Minor .deb updates

These headers should not be included in uClibc -- they should be
part of the ppp instead.

It turns out that asm/reg.h is missing from the 2.6.x kernel headers,
which prevents libthread_db (used by gdb) from compiling.  Include a
copy within include/sys/user.h for mips, per what was done in glibc.

blah

Fix grammar and spelling

Mention uwoody

Cope with gcc 3.4's more aggressive persuit of attribute unused

We don't need these...

Fix typo.

Fix a bug noticed by Oleg I. Vdovikin, oleg at cs dot msu dot su.
This code in uClibc would attempt to set the default facility if
none was specified.  Except none being specified is 0, which is
also LOG_KERN, therefore klogd could never get kernel messages
logged as LOG_KERN.

glibc does not provide pppio.h, and our doing so conflicts
with the ppp package

Per patch from Stefan Holst, do not try to compile these
syscalls if they are not supported.