Component: ld
Assignee: unassigned at sourceware dot org
Reporter: bugdal at aerifal dot cx
Target Milestone: ---
Created attachment 15453
--> https://sourceware.org/bugzilla/attachment.cgi?id=15453&action=edit
proposed fix
The sh linker's handling
Component: ld
Assignee: unassigned at sourceware dot org
Reporter: bugdal at aerifal dot cx
Target Milestone: ---
This looks like it may be the same as
https://sourceware.org/bugzilla/show_bug.cgi?id=22374 for other archs. I was
testing with the program:
#include
static void
Severity: normal
Priority: P2
Component: gold
Assignee: ccoutant at gmail dot com
Reporter: bugdal at aerifal dot cx
CC: ian at airs dot com
Target Milestone: ---
When targeting powerpc64, attempting to link a source file that calls
https://sourceware.org/bugzilla/show_bug.cgi?id=26541
--- Comment #5 from Rich Felker ---
Ah no, at least according to your patch the original gold bug seems to be there
still too.
--
You are receiving this mail because:
You are on the CC list for the bug.
https://sourceware.org/bugzilla/show_bug.cgi?id=26541
--- Comment #4 from Rich Felker ---
Oh, was this already fixed and the new comment just a matter of lustre
erroneously returning a value that's not a valid error code to userspace?
--
You are receiving this mail because:
You are on the CC li
https://sourceware.org/bugzilla/show_bug.cgi?id=26541
--- Comment #2 from Rich Felker ---
Why is this still open? It's trivial to fix.
--
You are receiving this mail because:
You are on the CC list for the bug.
https://sourceware.org/bugzilla/show_bug.cgi?id=26945
--- Comment #13 from Rich Felker ---
This patch keeps expanding way beyond the scope of what I can commit to review,
and I don't think it's nearing something that fixes the problem. It's not clear
to me if bfd_stat uses fstat on the open file
https://sourceware.org/bugzilla/show_bug.cgi?id=26945
--- Comment #11 from Rich Felker ---
Yes, see my footnote in comment 4. That works just as well.
--
You are receiving this mail because:
You are on the CC list for the bug.
https://sourceware.org/bugzilla/show_bug.cgi?id=26945
--- Comment #9 from Rich Felker ---
> > 1. make_tempname with fd_return==NULL is *always* a bug,
> Well, I would argue that it is only a bug if the temporary file is going to
be manipulated by other system functions.
Are you going to create
https://sourceware.org/bugzilla/show_bug.cgi?id=26945
--- Comment #4 from Rich Felker ---
That's a lot to review without being familiar with the code, but a couple
things I can tell you right off:
1. make_tempname with fd_return==NULL is *always* a bug, and defeats the whole
purpose of mkstemp.
https://sourceware.org/bugzilla/show_bug.cgi?id=26945
--- Comment #2 from Rich Felker ---
It's not that smart_rename should open it and use fchown+fchmod on the result;
indeed as you figured that has the same race. The problem is that you threw
away the only safe way to perform modifications on t
https://sourceware.org/bugzilla/show_bug.cgi?id=26945
Rich Felker changed:
What|Removed |Added
CC||siddhesh at sourceware dot org
--
You
Component: binutils
Assignee: unassigned at sourceware dot org
Reporter: bugdal at aerifal dot cx
Target Milestone: ---
At least objcopy and perhaps other utilities generate a temp file safely with
mkstemp then rename it to atomically replace the original, but the
https://sourceware.org/bugzilla/show_bug.cgi?id=25882
--- Comment #9 from Rich Felker ---
OK, I should have read more before commenting. My comments in particular
pertain specifically to powerpc64 although other archs might be affected too. I
see that the error was downgraded to a warning since t
https://sourceware.org/bugzilla/show_bug.cgi?id=25882
--- Comment #8 from Rich Felker ---
Also note that while, for musl targets, this issue could also be fixed just by
getting gcc not to build its ld128 functions in libgcc_s, there are also people
using glibc's ld64 ABI, and glibc necessarily ha
https://sourceware.org/bugzilla/show_bug.cgi?id=25882
Rich Felker changed:
What|Removed |Added
CC||bugdal at aerifal dot cx
--- Comment
Priority: P2
Component: gold
Assignee: ccoutant at gmail dot com
Reporter: bugdal at aerifal dot cx
CC: ian at airs dot com
Target Milestone: ---
In gold's output.cc gold_fallocate, after posix_fallocate() fails with a
non-fatal erro
https://sourceware.org/bugzilla/show_bug.cgi?id=23825
--- Comment #8 from Rich Felker ---
To clarify what I mean about ABI and copy relocations, prior to RISC-V doing
this, there was no arch where it was unsafe to have extensible (existing part's
meaning remains same, new parts added to end) arra
https://sourceware.org/bugzilla/show_bug.cgi?id=23825
Rich Felker changed:
What|Removed |Added
CC||bugdal at aerifal dot cx
--- Comment
Component: gas
Assignee: unassigned at sourceware dot org
Reporter: bugdal at aerifal dot cx
Target Milestone: ---
I don't understand the lisp-like language the opcode C files are generated
from, so I don't have a root cause for this, but the problem is in
op
https://sourceware.org/bugzilla/show_bug.cgi?id=25184
Rich Felker changed:
What|Removed |Added
Target||or1k-*-*
--
You are receiving this mai
https://sourceware.org/bugzilla/show_bug.cgi?id=25056
--- Comment #3 from Rich Felker ---
It is a correctness issue for static linking, where my intent (as the one who
introduced static pie) was that presence of any unresolved relocations except
load-address-relative ones is an error. If you thin
https://sourceware.org/bugzilla/show_bug.cgi?id=25056
--- Comment #1 from Rich Felker ---
Link for convenience: https://sourceware.org/bugzilla/show_bug.cgi?id=22570
I suspect there may be other targets with exactly the same issue. There should
probably be a generic test for all targets that ass
Component: ld
Assignee: unassigned at sourceware dot org
Reporter: bugdal at aerifal dot cx
Target Milestone: ---
Created attachment 12015
--> https://sourceware.org/bugzilla/attachment.cgi?id=12015&action=edit
patch to fix
Identical to #22570 for mips.
--
https://sourceware.org/bugzilla/show_bug.cgi?id=23683
--- Comment #2 from Rich Felker ---
Thanks. I was still using 2.27 and should have tagged that in the report, and
should have checked later history for a fix.
--
You are receiving this mail because:
You are on the CC list for the bug.
__
Component: ld
Assignee: unassigned at sourceware dot org
Reporter: bugdal at aerifal dot cx
Target Milestone: ---
In the help text (ld --help), -z stack-size=SIZE is misspelled as -z
stacksize=SIZE, which does not work.
--
You are receiving this mail because:
You are
https://sourceware.org/bugzilla/show_bug.cgi?id=23428
--- Comment #3 from Rich Felker ---
I don't object to the patch as-is, but wouldn't it make more sense to make the
program headers a first-class section with attribute alloc, so that they're
semantically forced to be visible at runtime rather
https://sourceware.org/bugzilla/show_bug.cgi?id=23428
Rich Felker changed:
What|Removed |Added
CC||bugdal at aerifal dot cx
--
You are
https://sourceware.org/bugzilla/show_bug.cgi?id=22263
--- Comment #17 from Rich Felker ---
Yes, see #22570 in the list of duplicates, which includes a proposed patch for
MIPS.
--
You are receiving this mail because:
You are on the CC list for the bug.
___
https://sourceware.org/bugzilla/show_bug.cgi?id=22570
--- Comment #2 from Rich Felker ---
Created attachment 10762
--> https://sourceware.org/bugzilla/attachment.cgi?id=10762&action=edit
proposed fix
--
You are receiving this mail because:
You are on the CC list for the bug.
_
https://sourceware.org/bugzilla/show_bug.cgi?id=22570
--- Comment #1 from Rich Felker ---
The details of my assessment seem to have been written in terms of 2.25; with
the changes from commit 0e1862bb40, the problem is now use of bfd_link_pic
where bfd_link_dll was presumably intended. I'm attach
Component: ld
Assignee: unassigned at sourceware dot org
Reporter: bugdal at aerifal dot cx
Target Milestone: ---
When a symbol is defined and referenced in the main program, DTPMOD relocations
should be bound at ld time to module 1. This is mandatory for static
https://sourceware.org/bugzilla/show_bug.cgi?id=22570
Rich Felker changed:
What|Removed |Added
Target||mips*-*-*-*
--
You are receiving this
https://sourceware.org/bugzilla/show_bug.cgi?id=21086
Rich Felker changed:
What|Removed |Added
CC||bugdal at aerifal dot cx
--- Comment
https://sourceware.org/bugzilla/show_bug.cgi?id=21017
--- Comment #1 from Rich Felker ---
Created attachment 9735
--> https://sourceware.org/bugzilla/attachment.cgi?id=9735&action=edit
patch to fix the bug
This seems to fix the issue, but there may be additional reloc types that
should be chec
Status: UNCONFIRMED
Severity: normal
Priority: P2
Component: ld
Assignee: unassigned at sourceware dot org
Reporter: bugdal at aerifal dot cx
Target Milestone: ---
When linking a program where all GOT-based relocations are GOT-relative
https://sourceware.org/bugzilla/show_bug.cgi?id=17699
--- Comment #7 from Rich Felker ---
Can you cite any documentation that a non-STT_TLS symbol reference is
permissible in a DTPMOD relocation?
--
You are receiving this mail because:
You are on the CC list for the bug.
___
https://sourceware.org/bugzilla/show_bug.cgi?id=17739
--- Comment #11 from Rich Felker ---
Ping?
--
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/mailm
https://sourceware.org/bugzilla/show_bug.cgi?id=20360
--- Comment #6 from Rich Felker ---
To add some context, I suggested building without -Bsymbolic* in case the cause
was relocations that would be textrels before -Bsymbolic* optimized them out.
Looking at the source, particularly calls/jmps to
https://sourceware.org/bugzilla/show_bug.cgi?id=20360
--- Comment #3 from Rich Felker ---
At this point there's no testcase smaller than the dotnet/coreclr build, but I
wanted to go ahead and post the bug here to get discussion moved to the tracker
where it's relevant. I believe they have a way t
Priority: P2
Component: ld
Assignee: unassigned at sourceware dot org
Reporter: bugdal at aerifal dot cx
Target Milestone: ---
This is based on a bug report from dotnet CoreCLR:
https://github.com/dotnet/coreclr/issues/6211
What seems to be happening is
https://sourceware.org/bugzilla/show_bug.cgi?id=17739
Rich Felker changed:
What|Removed |Added
CC||bugdal at aerifal dot cx
--- Comment
Severity: normal
Priority: P2
Component: ld
Assignee: unassigned at sourceware dot org
Reporter: bugdal at aerifal dot cx
Target Milestone: ---
If binutils is built with STT_COMMON support enabled, use of
-Bsymbolic-functions for libraries with visible data
https://sourceware.org/bugzilla/show_bug.cgi?id=19516
--- Comment #2 from Rich Felker ---
Created attachment 8978
--> https://sourceware.org/bugzilla/attachment.cgi?id=8978&action=edit
proposed fix
Producing a symbolic GLOB_DAT relocation for a symbol that has been made
local/hidden by setting
https://sourceware.org/bugzilla/show_bug.cgi?id=19516
Rich Felker changed:
What|Removed |Added
Target||microblaze*-*-*
--
You are receiving t
https://sourceware.org/bugzilla/show_bug.cgi?id=19091
Rich Felker changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://sourceware.org/bugzilla/show_bug.cgi?id=19516
--- Comment #1 from Rich Felker ---
For reference here is the link to the first version of musl affected which can
be used to reproduce the bug:
http://git.musl-libc.org/cgit/musl/commit/?id=ad1cd43a86645ba2d4f7c8747240452a349d6bc1
--
You a
Component: ld
Assignee: unassigned at sourceware dot org
Reporter: bugdal at aerifal dot cx
Target Milestone: ---
Current git versions of musl libc (since commit ad1cd43a86) are producing a
libc.so that crashes during startup due to a bogus GLOB_DAT relocations with
symbol
https://sourceware.org/bugzilla/show_bug.cgi?id=19091
--- Comment #4 from Rich Felker ---
OK. Should I submit a patch or do you just want to make the one-line change
yourself?
--
You are receiving this mail because:
You are on the CC list for the bug.
__
https://sourceware.org/bugzilla/show_bug.cgi?id=19091
--- Comment #2 from Rich Felker ---
Even with no GOT, not PLT, and no function descriptors, it's possible to have
code which accesses local data via @GOTOFF displacements. If DT_PLTGOT is not
emitted in the output _DYNAMIC, there is no way for
https://sourceware.org/bugzilla/show_bug.cgi?id=19091
Rich Felker changed:
What|Removed |Added
CC||kkojima at rr dot iij4u.or.jp
--
You a
Priority: P2
Component: ld
Assignee: unassigned at sourceware dot org
Reporter: bugdal at aerifal dot cx
Target Milestone: ---
Created attachment 8688
--> https://sourceware.org/bugzilla/attachment.cgi?id=8688&action=edit
minimal test case
Und
https://sourceware.org/bugzilla/show_bug.cgi?id=18561
--- Comment #15 from Rich Felker ---
Well that's not necessarily a show-stopper if there's a way to make it "work"
(evaluate at as-time) in .size or other contexts that aren't actually section
content and where a relocation cannot be emitted.
https://sourceware.org/bugzilla/show_bug.cgi?id=18561
--- Comment #13 from Rich Felker ---
Nick, can you confirm that your latest patch does not break typical use of
.size like Alan Modra was concerned it would?
--
You are receiving this mail because:
You are on the CC list for the bug.
__
https://sourceware.org/bugzilla/show_bug.cgi?id=18561
--- Comment #11 from Rich Felker ---
I filed the corresponding GCC bug, #66609, at the same time I filed this one.
Here is the link for reference:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66609
The bug report also contains a suggested fi
https://sourceware.org/bugzilla/show_bug.cgi?id=18561
--- Comment #9 from Rich Felker ---
If this is the intended behavior, it should be documented as such. At the very
least it's highly counter-intuitive, and a regression, albeit a very very old
one. In any case I'll submit the patch I have for
https://sourceware.org/bugzilla/show_bug.cgi?id=18561
--- Comment #7 from Rich Felker ---
I like it, but I think some effort should be put into understanding why the
wrong code was there to begin with and what the intent was. Here is the commit
that broke it:
https://sourceware.org/git/?p=binuti
https://sourceware.org/bugzilla/show_bug.cgi?id=18561
--- Comment #4 from Rich Felker ---
Indeed, that fixes it. However I'm skeptical of the way you're making the
change:
1. S_FORCE_RELOC already has logic to force relocations for weak symbols when
the strict argument is nonzero, so I wonder if
https://sourceware.org/bugzilla/show_bug.cgi?id=18561
--- Comment #2 from Rich Felker ---
No. I believe I actually tried something near-identical before reporting this
bug, but I just tried your patch to be sure, and there is still no relocation
in the output object file.
--
You are receiving t
Severity: normal
Priority: P2
Component: gas
Assignee: unassigned at sourceware dot org
Reporter: bugdal at aerifal dot cx
Target Milestone: ---
Minimal test case:
.global a
.global b
.weak a
.align 2
a: .long 0
b: .long a-.
The output should contain a
||bugdal at aerifal dot cx
Resolution|FIXED |---
--- Comment #20 from Rich Felker ---
The fix that was applied only fixed PC-relative relocations (direct calls).
Global data containing a function pointer to a weak function is still broken
exactly the
https://sourceware.org/bugzilla/show_bug.cgi?id=17699
Rich Felker changed:
What|Removed |Added
CC||bugdal at aerifal dot cx
--- Comment
https://sourceware.org/bugzilla/show_bug.cgi?id=17117
Rich Felker changed:
What|Removed |Added
CC||bugdal at aerifal dot cx
--- Comment
https://sourceware.org/bugzilla/show_bug.cgi?id=16698
Rich Felker changed:
What|Removed |Added
CC||bugdal at aerifal dot cx
--- Comment
Priority: P2
Component: ld
Assignee: unassigned at sourceware dot org
Reporter: bugdal at aerifal dot cx
Attempting to compile/link the following program:
__thread int my_tls = 1;
int *my_tls_addr() { return &my_tls; }
with the following gcc command line:
http://sourceware.org/bugzilla/show_bug.cgi?id=12549
--- Comment #15 from Rich Felker 2013-02-18 16:22:08
UTC ---
It would be really nice if the bug tracker messages for addressing/resolving
bugs actually contained an explanation of the resolution rather than cryptic
references to implementation
http://sourceware.org/bugzilla/show_bug.cgi?id=12549
--- Comment #11 from Rich Felker 2013-01-14 17:22:10
UTC ---
I agree that's possible as a workaround, but it seems to change the ABI for
libgcc_s, which is probably not acceptable. And it's an extra layer of hackery
needed to achieve the norma
http://sourceware.org/bugzilla/show_bug.cgi?id=12549
--- Comment #9 from Rich Felker 2013-01-14 15:50:47
UTC ---
I agree there's room for different opinions here, but I believe --as-needed
should not include the library when all it does is resolve weak references.
This matches the static-linking
http://sourceware.org/bugzilla/show_bug.cgi?id=14798
Bug #: 14798
Summary: gas mips target generates badly typed symbols
Product: binutils
Version: unspecified
Status: NEW
Severity: normal
Priority: P2
Component
http://sourceware.org/bugzilla/show_bug.cgi?id=13600
--- Comment #15 from Rich Felker 2012-10-12 19:48:59
UTC ---
Ping. Is anybody willing to look at this?
This bug has been open 10 months now and fixing it is as simple as removing the
offending code that's enforcing policy to protest a
http://sourceware.org/bugzilla/show_bug.cgi?id=14640
Bug #: 14640
Summary: gas generates broken relocations for symbols with weak
defs on microblaze
Product: binutils
Version: unspecified
Status: NEW
http://sourceware.org/bugzilla/show_bug.cgi?id=14512
Bug #: 14512
Summary: -z nodelete should be default for shared libraries
Product: binutils
Version: unspecified
Status: NEW
Severity: normal
Priority: P2
Comp
http://sourceware.org/bugzilla/show_bug.cgi?id=13600
Rich Felker changed:
What|Removed |Added
CC||bugdal at aerifal dot cx
--- Comment
http://sourceware.org/bugzilla/show_bug.cgi?id=14156
Bug #: 14156
Summary: --sort-section=alignment trashes init/fini sections,
and anything similar
Product: binutils
Version: unspecified
Status: NEW
Severity:
http://sourceware.org/bugzilla/show_bug.cgi?id=14037
--- Comment #3 from Rich Felker 2012-05-01 18:37:16
UTC ---
Another way of looking at the issue is that, if strip is going to break things
like this, the default linker scripts should order the sections such that
mandatory sections like text/d
http://sourceware.org/bugzilla/show_bug.cgi?id=14037
--- Comment #3 from Rich Felker 2012-05-01 18:37:16
UTC ---
Another way of looking at the issue is that, if strip is going to break things
like this, the default linker scripts should order the sections such that
mandatory sections like text/d
http://sourceware.org/bugzilla/show_bug.cgi?id=14037
Rich Felker changed:
What|Removed |Added
Component|ld |binutils
--- Comment #2 from Rich Felker
http://sourceware.org/bugzilla/show_bug.cgi?id=14037
Bug #: 14037
Summary: Bogus global symbol ABS redefinition when linking PIC
app to eh_frame-stripped shared library
Product: binutils
Version: unspecified
Status: NEW
http://sourceware.org/bugzilla/show_bug.cgi?id=13891
--- Comment #4 from Rich Felker 2012-03-24 21:06:21
UTC ---
Mike, a link to the workaround would be really helpful to anyone dealing with
this bug or trying to evaluate whether it's worth fixing/working around in
binutils.
Personally I don't
http://sourceware.org/bugzilla/show_bug.cgi?id=13891
Bug #: 13891
Summary: Binutils is impossible to static link due to broken
libtool behavior
Product: binutils
Version: unspecified
Status: NEW
Severity: criti
http://sourceware.org/bugzilla/show_bug.cgi?id=12549
Rich Felker changed:
What|Removed |Added
CC||bugdal at aerifal dot cx
--- Comment #1
81 matches
Mail list logo
