[Bug gold/12525] gold SEGV linking libgcc_s.so.1 on Solaris 11/x86

2011-07-01 Thread ian at airs dot com
http://sourceware.org/bugzilla/show_bug.cgi?id=12525

Ian Lance Taylor  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED

--- Comment #31 from Ian Lance Taylor  2011-07-02 05:35:23 
UTC ---
As far as I can tell, with the patch I just committed, all the issues in this
bug report should now be fixed.  In any case this report has gotten rather
complicated.  If there are any remaining issues on Solaris, please open a new
bug report.  Thanks.

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug gold/12952] Cannot compile ffmpeg with gold on FreeBSD 8 64bit

2011-07-01 Thread ian at airs dot com
http://sourceware.org/bugzilla/show_bug.cgi?id=12952

Ian Lance Taylor  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED

--- Comment #10 from Ian Lance Taylor  2011-07-02 05:33:45 
UTC ---
Thanks for the offer, but actually I may have found the problem anyhow.  I'm
going to close this bug report.  If you have a chance, please try the build
with the current development version of gold, and reopen this report if it
still fails.

Thanks for reporting the bug.

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug gold/12525] gold SEGV linking libgcc_s.so.1 on Solaris 11/x86

2011-07-01 Thread cvs-commit at gcc dot gnu.org
http://sourceware.org/bugzilla/show_bug.cgi?id=12525

--- Comment #30 from cvs-commit at gcc dot gnu.org  2011-07-02 05:30:08 UTC ---
CVSROOT:/cvs/src
Module name:src
Changes by:i...@sourceware.org2011-07-02 05:30:00

Modified files:
gold   : ChangeLog dynobj.cc dynobj.h resolve.cc 
gold/testsuite : Makefile.am Makefile.in weak_alias_test_main.cc 
Added files:
gold/testsuite : weak_alias_test.script weak_alias_test_5.cc 

Log message:
PR gold/12525
PR gold/12952
* resolve.cc (Symbol::override_base_with_special): Don't override
the version if the overriding symbol has a different name.
* dynobj.cc (Versions::add_def): Add dynpool parameter.  Change
all callers.  If we give an error about an undefined version,
define the base version if necessary.
* dynobj.h (class Versions): Update declaration.
* testsuite/weak_alias_test_5.cc: New file.
* testsuite/weak_alias_test.script: New file.
* testsuite/weak_alias_test_main.cc: Check that versioned_symbol
and versioned_alias have the right value, and call t2.
* testsuite/Makefile.am (weak_alias_test_DEPENDENCIES): Add
weak_alias_test_5.so.
(weak_alias_test_LDADD): Likewise.
(weak_alias_test_5_pic.o, weak_alias_test_5.so): New targets.
* testsuite/Makefile.in: Rebuild.

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/ChangeLog.diff?cvsroot=src&r1=1.783&r2=1.784
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/dynobj.cc.diff?cvsroot=src&r1=1.62&r2=1.63
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/dynobj.h.diff?cvsroot=src&r1=1.45&r2=1.46
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/resolve.cc.diff?cvsroot=src&r1=1.60&r2=1.61
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/testsuite/weak_alias_test.script.diff?cvsroot=src&r1=NONE&r2=1.1
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/testsuite/weak_alias_test_5.cc.diff?cvsroot=src&r1=NONE&r2=1.1
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/testsuite/Makefile.am.diff?cvsroot=src&r1=1.170&r2=1.171
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/testsuite/Makefile.in.diff?cvsroot=src&r1=1.179&r2=1.180
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/testsuite/weak_alias_test_main.cc.diff?cvsroot=src&r1=1.1&r2=1.2

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug gold/12952] Cannot compile ffmpeg with gold on FreeBSD 8 64bit

2011-07-01 Thread cvs-commit at gcc dot gnu.org
http://sourceware.org/bugzilla/show_bug.cgi?id=12952

--- Comment #9 from cvs-commit at gcc dot gnu.org  2011-07-02 05:30:08 UTC ---
CVSROOT:/cvs/src
Module name:src
Changes by:i...@sourceware.org2011-07-02 05:30:00

Modified files:
gold   : ChangeLog dynobj.cc dynobj.h resolve.cc 
gold/testsuite : Makefile.am Makefile.in weak_alias_test_main.cc 
Added files:
gold/testsuite : weak_alias_test.script weak_alias_test_5.cc 

Log message:
PR gold/12525
PR gold/12952
* resolve.cc (Symbol::override_base_with_special): Don't override
the version if the overriding symbol has a different name.
* dynobj.cc (Versions::add_def): Add dynpool parameter.  Change
all callers.  If we give an error about an undefined version,
define the base version if necessary.
* dynobj.h (class Versions): Update declaration.
* testsuite/weak_alias_test_5.cc: New file.
* testsuite/weak_alias_test.script: New file.
* testsuite/weak_alias_test_main.cc: Check that versioned_symbol
and versioned_alias have the right value, and call t2.
* testsuite/Makefile.am (weak_alias_test_DEPENDENCIES): Add
weak_alias_test_5.so.
(weak_alias_test_LDADD): Likewise.
(weak_alias_test_5_pic.o, weak_alias_test_5.so): New targets.
* testsuite/Makefile.in: Rebuild.

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/ChangeLog.diff?cvsroot=src&r1=1.783&r2=1.784
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/dynobj.cc.diff?cvsroot=src&r1=1.62&r2=1.63
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/dynobj.h.diff?cvsroot=src&r1=1.45&r2=1.46
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/resolve.cc.diff?cvsroot=src&r1=1.60&r2=1.61
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/testsuite/weak_alias_test.script.diff?cvsroot=src&r1=NONE&r2=1.1
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/testsuite/weak_alias_test_5.cc.diff?cvsroot=src&r1=NONE&r2=1.1
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/testsuite/Makefile.am.diff?cvsroot=src&r1=1.170&r2=1.171
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/testsuite/Makefile.in.diff?cvsroot=src&r1=1.179&r2=1.180
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/testsuite/weak_alias_test_main.cc.diff?cvsroot=src&r1=1.1&r2=1.2

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug gold/12952] Cannot compile ffmpeg with gold on FreeBSD 8 64bit

2011-07-01 Thread website-sourceware at interfasys dot ch
http://sourceware.org/bugzilla/show_bug.cgi?id=12952

--- Comment #8 from interfaSys sàrl  
2011-07-02 01:01:49 UTC ---
Would that help if I provided you with an archive containing all the libs?

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug gold/12525] gold SEGV linking libgcc_s.so.1 on Solaris 11/x86

2011-07-01 Thread ian at airs dot com
http://sourceware.org/bugzilla/show_bug.cgi?id=12525

--- Comment #29 from Ian Lance Taylor  2011-07-02 00:43:51 
UTC ---
The failure of two_file_shared_1_nonpic.so is not very interesting.  It happens
because on Solaris when gcc sees -shared it passes -z text to the linker.  GNU
ld does not support -z text and simply ignores it, as it ignores all
unrecognized -z options.  The gold linker does support -z text, which means to
not permit dynamic relocations in the text segment.  This test is specifically
testing that that works, so naturally it fails.

I just committed a patch to support -z notext, and to use it when appropriate,
so this test should no longer fail on Solaris.

I also committed a patch to avoid testing with -static in cases where it does
not work.

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug gold/12525] gold SEGV linking libgcc_s.so.1 on Solaris 11/x86

2011-07-01 Thread cvs-commit at gcc dot gnu.org
http://sourceware.org/bugzilla/show_bug.cgi?id=12525

--- Comment #28 from cvs-commit at gcc dot gnu.org  2011-07-02 00:39:15 UTC ---
CVSROOT:/cvs/src
Module name:src
Changes by:i...@sourceware.org2011-07-02 00:39:13

Modified files:
gold   : ChangeLog options.h 
gold/testsuite : Makefile.am Makefile.in 

Log message:
PR gold/12525
* options.h (class General_options): Support -z notext.
* testsuite/Makefile.am (two_file_shared_1_nonpic.so): Use
-Wl,-z,notext.
(two_file_shared_nonpic.so): Likewise.
(two_file_shared_mixed.so): Likewise.
(two_file_shared_mixed_1.so): Likewise.
(weak_undef_lib_nonpic.so): Likewise.
(alt/weak_undef_lib_nonpic.so): Likewise.
(tls_test_shared_nonpic.so): Likewise.
* testsuite/Makefile.in: Rebuild.

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/ChangeLog.diff?cvsroot=src&r1=1.782&r2=1.783
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/options.h.diff?cvsroot=src&r1=1.162&r2=1.163
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/testsuite/Makefile.am.diff?cvsroot=src&r1=1.169&r2=1.170
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/testsuite/Makefile.in.diff?cvsroot=src&r1=1.178&r2=1.179

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug gold/12525] gold SEGV linking libgcc_s.so.1 on Solaris 11/x86

2011-07-01 Thread cvs-commit at gcc dot gnu.org
http://sourceware.org/bugzilla/show_bug.cgi?id=12525

--- Comment #27 from cvs-commit at gcc dot gnu.org  2011-07-02 00:19:06 UTC ---
CVSROOT:/cvs/src
Module name:src
Changes by:i...@sourceware.org2011-07-02 00:19:04

Modified files:
gold   : ChangeLog configure configure.ac 
gold/testsuite : Makefile.am Makefile.in 

Log message:
PR gold/12525
* configure.ac: Test whether static linking works, setting
the automake conditional HAVE_STATIC.
* testsuite/Makefile.am: Disable tests using -static if
HAVE_STATIC is not true.
* configure, testsuite/Makefile.in: Rebuild.

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/ChangeLog.diff?cvsroot=src&r1=1.781&r2=1.782
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/configure.diff?cvsroot=src&r1=1.66&r2=1.67
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/configure.ac.diff?cvsroot=src&r1=1.63&r2=1.64
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/testsuite/Makefile.am.diff?cvsroot=src&r1=1.168&r2=1.169
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/testsuite/Makefile.in.diff?cvsroot=src&r1=1.177&r2=1.178

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug gold/12525] gold SEGV linking libgcc_s.so.1 on Solaris 11/x86

2011-07-01 Thread cvs-commit at gcc dot gnu.org
http://sourceware.org/bugzilla/show_bug.cgi?id=12525

--- Comment #26 from cvs-commit at gcc dot gnu.org  2011-07-02 00:03:28 UTC ---
CVSROOT:/cvs/src
Module name:src
Changes by:i...@sourceware.org2011-07-02 00:03:25

Modified files:
gold   : ChangeLog ehframe.cc i386.cc target.h x86_64.cc 

Log message:
PR gold/12525
* ehframe.cc (Eh_frame_hdr::get_fde_pc): Handle DW_EH_PE_datarel.
Assert if we see DW_EH_PE_indirect.
* target.h (Target::ehframe_datarel_base): New function.
(Target::do_ehframe_datarel_base): New target function.
* i386.cc (Target_i386::do_ehframe_datarel_base): New function.
* x86_64.cc (Target_x86_64::do_ehframe_datarel_base): New
function.

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/ChangeLog.diff?cvsroot=src&r1=1.780&r2=1.781
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/ehframe.cc.diff?cvsroot=src&r1=1.20&r2=1.21
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/i386.cc.diff?cvsroot=src&r1=1.133&r2=1.134
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/target.h.diff?cvsroot=src&r1=1.60&r2=1.61
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/x86_64.cc.diff?cvsroot=src&r1=1.131&r2=1.132

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug gold/12952] Cannot compile ffmpeg with gold on FreeBSD 8 64bit

2011-07-01 Thread ian at airs dot com
http://sourceware.org/bugzilla/show_bug.cgi?id=12952

--- Comment #7 from Ian Lance Taylor  2011-07-01 23:47:51 
UTC ---
Thanks for the test case.  Unfortunately, I am missing a lot of the input
libraries:

../gold/ld-new: error: cannot find -lavcodec
../gold/ld-new: error: cannot find -lavutil
../gold/ld-new: error: cannot find -lxvidcore
../gold/ld-new: error: cannot find -lx264
../gold/ld-new: error: cannot find -lvpx
../gold/ld-new: error: cannot find -lvpx
../gold/ld-new: error: cannot find -lvorbisenc
../gold/ld-new: error: cannot find -lvorbis
../gold/ld-new: error: cannot find -logg
../gold/ld-new: error: cannot find -lvo-amrwbenc
../gold/ld-new: error: cannot find -lvo-aacenc
../gold/ld-new: error: cannot find -ltheoraenc
../gold/ld-new: error: cannot find -ltheoradec
../gold/ld-new: error: cannot find -logg
../gold/ld-new: error: cannot find -lschroedinger-1.0
../gold/ld-new: error: cannot find -lopencv_core
../gold/ld-new: error: cannot find -lopencv_imgproc
../gold/ld-new: error: cannot find -lopencv_video
../gold/ld-new: error: cannot find -lopencv_flann
../gold/ld-new: error: cannot find -lopencore-amrwb
../gold/ld-new: error: cannot find -lopencore-amrnb
../gold/ld-new: error: cannot find -lmp3lame
../gold/ld-new: error: cannot find -lgsm
../gold/ld-new: error: cannot find -lfaac
../gold/ld-new: error: cannot find -lbz2

Since this is a -shared link I can do the link without them, but then it
succeeds.

In order to fix this bug, I will need to have some way to reproduce it in
isolation.

I suppose it must have something to do with the version script somehow getting
confused by the versions in the shared libraries included in the link, but I
don't see what it could be.

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug ld/12565] NOLOAD sections empty

2011-07-01 Thread jifl-bugzilla at jifvik dot org
http://sourceware.org/bugzilla/show_bug.cgi?id=12565

--- Comment #20 from Jonathan Larmour  
2011-07-01 23:22:43 UTC ---
Hi Nicl,

(In reply to comment #16) 
> >
> > It is documented in ld.texinfo in the 'Output Section Type' node:
> > @item NOLOAD
> > The section should be marked as not loadable, so that it will not be
> > loaded into memory when the program is run.
> 
> But that does not document the fact that the NOLOAD section will also be 
> without any contents.  This was the point that I was trying to get at.

Sorry I had thought you were asking whether there was any documentation on
NOLOAD at all. Evidently it was documented as including contents up to and
including 2.20.1. As per Alan's changes described in comment #14, it is only
then that what it said about contents was removed, becoming its current more
ambiguous state. Here's what it additionally said in 2.20.1:
-=-=-=-=-=-=-=-=-
   The linker normally sets the attributes of an output section based on
the input sections which map into it.  You can override this by using
the section type.  For example, in the script sample below, the `ROM'
section is addressed at memory location `0' and does not need to be
loaded when the program is run.  The contents of the `ROM' section will
appear in the linker output file as usual.
 SECTIONS {
   ROM 0 (NOLOAD) : { ... }
   ...
 }
-=-=-=-=-=-=-=-=-
The last sentence is the most relevant. Practice in the ldscripts of libgloss
and elsewhere fits with that.

Just to reiterate, if you want to replace NOLOAD with something which strips
contents, personally I think I should be able to deal with that in other ways.
But please don't keep NOLOAD but change its meaning from pre-2.21. That makes
problems harder to find. New behaviour should use a new name e.g. ALLOCONLY or
something.

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug gold/12571] .plt has no associated .eh_frame/.debug_frame

2011-07-01 Thread ian at airs dot com
http://sourceware.org/bugzilla/show_bug.cgi?id=12571

Ian Lance Taylor  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED

--- Comment #2 from Ian Lance Taylor  2011-07-01 22:07:13 
UTC ---
Fixed in gold.

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug gold/12571] .plt has no associated .eh_frame/.debug_frame

2011-07-01 Thread cvs-commit at gcc dot gnu.org
http://sourceware.org/bugzilla/show_bug.cgi?id=12571

--- Comment #1 from cvs-commit at gcc dot gnu.org  2011-07-01 22:05:06 UTC ---
CVSROOT:/cvs/src
Module name:src
Changes by:i...@sourceware.org2011-07-01 22:05:01

Modified files:
gold   : ChangeLog ehframe.cc ehframe.h i386.cc 
 layout.cc layout.h merge.cc options.h x86_64.cc 

Log message:
PR gold/12571
* options.h (class General_options): Add
--ld-generated-unwind-info.
* ehframe.cc (Fde::write): Add address parameter.  Change all
callers.  If associated with PLT, fill in address and size.
(Cie::set_output_offset): Only add merge mapping if there is an
object.
(Cie::write): Add address parameter.  Change all callers.
(Eh_frame::add_ehframe_for_plt): New function.
* ehframe.h (class Fde): Update declarations.  Move shndx_ and
input_offset_ fields into union u_, with new plt field.
(Fde::Fde): Adjust for new union field.
(Fde::Fde) [Output_data version]: New constructor.
(Fde::add_mapping): Only add merge mapping if there is an object.
(class Cie): Update declarations.
(class Eh_frame): Declare add_ehframe_for_plt.
* layout.cc (Layout::layout_eh_frame): Break out code into
make_eh_frame_section, and call it.
(Layout::make_eh_frame_section): New function.
(Layout::add_eh_frame_for_plt): New function.
* layout.h (class Layout): Update declarations.
* merge.cc (Merge_map::add_mapping): Add assertion.
* i386.cc: Include "dwarf.h".
(class Output_data_plt_i386): Make first_plt_entry,
dyn_first_plt_entry, exec_plt_entry, and dyn_plt_entry const.  Add
plt_eh_frame_cie_size, plt_eh_frame_fde_size, plt_eh_frame_cie,
and plt_eh_frame_fde.
(Output_data_plt_i386::Output_data_plt_i386): Align to 16-byte
boundary.  Call add_eh_frame_for_plt if appropriate.
* x86_64.cc: Include "dwarf.h".
(class Output_data_plt_x86_64): Align to 16-byte boundary.  Make
first_plt_entry, plt_entry and tlsdesc_plt_entry const.  Add
plt_eh_frame_cie_size, plt_eh_frame_fde_size, plt_eh_frame_cie,
and plt_eh_frame_fde.
(Output_data_plt_x86_64::init): Call add_eh_frame_for_plt if
appropriate.

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/ChangeLog.diff?cvsroot=src&r1=1.779&r2=1.780
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/ehframe.cc.diff?cvsroot=src&r1=1.19&r2=1.20
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/ehframe.h.diff?cvsroot=src&r1=1.16&r2=1.17
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/i386.cc.diff?cvsroot=src&r1=1.131&r2=1.132
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/layout.cc.diff?cvsroot=src&r1=1.209&r2=1.210
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/layout.h.diff?cvsroot=src&r1=1.96&r2=1.97
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/merge.cc.diff?cvsroot=src&r1=1.39&r2=1.40
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/options.h.diff?cvsroot=src&r1=1.161&r2=1.162
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gold/x86_64.cc.diff?cvsroot=src&r1=1.130&r2=1.131

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug binutils/12329] avr-objdump -d: wrong op-code

2011-07-01 Thread nickc at redhat dot com
http://sourceware.org/bugzilla/show_bug.cgi?id=12329

Nick Clifton  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 CC||nickc at redhat dot com
 Resolution||FIXED

--- Comment #2 from Nick Clifton  2011-07-01 16:14:00 
UTC ---
Hi Jesús,

  Thanks for reporting this problem.  I have checked in a patch to fix the
disassembly of the LPM instruction, and incidentally the ELPM and SPM
instructions which had the same problem.

Cheers
  Nick

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug binutils/12329] avr-objdump -d: wrong op-code

2011-07-01 Thread cvs-commit at gcc dot gnu.org
http://sourceware.org/bugzilla/show_bug.cgi?id=12329

--- Comment #1 from cvs-commit at gcc dot gnu.org  2011-07-01 16:11:32 UTC ---
CVSROOT:/cvs/src
Module name:src
Changes by:ni...@sourceware.org2011-07-01 16:11:27

Modified files:
opcodes: ChangeLog arm-dis.c avr-dis.c 
include/opcode : ChangeLog avr.h 

Log message:
PR binutils/12329
* avr-dis.c (avr_operand): Fix disassembly of ELPM, LPM and SPM
insns using post-increment addressing.

* avr.h (AVR_ISA_AVR6): Fix typo, adding AVR_ISA_SPMX.

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/src/opcodes/ChangeLog.diff?cvsroot=src&r1=1.1740&r2=1.1741
http://sourceware.org/cgi-bin/cvsweb.cgi/src/opcodes/arm-dis.c.diff?cvsroot=src&r1=1.146&r2=1.147
http://sourceware.org/cgi-bin/cvsweb.cgi/src/opcodes/avr-dis.c.diff?cvsroot=src&r1=1.23&r2=1.24
http://sourceware.org/cgi-bin/cvsweb.cgi/src/include/opcode/ChangeLog.diff?cvsroot=src&r1=1.441&r2=1.442
http://sourceware.org/cgi-bin/cvsweb.cgi/src/include/opcode/avr.h.diff?cvsroot=src&r1=1.16&r2=1.17

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug binutils/12325] ar --target isn't documented

2011-07-01 Thread cvs-commit at gcc dot gnu.org
http://sourceware.org/bugzilla/show_bug.cgi?id=12325

--- Comment #1 from cvs-commit at gcc dot gnu.org  2011-07-01 15:02:26 UTC ---
CVSROOT:/cvs/src
Module name:src
Changes by:ni...@sourceware.org2011-07-01 15:02:22

Modified files:
binutils   : ChangeLog 
binutils/doc   : binutils.texi 

Log message:
PR binutils/12325
* doc/binutils.texi (ar cmdline): Document --target, --version and
--help command line options.

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/src/binutils/ChangeLog.diff?cvsroot=src&r1=1.1815&r2=1.1816
http://sourceware.org/cgi-bin/cvsweb.cgi/src/binutils/doc/binutils.texi.diff?cvsroot=src&r1=1.179&r2=1.180

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug binutils/12325] ar --target isn't documented

2011-07-01 Thread nickc at redhat dot com
http://sourceware.org/bugzilla/show_bug.cgi?id=12325

Nick Clifton  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 CC||nickc at redhat dot com
 Resolution||FIXED

--- Comment #2 from Nick Clifton  2011-07-01 15:04:48 
UTC ---
Hi H.J.

  The --help output does include a mention of --target, and the selection of
targets that it supports.  You were correct however in noting that the
documentation did not mention this option.  I have checked in a patch to
address this failing, and at the same time include a mention of the --help and
--version command line options.

Cheers
  Nick

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug ld/12565] NOLOAD sections empty

2011-07-01 Thread nickc at redhat dot com
http://sourceware.org/bugzilla/show_bug.cgi?id=12565

--- Comment #18 from Nick Clifton  2011-07-01 13:10:20 
UTC ---
Hi Ian,

> If you look in the libgloss linker scripts you will see a bunch of uses
> of NOLOAD.
>
> I'm not sure whether any of them are really necessary, but they are
> certainly there.

Apart from a few .bss sections and one target which uses them to get the 
addresses of its hardware I/O ports, all of the rest are for debugging 
sections.  Which brings us back to the original point of this PR.  In 
theory it makes sense to mark debug sections as NOLOAD, since you do not 
need them in the running executable.  But, for ELF based targets, this 
must not be done because NOLOAD will also obliterate their contents on disk.

Maybe we need a new ELF section type, eg SHT_GNU_NOLOAD, which can be 
used to mark sections which *do* have contents on disk but which should 
not be loaded into running memory.  Or maybe we could use SHT_NOTE instead.

At the moment the linker identifies debugging sections by their name and 
arranges for them to be excluded from any loadable segments.  But it 
would be much cleaner if we could just examine the section type and if 
it was SHT_GNU_NOLOAD then decide not to map it to a loadable segment.

Cheers
   Nick

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


Re: [Bug ld/12565] NOLOAD sections empty

2011-07-01 Thread Nick Clifton

Hi Ian,


If you look in the libgloss linker scripts you will see a bunch of uses
of NOLOAD.

I'm not sure whether any of them are really necessary, but they are
certainly there.


Apart from a few .bss sections and one target which uses them to get the 
addresses of its hardware I/O ports, all of the rest are for debugging 
sections.  Which brings us back to the original point of this PR.  In 
theory it makes sense to mark debug sections as NOLOAD, since you do not 
need them in the running executable.  But, for ELF based targets, this 
must not be done because NOLOAD will also obliterate their contents on disk.


Maybe we need a new ELF section type, eg SHT_GNU_NOLOAD, which can be 
used to mark sections which *do* have contents on disk but which should 
not be loaded into running memory.  Or maybe we could use SHT_NOTE instead.


At the moment the linker identifies debugging sections by their name and 
arranges for them to be excluded from any loadable segments.  But it 
would be much cleaner if we could just examine the section type and if 
it was SHT_GNU_NOLOAD then decide not to map it to a loadable segment.


Cheers
  Nick



___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug ld/12565] NOLOAD sections empty

2011-07-01 Thread nickc at redhat dot com
http://sourceware.org/bugzilla/show_bug.cgi?id=12565

--- Comment #19 from Nick Clifton  2011-07-01 13:14:34 
UTC ---
Hi Doug,

> The ARM linux kernel also uses NOLOAD.

Do you know if this entirely for .bss style sections, or maybe for 
establishing a region of memory mapped hardware or I/O ports ?

The point I am interested in is if we changed the current behaviour of 
NOLOAD so that it *did* preserve the contents of any input sections (if 
those contents were non-zero) would this break anything ?  I am pretty 
certain that it would, so I am not going to create a patch to make this 
change.  But I do wonder if there any applications out there that are 
relying upon NOLOAD to actually get rid of the contents of input sections.

Cheers
   Nick

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug ld/12565] NOLOAD sections empty

2011-07-01 Thread nickc at redhat dot com
http://sourceware.org/bugzilla/show_bug.cgi?id=12565

--- Comment #17 from Nick Clifton  2011-07-01 12:52:14 
UTC ---
Hi H.J.

>> Also, as far as I can see, this behaviour is not documented anywhere.
>> Do you know of any applications that rely upon this feature ?
>
> NOLOAD means "don't load into memory", which maps to NOBITS.
> Otherwise, it will be loaded into memory.

OK, I now get that NOLOAD is effectively a marker for .bss type sections.

But it does seem to me that the linker should be aware that users might 
think that NOLOAD means: "do not load the contents of this section from 
the file into the running image", rather than its real meaning (for ELF 
based targets) of "do not give this section any space in the output 
file; if present in the running image, initialise its contents with zero".

Hence I now think that we should update the documentation to more 
properly describe the behaviour of NOLOAD and that we should fix the 
linker so that it issues a warning message whenever it throws away 
non-zero contents of an input section being mapped to a NOLOAD output 
section.

Cheers
   Nick

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


Re: [Bug ld/12565] NOLOAD sections empty

2011-07-01 Thread Nick Clifton

Hi H.J.


Also, as far as I can see, this behaviour is not documented anywhere.
Do you know of any applications that rely upon this feature ?


NOLOAD means "don't load into memory", which maps to NOBITS.
Otherwise, it will be loaded into memory.


OK, I now get that NOLOAD is effectively a marker for .bss type sections.

But it does seem to me that the linker should be aware that users might 
think that NOLOAD means: "do not load the contents of this section from 
the file into the running image", rather than its real meaning (for ELF 
based targets) of "do not give this section any space in the output 
file; if present in the running image, initialise its contents with zero".


Hence I now think that we should update the documentation to more 
properly describe the behaviour of NOLOAD and that we should fix the 
linker so that it issues a warning message whenever it throws away 
non-zero contents of an input section being mapped to a NOLOAD output 
section.


Cheers
  Nick



___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug ld/12565] NOLOAD sections empty

2011-07-01 Thread nickc at redhat dot com
http://sourceware.org/bugzilla/show_bug.cgi?id=12565

--- Comment #16 from Nick Clifton  2011-07-01 12:45:37 
UTC ---
Hi Jifl,

>> Also, as far as I can see, this behaviour is not documented anywhere.
>
> It is documented in ld.texinfo in the 'Output Section Type' node:
> @item NOLOAD
> The section should be marked as not loadable, so that it will not be
> loaded into memory when the program is run.

But that does not document the fact that the NOLOAD section will also be 
without any contents.  This was the point that I was trying to get at.

Cheers
   Nick

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


Re: [Bug ld/12565] NOLOAD sections empty

2011-07-01 Thread Nick Clifton

Hi Jifl,


Also, as far as I can see, this behaviour is not documented anywhere.


It is documented in ld.texinfo in the 'Output Section Type' node:
@item NOLOAD
The section should be marked as not loadable, so that it will not be
loaded into memory when the program is run.


But that does not document the fact that the NOLOAD section will also be 
without any contents.  This was the point that I was trying to get at.


Cheers
  Nick

___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug ld/12565] NOLOAD sections empty

2011-07-01 Thread jifl-bugzilla at jifvik dot org
http://sourceware.org/bugzilla/show_bug.cgi?id=12565

--- Comment #15 from Jonathan Larmour  
2011-07-01 08:57:41 UTC ---
(In reply to comment #14)
> 
> I spent quite a lot of time wrestling with NOLOAD before
> http://sourceware.org/ml/binutils/2010-09/msg00245.html.  I believe we
> have fairly strong historical precedent for the current behaviour on
> ELF, ie. that NOLOAD makes a .bss style section, noload, alloc, no
> contents.

But before H.J.'s change in bug #5522 it did include contents, which is why I
reported this change of behaviour in the first place, so I'm not sure what you
mean about the precedent. The documented behaviour was to include contents
prior to your (and H.J.'s) change.

I hadn't noticed your change before as I've only been using 2.20.1 which
doesn't include it.

> As others have replied, people make use of it this way.

We have made use of it with contents, as per the documentation. If you want
something that doesn't include contents, wouldn't it be better to make a new
output section type rather than changing the behaviour of an existing one?
Otherwise it could be confusing. If you want something different from the
previous behaviour of NOLOAD it would be better to remove NOLOAD so people can
decide what they want to do with their linker scripts once they see the error.
Personally, I had thought when I submitted this bug that I had found an
isolated instance, but it turns out there are a bunch of linker scripts
floating around my company which rely on NOLOAD's old behaviour.

libgloss currently relies on it including contents too.

Jifl

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils