All of lore.kernel.org
 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 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.