public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
From: Andreas Jaeger <aj@suse.de>
To: Michael Matz <matz@suse.de>
Cc: binutils@sourceware.org, Jan Hubicka <jh@suse.cz>,
		Daniel Jacobowitz <drow@false.org>
Subject: Re: [x86-64] Large model relocation
Date: Fri, 24 Mar 2006 01:11:00 -0000	[thread overview]
Message-ID: <ho8xr17epg.fsf@reger.suse.de> (raw)
In-Reply-To: <Pine.LNX.4.58.0603201834190.12554@wotan.suse.de> (Michael Matz's 	message of "Mon, 20 Mar 2006 18:45:48 +0100 (CET)")

[-- Attachment #1: Type: text/plain, Size: 1345 bytes --]

Michael Matz <matz@suse.de> writes:

> Hi,
>
> this patch adds the necessary support for the five new relocations for the
> large model.  I've tested it on i386 and x86-64; it builds and doesn't
> cause breakage in the testsuite for both.  Additionally we use this patch
> (or better a slightly adjusted version) since quite some time internally,
> so I know that it also builds several 1000 packages.
>
> Full large model support also would need a change in elf.sc moving the
> .plt after .text (so that .plt and .got have a chance to be nearer than
> 2GB).  This is because I didn't bother to actually implement the new .plt
> layout which would be necessary for more than an abysmal number of
> exported symbols (102261124 to be exact).  So the .plt code can't reach 
> more than 2GB away, and as long as .text is between it and .got it can't 
> be larger than the 2GB, which is the whole point of the large model.  I'll 
> post that as follow up after the heat about this one (should it rise up) 
> colds down :-)

Thanks, the patch looks fine, I have applied it after building and
testing binutils with it,

Andreas
-- 
 Andreas Jaeger, aj@suse.de, http://www.suse.de/~aj/
  SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
   GPG fingerprint = 93A3 365E CE47 B889 DF7F  FED1 389A 563C C272 A126

[-- Attachment #2: Type: application/pgp-signature, Size: 188 bytes --]

  reply	other threads:[~2006-03-23  8:22 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-03-20 18:36 Michael Matz
2006-03-24  1:11 ` Andreas Jaeger [this message]
2006-04-26 10:29   ` Andreas Jaeger

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ho8xr17epg.fsf@reger.suse.de \
    --to=aj@suse.de \
    --cc=binutils@sourceware.org \
    --cc=drow@false.org \
    --cc=jh@suse.cz \
    --cc=matz@suse.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).