All of lore.kernel.org
 help / color / mirror / Atom feed
From: Geoff Levand <geoffrey.levand@am.sony.com>
To: David Gibson <david@gibson.dropbear.id.au>
Cc: linuxppc-dev <Linuxppc-dev@ozlabs.org>,
	Olaf Hering <olaf@aepfle.de>, Paul Mackerras <paulus@samba.org>
Subject: Re: Okay to change default zImage link/load address?
Date: Fri, 09 Feb 2007 15:05:45 -0800	[thread overview]
Message-ID: <45CCFE49.9010406@am.sony.com> (raw)
In-Reply-To: <20070209224010.GB7339@localhost.localdomain>

David Gibson wrote:
> On Fri, Feb 09, 2007 at 11:34:05AM -0800, Geoff Levand wrote:
>> Mark A. Greer wrote:
>> > On Fri, Feb 09, 2007 at 07:51:07PM +0100, Olaf Hering wrote:
>> >> On Fri, Feb 09, Mark A. Greer wrote:
>> >> 
>> >> > Does anyone know of a problem downloading to 8MB instead of 4MB in
>> >> > arch/powerpc?  If not, would you object if it changed to 8MB?
>> >> 
>> >> IBM chrp 32 and 64bit has a free and usable memory range from 64k to
>> >> 12M.
>> >> Changing the link address from 4 to 8 MB will reduce the max zImage size
>> >> to 3MB on these systems.
>> > 
>> > Hmm, not good.
>> > 
>> > Paul, may we may this configurable (i.e., a CONFIG option)?
>>  
>> To have zImage.lds.S use a config option you'll need to run it through
>> the preprocessor, which in general seems like a good thing.  Now we just
>> have this:
>> 
>> $(obj)/zImage.lds $(obj)/zImage.coff.lds: $(obj)/%: $(srctree)/$(src)/%.S
>> 	@cp $< $@
> 
> No.  Well, at that, running zImage.lds.S through the preprocessor
> might not be a bad idea, but we definitely want to avoid using the
> config anywhere in the bootwrapper, except in the Makefile for
> choosing the platform passed to the wrapper script.  The idea is that
> it should always be possible to wrap a given vmlinux using the wrapper
> script and wrapper library, outside the context of the kernel source
> tree that built that vmlinux.
> 
> Note that the zImage is already build -fPIC and capable of relocating
> itself if loaded at an address other than 4MB...
> 

That line is from the makefile, not the wrapper, so the preprocessing,
and the use of the config file would be prior to running the wrapper.

It seems though that you are talking about something other than using
a config option to change the hard coded output location in the linker
script, which I thought was Mark's proposal.

-Geoff

  reply	other threads:[~2007-02-09 23:05 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-02-09 18:43 Okay to change default zImage link/load address? Mark A. Greer
2007-02-09 18:51 ` Olaf Hering
2007-02-09 18:55   ` Mark A. Greer
2007-02-09 19:14     ` Josh Boyer
2007-02-09 22:35       ` David Gibson
2007-02-10  1:27         ` Josh Boyer
2007-02-09 19:34     ` Geoff Levand
2007-02-09 22:40       ` David Gibson
2007-02-09 23:05         ` Geoff Levand [this message]
2007-02-09 23:21         ` Mark A. Greer
2007-02-09 23:25           ` Mark A. Greer
2007-02-10  0:53             ` David Gibson
2007-02-12 19:18               ` Mark A. Greer

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=45CCFE49.9010406@am.sony.com \
    --to=geoffrey.levand@am.sony.com \
    --cc=Linuxppc-dev@ozlabs.org \
    --cc=david@gibson.dropbear.id.au \
    --cc=olaf@aepfle.de \
    --cc=paulus@samba.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.