Matthias Klose | 2 Jul 00:44
Picon

Re: glibc is broken because of gcc

The proposed fix seems to be wrong, as it disables the backport on all
architectures.  Please could you recheck with current trunk? As jda
suggested, another backport is missing for hppa.

  Matthias

Sébastien Bernard writes:
> Ok, I nailed it.
> I removed PR20218 patch and generated a 4.1.2-12+b1 then build glibc.
> 
> Glibc build is ok now.
> I think the hppa toolchain can move on and build the glibc-2.5-11 and 
> then rebuild the portmap and all pie executable (pfeww... what a bug chain).
> 
> 	Seb
> 
> diff -r -u -b -B -w gcc-4.1-4.1.2-12/debian/changelog gcc-4.1-4.1.2-12+b1/debian/changelog
> --- gcc-4.1-4.1.2-12/debian/changelog	2007-06-13 22:34:38.000000000 +0200
> +++ gcc-4.1-4.1.2-12+b1/debian/changelog	2007-06-13 12:44:48.000000000 +0200
> @@ -1,3 +1,9 @@
> +gcc-4.1 (4.1.2-12+b1) unstable; urgency=low
> +
> +  * Revert 20218 patch that breaks gcc
> +
> + -- Sebastien Bernard <seb <at> frankengul.org>  Wed, 13 Jun 2007 12:44:07 +0200
> +
>  gcc-4.1 (4.1.2-12) unstable; urgency=high
>  
>    * i386-biarch.dpatch: Update for the backport for PR target/31868.
> diff -r -u -b -B -w gcc-4.1-4.1.2-12/debian/rules.patch gcc-4.1-4.1.2-12+b1/debian/rules.patch
(Continue reading)

Matthias Klose | 2 Jul 18:29
Picon

GCC trunk 20070630

Testing 20070630 including the fix for PR rtl-optimization/32296 shows
many new C++ and libstdc++ regressions.

A Debian package for unstable will be available tonight in the archives.

  Matthias

Results for 4.3.0 20070630 (experimental) testsuite on hppa-unknown-linux-gnu
LAST_UPDATED: 
Target: hppa-linux-gnu
gcc version 4.3.0 20070630 (experimental)
Native configuration is hppa-unknown-linux-gnu

		=== g++ tests ===

Running target unix
FAIL: g++.dg/compat/eh/ctor1 cp_compat_x_tst.o-cp_compat_y_tst.o execute 
FAIL: g++.dg/compat/eh/new1 cp_compat_x_tst.o-cp_compat_y_tst.o execute 
FAIL: g++.dg/compat/eh/template1 cp_compat_x_tst.o-cp_compat_y_tst.o execute 
WARNING: program timed out.
FAIL: g++.dg/compat/eh/unexpected1 cp_compat_x_tst.o-cp_compat_y_tst.o execute 
FAIL: g++.dg/compat/init/array5 cp_compat_x_tst.o-cp_compat_y_tst.o execute 
FAIL: tmpdir-g++.dg-struct-layout-1/t028 cp_compat_y_tst.o compile,  (internal compiler error)
UNRESOLVED: tmpdir-g++.dg-struct-layout-1/t028 cp_compat_x_tst.o-cp_compat_y_tst.o link 
UNRESOLVED: tmpdir-g++.dg-struct-layout-1/t028 cp_compat_x_tst.o-cp_compat_y_tst.o execute 
FAIL: g++.dg/cpp0x/variadic73.C execution test
FAIL: g++.dg/eh/alias1.C execution test
FAIL: g++.dg/eh/cond1.C execution test
FAIL: g++.dg/eh/ctor1.C execution test
FAIL: g++.dg/eh/elide1.C execution test
(Continue reading)

John David Anglin | 2 Jul 18:53
Picon

Re: GCC trunk 20070630

> Testing 20070630 including the fix for PR rtl-optimization/32296 shows
> many new C++ and libstdc++ regressions.

Most of these were introduced by the dataflow merge (r125624).  I'm
currently working to confirm this.

The C++ and libstdc++ regressions are an EH bug in libgcc.  I tried
debugging this yesterday but didn't find the problem.  I know the
problems in libgcc since the test that I was debugging doesn't fail
with the system libgcc.

We also have problems with building hppa64 and Ada.  So, things are
a mess at the moment...

Dave
--

-- 
J. David Anglin                                  dave.anglin <at> nrc-cnrc.gc.ca
National Research Council of Canada              (613) 990-0752 (FAX: 952-6602)
Gravatar

Re: [parisc-linux] glibc is broken because of gcc

Matthias Klose a écrit :
> The proposed fix seems to be wrong, as it disables the backport on all
> architectures.  Please could you recheck with current trunk? As jda
> suggested, another backport is missing for hppa.
> 
>   Matthias
> 

Oh well, I didn't say I got it all right.

I just said the problem was there.
I submited the bug to the gcc bugtracker and to the debian gcc, but I
didn't submit this patch as a potential fix.
There was no reaction on either site.

If you watch carefuly, this patch is not applied against some archs :
-ifneq (,$(filter $(DEB_TARGET_ARCH), amd64 i386 powerpc ppc64 sparc s390))
-  debian_patches += pr20218
-endif
-
So I don't do much more than what is already on the debian version.

I'm willing add the new patch if one can direct me to the location of 
this famous complementary patch.

For now, I think reverting this patch is doing more good than harm.

But I'm afraid that tracking the bug and understanding what is really 
going on in this bug is quite beyond my capacities. I'm just doing trial 
and errors to identify the problematic patch.
(Continue reading)

John David Anglin | 2 Jul 23:28
Picon

Re: glibc is broken because of gcc

> Matthias Klose a écrit :
> > The proposed fix seems to be wrong, as it disables the backport on all
> > architectures.  Please could you recheck with current trunk? As jda
> > suggested, another backport is missing for hppa.

Looking at this some more, I don't think there's a missing backport for hppa.

> There was no reaction on either site.

The comment was too vague and will be ignored since you are modifying
the gcc tree.  If a gcc bug is identified, then a new gcc PR is needed.

Looking at http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=428509,
I don't see that gcc is doing anything wrong.  Currently, I see the
following with readelf for __librt_multiple_threads in librt-2.5.so
and libc-2.5.so on testing, respectively:

    44: 00000000     4 OBJECT  GLOBAL DEFAULT  UND __librt_multiple_threads <at> GLIBC_PRIVATE (10)

  1230: 0013e328     4 OBJECT  GLOBAL DEFAULT   32 __librt_multiple_threads@@GLIBC_PRIVATE

The link error suggests that __librt_multiple_threads is hidden in libc-2.5.so.
This will only occur if the symbol is defined with the .hidden attribute.
If that's the case, the link error is expected.  So, either the symbol
shouldn't be hidden or it needs to be defined in librt as well.  This is
a glibc issue.

I should note that pa gcc backend doesn't do anything special wrt
.hidden.  .hidden support is handled completely by the middle-end.
Thus, any problem wrt .hidden in gcc is likely to apply to all targets
(Continue reading)

Gravatar

Re: glibc is broken because of gcc

John David Anglin a écrit :
> Looking at this some more, I don't think there's a missing backport
> for hppa.
> 
>> There was no reaction on either site.
> 
> The comment was too vague and will be ignored since you are modifying
>  the gcc tree.  If a gcc bug is identified, then a new gcc PR is
> needed.

Well, it was a pr on a backport. I though  it was appropriate to add the
problem to the long thread.

> 
> Looking at http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=428509, I
> don't see that gcc is doing anything wrong.  Currently, I see the 
> following with readelf for __librt_multiple_threads in librt-2.5.so 
> and libc-2.5.so on testing, respectively:
> 
> 44: 00000000     4 OBJECT  GLOBAL DEFAULT  UND
> __librt_multiple_threads <at> GLIBC_PRIVATE (10)
> 
> 1230: 0013e328     4 OBJECT  GLOBAL DEFAULT   32
> __librt_multiple_threads@@GLIBC_PRIVATE
> 
> The link error suggests that __librt_multiple_threads is hidden in
> libc-2.5.so. This will only occur if the symbol is defined with the
> .hidden attribute. If that's the case, the link error is expected.
> So, either the symbol shouldn't be hidden or it needs to be defined
> in librt as well.  This is a glibc issue.
(Continue reading)

John David Anglin | 3 Jul 03:58
Picon

Re: glibc is broken because of gcc

> Here is the readelf of the broken build :
> seb <at> hpnux:~/glibc-2.5/build-tree/hppa-libc/rt$ readelf -a ./libc.so 
> |grep multiple_threads
>    1230: 0013e328     4 OBJECT  GLOBAL DEFAULT   32
> __librt_multiple_threads@@GLIBC_PRIVATE
>    2737: 0013e328     4 OBJECT  LOCAL  HIDDEN   32 __libc_multiple_threads
>    3961: 0013e328     4 OBJECT  GLOBAL DEFAULT   32 __librt_multiple_threads
> 
> seb <at> hpnux:~/dev/glibc/glibc-2.5/build-tree/hppa-libc/rt$ readelf -a
> librt_pic.a |grep multiple_threads
>      18: 00000000     0 NOTYPE  GLOBAL HIDDEN  UND __librt_multiple_threads
>       6: 00000000     0 NOTYPE  GLOBAL DEFAULT  UND __librt_multiple_threads
>       6: 00000000     0 NOTYPE  GLOBAL DEFAULT  UND __librt_multiple_threads
> 
> I don't see anything wrong. Moreover, the readelf on the correct library 
> gives exactly the same result.

The error in bug 428509
is:

...
/home/seb/dev/glibc-2.5/build-tree/hppa-libc/rt/librt_pic.a(mq_timedreceive.os):(.text+0x20):
more undefined references to `__librt_multiple_threads' follow
/usr/bin/ld: /home/seb/dev/glibc-2.5/build-tree/hppa-libc/rt/librt.so: hidden symbol
`__librt_multiple_threads' isn't defined
/usr/bin/ld: final link failed: Nonrepresentable section on output

Looking at gcc PR 20218, it's clear that there are some very subtle
issues here.  So, it's not clear to me where the problem lies.  It
could be in glibc (hppa sysdep-cancel.h), binutils or gcc.  The handling
(Continue reading)

Matthias Klose | 3 Jul 11:58
Picon

Re: GCC trunk 20070630

John David Anglin writes:
> > Testing 20070630 including the fix for PR rtl-optimization/32296 shows
> > many new C++ and libstdc++ regressions.
> 
> We also have problems with building hppa64 and Ada.  So, things are
> a mess at the moment...

The hppa64 cross compiler at least did build; Ada is broken on other
archs as well (at least 20070702).
John David Anglin | 3 Jul 15:48
Picon

Re: GCC trunk 20070630

> The hppa64 cross compiler at least did build;

Since the dataflow merge, the hppa64-hp-hpux11.11 build fails at
the start of stage3.  More info is available in PR middle-end/32398.
The ptr-plus merge also has some affect.  After the pointer plus
merge, there's a small test case in the PR that's miscompiled
with the stage1 compiler.

Dave
--

-- 
J. David Anglin                                  dave.anglin <at> nrc-cnrc.gc.ca
National Research Council of Canada              (613) 990-0752 (FAX: 952-6602)
Peter Zijlstra | 3 Jul 17:00
Picon

Re: [patch 2/3] audit: rework execve audit

On Tue, 2007-06-26 at 15:55 -0700, Andrew Morton wrote:
> On Wed, 13 Jun 2007 12:03:36 +0200
> Peter Zijlstra <a.p.zijlstra <at> chello.nl> wrote:
> 
> > +#ifdef CONFIG_AUDITSYSCALL
> > +	{
> > +		.ctl_name	= CTL_UNNUMBERED,
> > +		.procname	= "audit_argv_kb",
> > +		.data		= &audit_argv_kb,
> > +		.maxlen		= sizeof(int),
> > +		.mode		= 0644,
> > +		.proc_handler	= &proc_dointvec,
> > +	},
> > +#endif
> 
> Please document /proc entries in Documentation/filesystems/proc.txt

Signed-off-by: Peter Zijlstra <a.p.zijlstra <at> chello.nl>
---
 Documentation/filesystems/proc.txt |    7 +++++++
 1 file changed, 7 insertions(+)

Index: linux-2.6/Documentation/filesystems/proc.txt
===================================================================
--- linux-2.6.orig/Documentation/filesystems/proc.txt
+++ linux-2.6/Documentation/filesystems/proc.txt
@@ -1075,6 +1075,13 @@ check the amount of free space (value is
 resume it  if we have a value of 3 or more percent; consider information about
 the amount of free space valid for 30 seconds

(Continue reading)


Gmane