Linux MIPS Architecture development
 help / color / mirror / Atom feed
From: Florian Fainelli <florian@openwrt.org>
To: Manuel Lauss <mano@roarinelk.homelinux.net>
Cc: Kevin Hickey <khickey@rmicorp.com>,
	Sergei Shtylyov <sshtylyov@ru.mvista.com>,
	Linux-MIPS <linux-mips@linux-mips.org>
Subject: Re: [PATCH 0/3] Alchemy: platform updates
Date: Mon, 30 Mar 2009 10:39:40 +0200	[thread overview]
Message-ID: <200903301039.41398.florian@openwrt.org> (raw)
In-Reply-To: <20090329175243.04ebfd56@scarran.roarinelk.net>

Hi Manuel, Kevin,

Le Sunday 29 March 2009 17:52:43 Manuel Lauss, vous avez écrit :
> On Sun, 29 Mar 2009 10:27:46 -0500
>
> Kevin Hickey <khickey@rmicorp.com> wrote:
> > On Sun, 2009-03-29 at 17:03 +0400, Sergei Shtylyov wrote:
> > >   Single kernel binary? If it's at all possible, I am all for it.
> >
> > On some level, I agree but not at the expense of a larger kernel or
> > longer boot times.  Maybe I'm just not following how your implementation
> > works but it seems to me that runtime checks will add to boot time.
> > More importantly it adds to the kernel memory footprint as the tables of
> > constants for multiple CPUs will have to be compiled in.  If I'm
> > designing a board with an Au1250 in it, I don't care about the interrupt
> > numbers for Au1100 or Au1500.  This problem compounds when we introduce
> > Au1300 - several of its subsystems (like the interrupt controller) are
> > new requiring not only a new table of constants but a new object as
> > well.  In the desktop space I can understand this approach, but in the
> > embedded space it seems like an unnecessary resource burden.
> >
> > Please enlighten me :)
>
> You're right, from a single-cpu-board POV it doesn't make sense.
> However if you have a few boards which mostly differ in the Alchemy
> chip used (and not much else difference in board support code), I find
> this to be highly beneficial.  If I can have a single binary for the
> folks testing these boards, all the better!

I definitively agree, from a distribution point of view, that's even better.  
For instance Maxime did an excellent job with bcm63xx [1] which has both 
different base addresses for the SoC registers and even different offsets for 
the same things inside those registers. Resulting kernel is not that slower 
even though I do not have figures to show. Additionnaly you can still choose 
which BCM63xx SoC you are compiling for.

>
> Yes, increased binary size is to be expected, but I don't expect it to
> be in the megabyte range.
>
> I'm primarily doing this for company-internal purposes; I just thought
> I'd share the final result, maybe someone else might find it useful.

[1] : 
http://www.linux-mips.org/git?p=linux-bcm63xx.git;a=blob;f=arch/mips/bcm63xx/cpu.c;h=0a403dd07cf48109c904486cc1106d99ce036aad;hb=30c20e2899bbf31069aee0bdc4258c211f7a3d0f
-- 
Best regards, Florian Fainelli
Email : florian@openwrt.org
http://openwrt.org
-------------------------------

  reply	other threads:[~2009-03-30  8:39 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-29  9:26 [PATCH 0/3] Alchemy: platform updates Manuel Lauss
2009-03-29  9:27 ` [PATCH 1/3] Alchemy: get rid of common/platform.c Manuel Lauss
2009-03-29  9:27   ` [PATCH 2/3] Alchemy: add RTC device to devboards Manuel Lauss
2009-03-29  9:27     ` [PATCH 3/3] Alchemy: convert to physmap flash Manuel Lauss
2009-04-06  7:57       ` Florian Fainelli
2009-03-29 11:36   ` [PATCH 1/3] Alchemy: get rid of common/platform.c Sergei Shtylyov
2009-03-29 11:37   ` Sergei Shtylyov
2009-03-29 11:35 ` [PATCH 0/3] Alchemy: platform updates Sergei Shtylyov
2009-03-29 12:38   ` Manuel Lauss
2009-03-29 13:03     ` Sergei Shtylyov
2009-03-29 15:27       ` Kevin Hickey
2009-03-29 15:52         ` Manuel Lauss
2009-03-30  8:39           ` Florian Fainelli [this message]
2009-03-30 16:23           ` Ralf Baechle
2009-03-29 15:43       ` Manuel Lauss
2009-03-29 20:48       ` Sergei Shtylyov

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=200903301039.41398.florian@openwrt.org \
    --to=florian@openwrt.org \
    --cc=khickey@rmicorp.com \
    --cc=linux-mips@linux-mips.org \
    --cc=mano@roarinelk.homelinux.net \
    --cc=sshtylyov@ru.mvista.com \
    /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