Building glibc 2.5 on Hurd

Aurelien Jarno · External Since: Nov 08, 2004 Posts: 527

Hi all,

I am currently trying to port the glibc 2.5 on Hurd. I am facing the
problem described here:

http://savannah.gnu.org/bugs/?18216

Could somebody take a decision about this bug and propose a patch?

Thanks,
Aurelien

--
.''`. Aurelien Jarno | GPG: 1024D/F1BCDB73
: :' : Debian developer | Electrical Engineer
`. `' aurel32.DeleteThis@debian.org | aurelien.DeleteThis@aurel32.net
`- people.debian.org/~aurel32 | www.aurel32.net

--
To UNSUBSCRIBE, email to debian-hurd-REQUEST.DeleteThis@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster.DeleteThis@lists.debian.org

Barry deFreese · External Since: Jun 28, 2005 Posts: 21

Thomas Schwinge wrote:
> Hello!
>
> Support for tls for Hurd systems is still not fixed properly. Details
> are at <http://savannah.gnu.org/bugs/?17644>. To get a functional
> glibc-2_5-branch, this either needs to be fixed or the tls-requiring bits
> of glibc-2_5-branch have to be bend over to not require it.
>
>
> Regards,
> Thomas
>
Hi folks,

Just an FYI, I have built glibc-2.5 with all of Thomas's patches.
Currently I am getting the following error:

/devel3/bdefreese/glibc-2.5/glibc-2.5/build/libc_pic.os: In function
`mach_open_devstream':
/devel3/bdefreese/glibc-2.5/glibc-2.5/mach/devstream.c:137: undefined
reference to `__libc_errno'
/devel3/bdefreese/glibc-2.5/glibc-2.5/build/libc_pic.os: In function
`dealloc_ref':
/devel3/bdefreese/glibc-2.5/glibc-2.5/mach/devstream.c:117: undefined
reference to `__libc_errno'
/devel3/bdefreese/glibc-2.5/glibc-2.5/build/libc_pic.os: In function
`write_some':
/devel3/bdefreese/glibc-2.5/glibc-2.5/mach/devstream.c:47: undefined
reference to `__libc_errno'
/devel3/bdefreese/glibc-2.5/glibc-2.5/build/libc_pic.os: In function
`devstream_read':
/devel3/bdefreese/glibc-2.5/glibc-2.5/mach/devstream.c:94: undefined
reference to `__libc_errno'
/devel3/bdefreese/glibc-2.5/glibc-2.5/build/libc_pic.os: In function
`__hurd_fail':
.../hurd/hurd.h:76: undefined reference to `__libc_errno'
/devel3/bdefreese/glibc-2.5/glibc-2.5/build/libc_pic.os:/devel3/bdefreese/glibc-2.5/glibc-2.5/hurd/hurdselect.c:202:
more undefined references to `__libc_errno' follow
/devel3/bdefreese/glibc-2.5/glibc-2.5/build/libc_pic.os: In function
`__pause_nocancel':
.../sysdeps/posix/pause.c:55: undefined reference to `sigsuspend_not_cancel'
/devel3/bdefreese/glibc-2.5/glibc-2.5/build/libc_pic.os: In function
`__hurd_fail':
.../hurd/hurd.h:76: undefined reference to `__libc_errno'
.../hurd/hurd.h:76: undefined reference to `__libc_errno'
.../hurd/hurd.h:76: undefined reference to `__libc_errno'
/devel3/bdefreese/glibc-2.5/glibc-2.5/build/libc_pic.os: In function
`execvp':
/devel3/bdefreese/glibc-2.5/glibc-2.5/posix/execvp.c:66: undefined
reference to `__libc_errno'
/devel3/bdefreese/glibc-2.5/glibc-2.5/posix/execvp.c:77: undefined
reference to `__libc_errno'
/devel3/bdefreese/glibc-2.5/glibc-2.5/build/libc_pic.os:/devel3/bdefreese/glibc-2.5/glibc-2.5/posix/execvp.c:115:
more undefined references to `__libc_errno' follow
collect2: ld returned 1 exit status
make[1]: *** [/devel3/bdefreese/glibc-2.5/glibc-2.5/build/libc.so] Error 1
make[1]: Leaving directory `/devel3/bdefreese/glibc-2.5/glibc-2.5'
make: *** [all] Error 2

Thanks,

Barry deFreese (aka bddebian)

--
To UNSUBSCRIBE, email to debian-hurd-REQUEST.TakeThisOut@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster.TakeThisOut@lists.debian.org

Aurelien Jarno · External Since: Nov 08, 2004 Posts: 527

Thomas Schwinge a écrit :
> Hello!
>
> Support for tls for Hurd systems is still not fixed properly. Details
> are at <http://savannah.gnu.org/bugs/?17644>. To get a functional
> glibc-2_5-branch, this either needs to be fixed or the tls-requiring bits
> of glibc-2_5-branch have to be bend over to not require it.

Thanks to Petr Salinger who send me a patch we have chosen this solution
(as m68k is also affected). That's why I tried to build the glibc on
Hurd (it was easier than on m68k), to make sure the patch was enough.

--
.''`. Aurelien Jarno | GPG: 1024D/F1BCDB73
: :' : Debian developer | Electrical Engineer
`. `' aurel32 RemoveThis @debian.org | aurelien RemoveThis @aurel32.net
`- people.debian.org/~aurel32 | www.aurel32.net

--
To UNSUBSCRIBE, email to debian-hurd-REQUEST RemoveThis @lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster RemoveThis @lists.debian.org

Barry deFreese · External Since: Jun 28, 2005 Posts: 21

Aurelien Jarno wrote:
> Thomas Schwinge a écrit :
>
>> Hello!
>>
>> Support for tls for Hurd systems is still not fixed properly. Details
>> are at <http://savannah.gnu.org/bugs/?17644>. To get a functional
>> glibc-2_5-branch, this either needs to be fixed or the tls-requiring bits
>> of glibc-2_5-branch have to be bend over to not require it.
>>
>
> Thanks to Petr Salinger who send me a patch we have chosen this solution
> (as m68k is also affected). That's why I tried to build the glibc on
> Hurd (it was easier than on m68k), to make sure the patch was enough.
>
Oh, I forgot to mention. I also have Jeroen's patch built into the
gnumach that I am building with on my machine.

If any of you want access to the Hurd box I'm building on, please let me
know. Maybe we can co-ordinate efforts? Even though I'm way over my
head as usual.

Thanks!

Barry deFreese (aka bddebian)

--
To UNSUBSCRIBE, email to debian-hurd-REQUEST DeleteThis @lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster DeleteThis @lists.debian.org

Aurelien Jarno · External Since: Nov 08, 2004 Posts: 527

Barry deFreese a écrit :
> Aurelien Jarno wrote:
>> Thomas Schwinge a écrit :
>>
>>> Hello!
>>>
>>> Support for tls for Hurd systems is still not fixed properly. Details
>>> are at <http://savannah.gnu.org/bugs/?17644>. To get a functional
>>> glibc-2_5-branch, this either needs to be fixed or the tls-requiring bits
>>> of glibc-2_5-branch have to be bend over to not require it.
>>>
>> Thanks to Petr Salinger who send me a patch we have chosen this solution
>> (as m68k is also affected). That's why I tried to build the glibc on
>> Hurd (it was easier than on m68k), to make sure the patch was enough.
>>
> Oh, I forgot to mention. I also have Jeroen's patch built into the
> gnumach that I am building with on my machine.
>
> If any of you want access to the Hurd box I'm building on, please let me
> know. Maybe we can co-ordinate efforts? Even though I'm way over my
> head as usual.

In my case I am building the glibc 2.5 directly from the Debian SVN, on
a standard hurd installation and binutils from experimental. I am
currently looking at the patches from Thomas, I will commit them if they
are necessary for the 2.5 branch.

[1] http://svn.debian.org/wsvn/pkg-glibc

--
.''`. Aurelien Jarno | GPG: 1024D/F1BCDB73
: :' : Debian developer | Electrical Engineer
`. `' aurel32.DeleteThis@debian.org | aurelien.DeleteThis@aurel32.net
`- people.debian.org/~aurel32 | www.aurel32.net

--
To UNSUBSCRIBE, email to debian-hurd-REQUEST.DeleteThis@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster.DeleteThis@lists.debian.org

Aurelien Jarno · External Since: Nov 08, 2004 Posts: 527

Thomas Schwinge a écrit :
> Hello!
>
> See <http://savannah.gnu.org/bugs/?17646> for details. This should be
> fairly easy to fix, but has to be tracked down.

We are still using libc_cv_z_relro=no in the Debian package. The change
to configure.in has been done after the 2.5 release, therefore I guess
we don't have to change anything for the Debian package. But a fix would
be appreciated.

--
.''`. Aurelien Jarno | GPG: 1024D/F1BCDB73
: :' : Debian developer | Electrical Engineer
`. `' aurel32.RemoveThis@debian.org | aurelien.RemoveThis@aurel32.net
`- people.debian.org/~aurel32 | www.aurel32.net

--
To UNSUBSCRIBE, email to debian-hurd-REQUEST.RemoveThis@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster.RemoveThis@lists.debian.org

Aurelien Jarno · External Since: Nov 08, 2004 Posts: 527

Thomas Schwinge a écrit :
> Needed for HEAD and glibc-2_5-branch.
>
> 2007-02-07 Thomas Schwinge <tschwinge RemoveThis @gnu.org>
>
> * stdlib/fmtmsg.c: Include <stdint.h>.
>
> Index: stdlib/fmtmsg.c
> ===================================================================
> RCS file: /cvs/glibc/libc/stdlib/fmtmsg.c,v
> retrieving revision 1.20
> diff -u -p -r1.20 fmtmsg.c
> --- stdlib/fmtmsg.c 15 May 2006 18:41:18 -0000 1.20
> +++ stdlib/fmtmsg.c 7 Feb 2007 12:12:36 -0000
> @@ -19,6 +19,7 @@
>
> #include <fmtmsg.h>
> #include <bits/libc-lock.h>
> +#include <stdint.h>
> #include <stdio.h>
> #include <stdlib.h>
> #include <string.h>
>

We have the same patch in our SVN. But it has been refused upstream. See
http://sourceware.org/bugzilla/show_bug.cgi?id=3992

--
.''`. Aurelien Jarno | GPG: 1024D/F1BCDB73
: :' : Debian developer | Electrical Engineer
`. `' aurel32 RemoveThis @debian.org | aurelien RemoveThis @aurel32.net
`- people.debian.org/~aurel32 | www.aurel32.net

--
To UNSUBSCRIBE, email to debian-hurd-REQUEST RemoveThis @lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster RemoveThis @lists.debian.org

Aurelien Jarno · External Since: Nov 08, 2004 Posts: 527

Thomas Schwinge a écrit :
> Hello!
>
> Needed on glibc-2_5-branch and HEAD. Reported at
> <http://savannah.gnu.org/bugs/?18217> and finally on
> <http://sourceware.org/bugzilla/show_bug.cgi?id=3748>, where it was
> rejected by Ulrich Drepper. Roland wanted to have a look.
>
> 2006-12-18 Thomas Schwinge <tschwinge DeleteThis @gnu.org>
>
> * bits/libc-lock.h (__libc_once_else): New definiton.
> * sysdeps/mach/bits/libc-lock.h: Likewise.
> * sysdeps/mach/hurd/bits/libc-lock.h: Likewise.
> * nptl/sysdeps/pthread/bits/libc-lock.h: Likewise.
> * sysdeps/posix/getaddrinfo.c (getaddrinfo): Use __libc_once_else and a
> new local function instead of using implementational details.

Commited in our SVN. Note that the same change is needed for linuxthreads.

--
.''`. Aurelien Jarno | GPG: 1024D/F1BCDB73
: :' : Debian developer | Electrical Engineer
`. `' aurel32 DeleteThis @debian.org | aurelien DeleteThis @aurel32.net
`- people.debian.org/~aurel32 | www.aurel32.net

--
To UNSUBSCRIBE, email to debian-hurd-REQUEST DeleteThis @lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster DeleteThis @lists.debian.org

Aurelien Jarno · External Since: Nov 08, 2004 Posts: 527

Aurelien Jarno a écrit :
> Hi all,
>
> I am currently trying to port the glibc 2.5 on Hurd. I am facing the
> problem described here:
>

I am facing another problem now...

sigsuspend_not_cancel is not defined for Hurd in
sysdeps/mach/hurd/sigsuspend.c. This function is called from
sysdeps/posix/pause.c.

Any volunteer to implement it?

I have tried to remove the call from sysdeps/posix/pause.c to see how
far I am able to build the glibc on Hurd, even if this is a big hack.
Using the other patches mentionned in this thread, I am able to build
the glibc 2.5 up to the point where ld.so and libc.so are available (it
is currently still building). Seems promising.

Cheers,
Aurelien

--
.''`. Aurelien Jarno | GPG: 1024D/F1BCDB73
: :' : Debian developer | Electrical Engineer
`. `' aurel32.TakeThisOut@debian.org | aurelien.TakeThisOut@aurel32.net
`- people.debian.org/~aurel32 | www.aurel32.net

--
To UNSUBSCRIBE, email to debian-hurd-REQUEST.TakeThisOut@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster.TakeThisOut@lists.debian.org

Aurelien Jarno · External Since: Nov 08, 2004 Posts: 527

Thomas Schwinge a écrit :
> Hello!
>
> Needed because of GCC changes. Not everything is ready for upstream
> inclusion.
>
>
> This is no proper fix, but a workaround to not make argp-enabled programs
> segfault on ``PROGRAM --help'' when glibc is build with GCC 4.1.
> Reported on
> <http://lists.gnu.org/archive/html/bug-hurd/2006-12/msg00064.html>.

Given that this patch touches some non Hurd parts of the glibc, I prefer
to stick to GCC 4.0 in the Debian package for now.

But I don't know how long this compiler will be available in Debian, so
a proper patch is welcome.

--
.''`. Aurelien Jarno | GPG: 1024D/F1BCDB73
: :' : Debian developer | Electrical Engineer
`. `' aurel32 DeleteThis @debian.org | aurelien DeleteThis @aurel32.net
`- people.debian.org/~aurel32 | www.aurel32.net

--
To UNSUBSCRIBE, email to debian-hurd-REQUEST DeleteThis @lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster DeleteThis @lists.debian.org

Aurelien Jarno · External Since: Nov 08, 2004 Posts: 527

Thomas Schwinge a écrit :
> Hello!
>
> This is not a proper fix, but a workaround that is needed on HEAD. (Have
> to check glibc-2_5-branch.) Reported on
> <http://savannah.gnu.org/bugs/?18218>.
>

This workaround is also necessary for the 2.5 branch. Applied in our SVN
as it seems harmless for other architectures.

--
.''`. Aurelien Jarno | GPG: 1024D/F1BCDB73
: :' : Debian developer | Electrical Engineer
`. `' aurel32 RemoveThis @debian.org | aurelien RemoveThis @aurel32.net
`- people.debian.org/~aurel32 | www.aurel32.net

--
To UNSUBSCRIBE, email to debian-hurd-REQUEST RemoveThis @lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster RemoveThis @lists.debian.org

Samuel Thibault · External Since: Nov 25, 2004 Posts: 499

Hi,

Aurelien Jarno, le Thu 08 Feb 2007 20:52:33 +0100, a écrit :
> sigsuspend_not_cancel is not defined for Hurd in
> sysdeps/mach/hurd/sigsuspend.c. This function is called from
> sysdeps/posix/pause.c.
>
> Any volunteer to implement it?

Mmm, it looks to me like the current __sigsuspend() doesn't handle
cancellation anyway (probably because libpthread on the Hurd is a
separate library). So something like the following could be just fine:

diff -u -p -r1.23 sigsuspend.c
--- sysdeps/mach/hurd/sigsuspend.c 3 Aug 2002 06:54:20 -0000 1.23
+++ sysdeps/mach/hurd/sigsuspend.c 11 Feb 2007 00:30:43 -0000
@@ -80,4 +80,5 @@ __sigsuspend (set)
return -1;
}
libc_hidden_def (__sigsuspend)
+strong_alias (__sigsuspend, __sigsuspend_nocancel)
weak_alias (__sigsuspend, sigsuspend)

Samuel

--
To UNSUBSCRIBE, email to debian-hurd-REQUEST.DeleteThis@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster.DeleteThis@lists.debian.org

Aurelien Jarno · External Since: Nov 08, 2004 Posts: 527

On Sun, Feb 11, 2007 at 01:31:35AM +0100, Samuel Thibault wrote:
> Hi,
>
> Aurelien Jarno, le Thu 08 Feb 2007 20:52:33 +0100, a écrit :
> > sigsuspend_not_cancel is not defined for Hurd in
> > sysdeps/mach/hurd/sigsuspend.c. This function is called from
> > sysdeps/posix/pause.c.
> >
> > Any volunteer to implement it?
>
> Mmm, it looks to me like the current __sigsuspend() doesn't handle
> cancellation anyway (probably because libpthread on the Hurd is a
> separate library). So something like the following could be just fine:

I have commited a slightly different patch, the name of the function was
not correct Smile

(see below)

Could you please forward it upstream?

With this patch the glibc 2.5 is now building correctly. An upload will
happen soon.

Thanks
Aurelien

--- sysdeps/mach/hurd/sigsuspend.c.orig 2007-02-12 02:54:19.000000000 +0100
+++ sysdeps/mach/hurd/sigsuspend.c 2007-02-12 02:54:35.000000000 +0100
@@ -80,4 +80,5 @@
return -1;
}
libc_hidden_def (__sigsuspend)
+strong_alias (__sigsuspend, sigsuspend_not_cancel)
weak_alias (__sigsuspend, sigsuspend)

--
.''`. Aurelien Jarno | GPG: 1024D/F1BCDB73
: :' : Debian developer | Electrical Engineer
`. `' aurel32 DeleteThis @debian.org | aurelien DeleteThis @aurel32.net
`- people.debian.org/~aurel32 | www.aurel32.net

--
To UNSUBSCRIBE, email to debian-hurd-REQUEST DeleteThis @lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster DeleteThis @lists.debian.org