From: Helge Deller <deller@gmx.de>
To: John David Anglin <dave@hiauly1.hia.nrc.ca>
Cc: linux-parisc@vger.kernel.org, linux-kernel@vger.kernel.org,
kyle@mcmartin.ca, randolph@tausq.org, jmm@inutil.org,
torvalds@linux-foundation.org, akpm@linux-foundation.org,
Helge Deller <deller@gmx.de>
Subject: Re: [PATCH 2/2] parisc: fix module loading failure of large modules
Date: Mon, 29 Dec 2008 18:56:40 +0100 [thread overview]
Message-ID: <49590F58.9040909@gmx.de> (raw)
In-Reply-To: <20081229161446.C91EB4EDA@hiauly1.hia.nrc.ca>
John David Anglin wrote:
> Hi Helge,
>
>> With this patch we now instead append individual PLT stub entries
>> directly at the end of the code sections where the stubs are actually
>> called. This reduces the distance between the PCREL location and the
>> stub entry so that the relocations can be fulfilled.
>
> GCC for PA-RISC assumes that the stubs will be placed before the code
> section. This is what HP-UX does. It's also not possible to compute
> the distance to the end of a function in GCC due to the way branch
> shortening is done. If the distance to the start of the code section
> is too large, GCC outputs a long call.
Interesting. I didn't knew that.
> If you can insert them before the code section where the stubs are
> called, this will minimize the chance that a pc-relative call will
> not reach its stub.
Yes, good idea.
I'll change that and send an updated patch.
> Great patch!
Thanks!
Helge
next prev parent reply other threads:[~2008-12-29 17:56 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-12-29 14:07 [PATCH] parisc: fix module loading failure of large kernel modules Helge Deller
2008-12-29 14:07 ` Helge Deller
2008-12-29 14:09 ` [PATCH 1/2] module.c: fix module loading failure of large modules Helge Deller
2008-12-29 14:09 ` Helge Deller
2008-12-29 14:10 ` [PATCH 2/2] parisc: " Helge Deller
2008-12-29 14:10 ` Helge Deller
2008-12-29 15:46 ` Sam Ravnborg
2008-12-29 16:09 ` Helge Deller
2008-12-29 16:14 ` John David Anglin
2008-12-29 17:56 ` Helge Deller [this message]
-- strict thread matches above, loose matches on Subject: below --
2008-12-29 20:34 [PATCH] parisc: fix module loading failure of large kernel modules (take 2) Helge Deller
2008-12-30 22:45 ` Rusty Russell
2008-12-31 11:31 ` [PATCH] parisc: fix module loading failure of large kernel modules (take 4) Helge Deller
2008-12-31 11:36 ` [PATCH 2/2] parisc: fix module loading failure of large modules Helge Deller
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=49590F58.9040909@gmx.de \
--to=deller@gmx.de \
--cc=akpm@linux-foundation.org \
--cc=dave@hiauly1.hia.nrc.ca \
--cc=jmm@inutil.org \
--cc=kyle@mcmartin.ca \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-parisc@vger.kernel.org \
--cc=randolph@tausq.org \
--cc=torvalds@linux-foundation.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.