* Re: initramfs size limitation
From: Rob Landley @ 2008-06-04 17:24 UTC (permalink / raw)
To: Mike Frysinger; +Cc: pwilshire, linux-embedded
In-Reply-To: <8bd0f97a0806031454g6d8fe57ahca942b2a81b737aa@mail.gmail.com>
On Tuesday 03 June 2008 16:54:38 Mike Frysinger wrote:
> > but they just ripped out the old code
> > generator in favor of a new one
> > (See http://lists.gnu.org/archive/html/qemu-devel/2008-02/msg00011.html )
> > so there's not much point in going there right now. (Hopefully they'll
> > have a 1.0 release within our lifetimes.)
>
> i'm aware the tcg stuff exists, but that's about it. only hope for
> new ports atm is to contract a qemu guy. not that the qemu guys would
> mind. thanks for the link.
I have once again allowed myself to get buried in dayjob, but this one only
lasts until the end of the month and _then_ I plan to get
http://landley.net/qemu caught up to the present.
I'm the meantime, I'm still trying to extend my FWL scripts to create a cross
compiler and bootable filesystem for every target qemu supports. Mips
regressed in 2.6.25 because they yanked out the specific qemu target
(http://www.kernel.org/hg/index.cgi/linux-2.6/rev/79936) so I'm migrating the
sucker to use a malta kernel .config, and then I can do a 0.4.0 release with
the distcc trick working out of the box.
(QEMU remains a _marvelous_ toy for doing embedded development, to the point
it's now kind of hard for me to take seriously a platform that doesn't have a
decent emulator I can run on my laptop.)
> -mike
Rob
--
"One of my most productive days was throwing away 1000 lines of code."
- Ken Thompson.
^ permalink raw reply
* Re: [PATCH] console - Add configurable support for console charset translation
From: Rob Landley @ 2008-06-04 17:19 UTC (permalink / raw)
To: Holger Schurig; +Cc: linux-tiny, Matt Mackall, linux-embedded
In-Reply-To: <200806040904.13483.hs4233@mail.mn-solutions.de>
On Wednesday 04 June 2008 02:04:13 Holger Schurig wrote:
> > Linux-embedded is the place to be, folks.
>
> Another thing that I just noticed: According to
>
> http://vger.kernel.org/vger-lists.html#linux-embedded
>
> there is no mailing list archive for this list.
Yes, but according to the comments on http://lwn.net/Articles/283749/ there
is. Unfortunately, there seems to be no way to beat a threaded user
interface out of it.
I have a complete mbox going back to the beginning (thanks Randy!) and am
pondering putting up an archive in the format I like (threaded index, grouped
monthly) this weekend. Depends on whether or not I can convince mailman to
archive a list it isn't hosting, or if I get bored enough to write my own
indexer in python...
Rob
--
"One of my most productive days was throwing away 1000 lines of code."
- Ken Thompson.
^ permalink raw reply
* Re: [PATCH] console - Add configurable support for console charset translation
From: Tim Bird @ 2008-06-04 15:35 UTC (permalink / raw)
To: David Woodhouse; +Cc: linux-tiny, linux-embedded, linux kernel
In-Reply-To: <1212538597.4042.29.camel@shinybook.infradead.org>
David Woodhouse wrote:
> On Tue, 2008-06-03 at 17:01 -0700, Tim Bird wrote:
>> This is clearly an improvement. But it is missing this part of the
>> original patch:
>
> Oops, well spotted. I've updated the patch in the git tree; thanks.
>
> (that's what comes of applying patches by hand -- I _knew_ I had to do
> that hunk, but managed to forget about it within the space of about 30
> seconds when I was doing the other parts. I plead stupidity)
We can chalk it up to my stupidity for sending a patch
that couldn't be applied automatically...
Thanks very much for the improvements!
-- Tim
=============================
Tim Bird
Architecture Group Chair, CE Linux Forum
Senior Staff Engineer, Sony Corporation of America
=============================
^ permalink raw reply
* Re: [PATCH] console - Add configurable support for console charset translation
From: Randy Dunlap @ 2008-06-04 15:24 UTC (permalink / raw)
To: Dave Hylands
Cc: Holger Schurig, linux-tiny, Matt Mackall, Rob Landley,
linux-embedded
In-Reply-To: <c32efac40806040036i744df271p68d42a103ddbaefe@mail.gmail.com>
On Wed, 4 Jun 2008 00:36:48 -0700 Dave Hylands wrote:
> > Another thing that I just noticed: According to
> >
> > http://vger.kernel.org/vger-lists.html#linux-embedded
>
> Mike Frysinger setup the following on May 27:
> <http://news.gmane.org/gmane.linux.kernel.embedded>
It's also available at http://marc.info/?l=linux-embedded
---
~Randy
<quote:>
"It's the Government of the United States." ... The largest, and yet
the least efficient, producer of computer software in the world.
^ permalink raw reply
* Re: [PATCH] console - Add configurable support for console charset translation
From: David Woodhouse @ 2008-06-04 14:27 UTC (permalink / raw)
To: Alan Cox; +Cc: Tim Bird, linux-tiny, linux-embedded, linux kernel
In-Reply-To: <20080604150747.3ec7ee57@core>
On Wed, 2008-06-04 at 15:07 +0100, Alan Cox wrote:
> > If I merge this incremental patch, does that address your objections?
>
> Yes
http://git.infradead.org/embedded-2.6.git?a=commitdiff;h=a29ccf6f8
--
dwmw2
^ permalink raw reply
* Re: [PATCH] console - Add configurable support for console charset translation
From: Alan Cox @ 2008-06-04 14:07 UTC (permalink / raw)
To: David Woodhouse; +Cc: Tim Bird, linux-tiny, linux-embedded, linux kernel
In-Reply-To: <1212587753.32207.86.camel@pmac.infradead.org>
> If I merge this incremental patch, does that address your objections?
Yes
> (Note that I've changed the expression to evaluate (c) only once, just
> because that's best practice in macro definitions. Wasn't really worth
> doing it for (vc) though.)
Agreed
^ permalink raw reply
* Re: [PATCH] console - Add configurable support for console charset translation
From: David Woodhouse @ 2008-06-04 13:55 UTC (permalink / raw)
To: Alan Cox; +Cc: Tim Bird, linux-tiny, linux-embedded, linux kernel
In-Reply-To: <20080604125500.0fce2ea4@core>
On Wed, 2008-06-04 at 12:55 +0100, Alan Cox wrote:
> Can we please get the ifdefs tided up before this goes in.
>
> For the moment this has a NAK from the tty maintainer but if the
> ifdefs
> turned went into headers where they belong and the code looked like
> say
>
> tc = vc_translate(vc, c);
>
> with two versions of vc_translate (one being vc_translate(x) (x)) it
> might be more reasonable.
Good point. I did think about that originally, but then just completely
forgot to process that hunk of the original patch altogether, which was
another way of avoiding the ifdefs :)
If I merge this incremental patch, does that address your objections?
(Note that I've changed the expression to evaluate (c) only once, just
because that's best practice in macro definitions. Wasn't really worth
doing it for (vc) though.)
diff --git a/drivers/char/vt.c b/drivers/char/vt.c
index de52f99..18b7fb0 100644
--- a/drivers/char/vt.c
+++ b/drivers/char/vt.c
@@ -2208,11 +2208,7 @@ rescan_last_byte:
c = 0xfffd;
tc = c;
} else { /* no utf or alternate charset mode */
-#ifdef CONFIG_CONSOLE_TRANSLATIONS
- tc = vc->vc_translate[vc->vc_toggle_meta ? (c | 0x80) : c];
-#else
- tc = c;
-#endif
+ tc = vc_translate(vc, c);
}
param.c = tc;
diff --git a/include/linux/vt_kern.h b/include/linux/vt_kern.h
index 81ed155..929ee80 100644
--- a/include/linux/vt_kern.h
+++ b/include/linux/vt_kern.h
@@ -72,6 +72,9 @@ int con_set_default_unimap(struct vc_data *vc);
void con_free_unimap(struct vc_data *vc);
void con_protect_unimap(struct vc_data *vc, int rdonly);
int con_copy_unimap(struct vc_data *dst_vc, struct vc_data *src_vc);
+
+#define vc_translate(vc, c) ((vc)->vc_translate[(c) | \
+ (vc)->vc_toggle_meta ? 0x80 : 0])
#else
#define con_set_trans_old(arg) (0)
#define con_get_trans_old(arg) (-EINVAL)
@@ -83,6 +86,8 @@ int con_copy_unimap(struct vc_data *dst_vc, struct vc_data *src_vc);
#define con_copy_unimap(d, s) (0)
#define con_get_unimap(vc, ct, uct, list) (-EINVAL)
#define con_free_unimap(vc) do { ; } while (0)
+
+#define vc_translate(vc, c) (c)
#endif
/* vt.c */
--
dwmw2
^ permalink raw reply related
* Re: [PATCH] console - Add configurable support for console charset translation
From: Alan Cox @ 2008-06-04 11:55 UTC (permalink / raw)
To: Tim Bird; +Cc: David Woodhouse, linux-tiny, linux-embedded, linux kernel
In-Reply-To: <4845DB46.3020308@am.sony.com>
> This is clearly an improvement. But it is missing this part of the
> original patch:
>
> --- a/drivers/char/vt.c
> +++ b/drivers/char/vt.c
> @@ -2198,7 +2198,11 @@ rescan_last_byte:
> c = 0xfffd;
> tc = c;
> } else { /* no utf or alternate charset mode */
> +#ifdef CONFIG_CONSOLE_TRANSLATIONS
> tc = vc->vc_translate[vc->vc_toggle_meta ? (c | 0x80) : c];
> +#else
> + tc = c;
> +#endif
Can we please get the ifdefs tided up before this goes in.
For the moment this has a NAK from the tty maintainer but if the ifdefs
turned went into headers where they belong and the code looked like say
tc = vc_translate(vc, c);
with two versions of vc_translate (one being vc_translate(x) (x)) it
might be more reasonable.
We can't stick random ifdefs in bits of code for arbitary 6K savings or
the entire kernel would be ifdefs.
Sor for the moment
Nacked-by: Alan Cox <alan@redhat.com>
but scope for change
^ permalink raw reply
* Re: [PATCH] console - Add configurable support for console charset translation
From: Adrian Bunk @ 2008-06-04 10:33 UTC (permalink / raw)
To: Tim Bird; +Cc: linux-tiny, linux-embedded, linux kernel
In-Reply-To: <48447615.5050806@am.sony.com>
On Mon, Jun 02, 2008 at 03:37:09PM -0700, Tim Bird wrote:
> With CONSOLE_TRANSLATIONS turned off, this saves about 6K
> on my kernel configured for an ARM development board (OMAP
> 5912 OSK). In embedded products I'm familiar with,
> console translations are not needed.
>
> This was taken from the Linux-tiny project and updated slightly
> for 2.6.25.
>...
It seems to be always me who asks the controversial questions...:
Does the linux-tiny approach of adding a kconfig variable for each 5kB
of code actually make sense? I'm asking since an exploding amount of
kconfig variables and their interdependencies have a not so small
maintainance impact in the long term.
And I'm wondering whether it's the best approach for reaching
measurable results. E.g. my small patch that removed
-maccumulate-outgoing-args on 32-bit x86 reduced the kernel size
there for the CONFIG_CC_OPTIMIZE_FOR_SIZE=y case by at about 2.5% (sic)
without adding any kconfig variables or #ifdef's.
Can you take a reasonable (best-case) .config for an existing device and
get the following numbers:
- Ignoring patches that came from linux-tiny, by how many percent did
the size of the kernel increase or decrease between 2.6.16 and 2.6.26?
- By how many percent did the kernel size decrease due to patches from
the linux-tiny project that added such kconfig options for a few kB
of code in the same timeframe?
My gut feeling is that the influence of this kind of linux-tiny patches
is hardly noticably compared to the overall code size development, but
if you have numbers that prove me wrong just point me to them and I'll
stand corrected.
cu
Adrian
BTW: I'm not insisting on "between 2.6.16 and 2.6.26", that's just some
timeframe big enough for showing general trends.
--
"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed
^ permalink raw reply
* Re: [PATCH] console - Add configurable support for console charset translation
From: David Woodhouse @ 2008-06-04 10:10 UTC (permalink / raw)
To: Adrian Bunk
Cc: Josh Boyer, Tim Bird, linux-tiny, linux-embedded, linux kernel
In-Reply-To: <20080604100721.GB27335@cs181133002.pp.htv.fi>
On Wed, 2008-06-04 at 13:07 +0300, Adrian Bunk wrote:
> On Wed, Jun 04, 2008 at 10:16:22AM +0100, David Woodhouse wrote:
> > On Tue, 2008-06-03 at 22:13 -0500, Josh Boyer wrote:
> > >
> > > > git.infradead.org/embedded-2.6.git
> > >
> > > Do you have plans to get that in -mm, or linux-next?
> >
> > Should be already there in today's linux-next.
>
> Does this imply you want Linus to merge from this tree?
Yes.
> How does this tree fit into the usual maintainership inside the Linux
> kernel?
Well, I was trying to avoid creating it -- because largely it doesn't.
I was hoping that everything could go upstream through other routes.
But Tim's patch really was a candidate for handling it myself (unless
I'm just going to punt everything to Andrew), so I capitulated.
--
dwmw2
^ permalink raw reply
* Re: [PATCH] console - Add configurable support for console charset translation
From: Adrian Bunk @ 2008-06-04 10:07 UTC (permalink / raw)
To: David Woodhouse
Cc: Josh Boyer, Tim Bird, linux-tiny, linux-embedded, linux kernel
In-Reply-To: <1212570982.4042.64.camel@shinybook.infradead.org>
On Wed, Jun 04, 2008 at 10:16:22AM +0100, David Woodhouse wrote:
> On Tue, 2008-06-03 at 22:13 -0500, Josh Boyer wrote:
> >
> > > git.infradead.org/embedded-2.6.git
> >
> > Do you have plans to get that in -mm, or linux-next?
>
> Should be already there in today's linux-next.
Does this imply you want Linus to merge from this tree?
How does this tree fit into the usual maintainership inside the Linux
kernel?
> dwmw2
cu
Adrian
--
"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed
^ permalink raw reply
* Re: [PATCH] console - Add configurable support for console charset translation
From: David Woodhouse @ 2008-06-04 9:16 UTC (permalink / raw)
To: Josh Boyer; +Cc: Tim Bird, linux-tiny, linux-embedded, linux kernel
In-Reply-To: <20080603221309.5e2762be@vader.jdub.homelinux.org>
On Tue, 2008-06-03 at 22:13 -0500, Josh Boyer wrote:
>
> > git.infradead.org/embedded-2.6.git
>
> Do you have plans to get that in -mm, or linux-next?
Should be already there in today's linux-next.
--
dwmw2
^ permalink raw reply
* Re: [PATCH] add a printk_init variant storing format strings in __initdata
From: MinChan Kim @ 2008-06-04 8:32 UTC (permalink / raw)
To: Andrew Morton
Cc: Andy Whitcroft, David Brownell, pavel, linux-kernel, linux-pm,
mingo, linux-embedded
In-Reply-To: <20080604011607.15dec5a7.akpm@linux-foundation.org>
It is important about embedded system, too.
So I add CC, linux-embedded.
On Wed, Jun 4, 2008 at 5:16 PM, Andrew Morton <akpm@linux-foundation.org> wrote:
> On Tue, 3 Jun 2008 10:27:32 +0100 Andy Whitcroft <apw@shadowen.org> wrote:
>
>>
>> [As gcc seems unable to help us out selecting the appropriate data segment
>> for the code, how about we did something like this?]
>>
>> When using printk from __init functions it would be desirable to place
>> the printk format strings in __initdata. Add a printk_init() variant
>> which does this.
>>
>> This printk_init() is necessarily a #define so that we can declare the
>> format string in static scope and mark it __initdata. We then call a
>> newly introduced __printk_init() variant which is identicle to printk() but
>> marked __init itself. By ensuring that an __init variant of printk is used
>> we get proper section violation warnings when this is used incorrectly:
>>
>> WARNING: vmlinux.o(.text+0x3): Section mismatch in reference from the
>> function something() to the variable .init.data:__printk_init_fmt.31426
>> The function something() references
>> the variable __initdata __printk_init_fmt.31426.
>> This is often because something lacks a __initdata
>> annotation or the annotation of __printk_init_fmt.31426 is wrong.
>>
>> Note I have followed printk's pattern for __cold annotations.
>>
>
> Ho hum. This give everyone another way in which to bury everyone else
> with patches.
>
> Wouldn't it be great if checkpatch were to detect
> fail-to-use-printk_init() in an __init function?
>
> oh, speaking of checkpatch: please use it :)
>
>> ---
>> include/linux/kernel.h | 10 ++++++++++
>> kernel/printk.c | 12 ++++++++++++
>> 2 files changed, 22 insertions(+), 0 deletions(-)
>> diff --git a/include/linux/kernel.h b/include/linux/kernel.h
>> index 792bf0a..7754196 100644
>> --- a/include/linux/kernel.h
>> +++ b/include/linux/kernel.h
>> @@ -180,6 +180,13 @@ struct pid;
>> extern struct pid *session_of_pgrp(struct pid *pgrp);
>>
>> #ifdef CONFIG_PRINTK
>> +#define printk_init(fmt, args...) \
>> +do { \
>> + static char __printk_init_fmt[] __initdata = fmt; \
>> + __printk_init(__printk_init_fmt, ##args); \
>> +} while (0)
>> +asmlinkage int __printk_init(const char * fmt, ...)
>> + __attribute__ ((format (printf, 1, 2))) __cold;
>> asmlinkage int vprintk(const char *fmt, va_list args)
>> __attribute__ ((format (printf, 1, 0)));
>> asmlinkage int printk(const char * fmt, ...)
>> @@ -196,6 +203,9 @@ extern int __printk_ratelimit(int ratelimit_jiffies, int ratelimit_burst);
>> extern bool printk_timed_ratelimit(unsigned long *caller_jiffies,
>> unsigned int interval_msec);
>> #else
>> +asmlinkage int printk_init(const char * fmt, ...)
>> + __attribute__ ((format (printf, 1, 2))) __cold;
>> +static inline int __cold printk_init(const char *s, ...) { return 0; }
>> static inline int vprintk(const char *s, va_list args)
>> __attribute__ ((format (printf, 1, 0)));
>> static inline int vprintk(const char *s, va_list args) { return 0; }
>> diff --git a/kernel/printk.c b/kernel/printk.c
>> index 8fb01c3..992a5c0 100644
>> --- a/kernel/printk.c
>> +++ b/kernel/printk.c
>> @@ -616,6 +616,18 @@ asmlinkage int printk(const char *fmt, ...)
>> return r;
>> }
>>
>> +asmlinkage __init int __printk_init(const char *fmt, ...)
>> +{
>> + va_list args;
>> + int r;
>> +
>> + va_start(args, fmt);
>> + r = vprintk(fmt, args);
>> + va_end(args);
>> +
>> + return r;
>> +}
>
> We're going to want to be able to call printk_init() from modules.
> Please fix and test that, if we decide to proceed.
>
> Oh, and we're going to need printk_meminit() and printk_cpuinit() and
> whatever.
>
> Which probably means that __printk_init() can't be __init, unless all
> the CONFIG_ settings which control __cpuinit, __meminit etc are blowing
> in the right direction.
>
> It would be good if we could get some idea of the savings here, because
> boy this is going to be a pain.
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>
--
Kinds regards,
MinChan Kim
^ permalink raw reply
* Re: [PATCH] console - Add configurable support for console charset translation
From: Dave Hylands @ 2008-06-04 7:36 UTC (permalink / raw)
To: Holger Schurig; +Cc: linux-tiny, Matt Mackall, Rob Landley, linux-embedded
In-Reply-To: <200806040904.13483.hs4233@mail.mn-solutions.de>
> Another thing that I just noticed: According to
>
> http://vger.kernel.org/vger-lists.html#linux-embedded
Mike Frysinger setup the following on May 27:
<http://news.gmane.org/gmane.linux.kernel.embedded>
--
Dave Hylands
Vancouver, BC, Canada
http://www.DaveHylands.com/
^ permalink raw reply
* Re: [PATCH] console - Add configurable support for console charset translation
From: Holger Schurig @ 2008-06-04 7:04 UTC (permalink / raw)
To: linux-tiny; +Cc: Matt Mackall, Rob Landley, linux-embedded
In-Reply-To: <1212528641.3953.70.camel@calx>
> Linux-embedded is the place to be, folks.
Another thing that I just noticed: According to
http://vger.kernel.org/vger-lists.html#linux-embedded
there is no mailing list archive for this list.
^ permalink raw reply
* Re: [PATCH] console - Add configurable support for console charset translation
From: Holger Schurig @ 2008-06-04 7:02 UTC (permalink / raw)
To: linux-tiny; +Cc: Matt Mackall, Rob Landley, linux-embedded
In-Reply-To: <1212528641.3953.70.camel@calx>
> Linux-embedded is the place to be, folks.
This is the first time I've heard about linux-embedded.
^ permalink raw reply
* Re: [PATCH] console - Add configurable support for console charset translation
From: Josh Boyer @ 2008-06-04 3:13 UTC (permalink / raw)
To: David Woodhouse; +Cc: Tim Bird, linux-tiny, linux-embedded, linux kernel
In-Reply-To: <1212541541.4042.41.camel@shinybook.infradead.org>
On Wed, 04 Jun 2008 02:05:40 +0100
David Woodhouse <dwmw2@infradead.org> wrote:
> On Tue, 2008-06-03 at 20:03 -0500, Josh Boyer wrote:
> > On Wed, 04 Jun 2008 01:16:37 +0100
> > David Woodhouse <dwmw2@infradead.org> wrote:
> >
> > > On Tue, 2008-06-03 at 17:01 -0700, Tim Bird wrote:
> > > > This is clearly an improvement. But it is missing this part of the
> > > > original patch:
> > >
> > > Oops, well spotted. I've updated the patch in the git tree; thanks.
> >
> > What git tree?
>
> git.infradead.org/embedded-2.6.git
Do you have plans to get that in -mm, or linux-next?
(Please say yes).
josh
^ permalink raw reply
* Re: [PATCH] console - Add configurable support for console charset translation
From: Paul Mundt @ 2008-06-04 2:56 UTC (permalink / raw)
To: H. Peter Anvin; +Cc: Rob Landley, linux-tiny, linux-embedded
In-Reply-To: <4845C7A3.1020607@zytor.com>
On Tue, Jun 03, 2008 at 03:37:23PM -0700, H. Peter Anvin wrote:
> Rob Landley wrote:
> >>Actually, lots have frame buffers these days.
> >
> >Cell phones, for instance.
>
> Sure, but do you want to use them as consoles?
>
Unless your name is Pavel, no one actually wants a console on their
phone. So no, just framebuffers, as is to be expected :-)
^ permalink raw reply
* Re: [PATCH] console - Add configurable support for console charset translation
From: David Woodhouse @ 2008-06-04 1:05 UTC (permalink / raw)
To: Josh Boyer; +Cc: Tim Bird, linux-tiny, linux-embedded, linux kernel
In-Reply-To: <20080603200349.3b1957e1@vader.jdub.homelinux.org>
On Tue, 2008-06-03 at 20:03 -0500, Josh Boyer wrote:
> On Wed, 04 Jun 2008 01:16:37 +0100
> David Woodhouse <dwmw2@infradead.org> wrote:
>
> > On Tue, 2008-06-03 at 17:01 -0700, Tim Bird wrote:
> > > This is clearly an improvement. But it is missing this part of the
> > > original patch:
> >
> > Oops, well spotted. I've updated the patch in the git tree; thanks.
>
> What git tree?
git.infradead.org/embedded-2.6.git
--
dwmw2
^ permalink raw reply
* Re: [PATCH] console - Add configurable support for console charset translation
From: Josh Boyer @ 2008-06-04 1:03 UTC (permalink / raw)
To: David Woodhouse; +Cc: Tim Bird, linux-tiny, linux-embedded, linux kernel
In-Reply-To: <1212538597.4042.29.camel@shinybook.infradead.org>
On Wed, 04 Jun 2008 01:16:37 +0100
David Woodhouse <dwmw2@infradead.org> wrote:
> On Tue, 2008-06-03 at 17:01 -0700, Tim Bird wrote:
> > This is clearly an improvement. But it is missing this part of the
> > original patch:
>
> Oops, well spotted. I've updated the patch in the git tree; thanks.
What git tree?
josh
^ permalink raw reply
* Re: [PATCH] console - Add configurable support for console charset translation
From: David Woodhouse @ 2008-06-04 0:16 UTC (permalink / raw)
To: Tim Bird; +Cc: linux-tiny, linux-embedded, linux kernel
In-Reply-To: <4845DB46.3020308@am.sony.com>
On Tue, 2008-06-03 at 17:01 -0700, Tim Bird wrote:
> This is clearly an improvement. But it is missing this part of the
> original patch:
Oops, well spotted. I've updated the patch in the git tree; thanks.
(that's what comes of applying patches by hand -- I _knew_ I had to do
that hunk, but managed to forget about it within the space of about 30
seconds when I was doing the other parts. I plead stupidity)
--
dwmw2
^ permalink raw reply
* Re: [PATCH] console - Add configurable support for console charset translation
From: Tim Bird @ 2008-06-04 0:01 UTC (permalink / raw)
To: David Woodhouse; +Cc: linux-tiny, linux-embedded, linux kernel
In-Reply-To: <1212500751.16924.322.camel@pmac.infradead.org>
David Woodhouse wrote:
> On Mon, 2008-06-02 at 15:37 -0700, Tim Bird wrote:
>> With CONSOLE_TRANSLATIONS turned off, this saves about 6K
>> on my kernel configured for an ARM development board (OMAP
>> 5912 OSK). In embedded products I'm familiar with,
>> console translations are not needed.
>>
>> This was taken from the Linux-tiny project and updated slightly
>> for 2.6.25.
>
> I prefer it like this... we can drop consolemap.o and
> consolemap_deftbl.o from the build completely. It saves 7.2KiB on a
> ppc32 build here.
>
> diff --git a/drivers/char/Kconfig b/drivers/char/Kconfig
> index 595a925..f740190 100644
> --- a/drivers/char/Kconfig
> +++ b/drivers/char/Kconfig
> ...
This is clearly an improvement. But it is missing this part of the
original patch:
--- a/drivers/char/vt.c
+++ b/drivers/char/vt.c
@@ -2198,7 +2198,11 @@ rescan_last_byte:
c = 0xfffd;
tc = c;
} else { /* no utf or alternate charset mode */
+#ifdef CONFIG_CONSOLE_TRANSLATIONS
tc = vc->vc_translate[vc->vc_toggle_meta ? (c | 0x80) : c];
+#else
+ tc = c;
+#endif
}
/* If the original code was a control character we
With the set_translate function stubbed, and the actual translation
operation left intact, I think the code might have problems.
I ran your patch fine on my OSK board here, but I must not have hit a
character translation case.
-- Tim
=============================
Tim Bird
Architecture Group Chair, CE Linux Forum
Senior Staff Engineer, Sony Corporation of America
=============================
^ permalink raw reply
* Re: [PATCH] console - Add configurable support for console charset translation
From: H. Peter Anvin @ 2008-06-03 22:37 UTC (permalink / raw)
To: Rob Landley; +Cc: Matt Mackall, Tim Bird, linux-tiny, linux-embedded
In-Reply-To: <200806031631.09104.rob@landley.net>
Rob Landley wrote:
>> Actually, lots have frame buffers these days.
>
> Cell phones, for instance.
Sure, but do you want to use them as consoles?
-hpa
^ permalink raw reply
* Re: initramfs size limitation
From: Mike Frysinger @ 2008-06-03 21:54 UTC (permalink / raw)
To: Rob Landley; +Cc: pwilshire, linux-embedded
In-Reply-To: <200806031559.55526.rob@landley.net>
On Tue, Jun 3, 2008 at 4:59 PM, Rob Landley wrote:
> On Sunday 01 June 2008 19:25:11 Mike Frysinger wrote:
>> On Sun, Jun 1, 2008 at 8:03 PM, Rob Landley wrote:
>> > fact it would be really nice if qemu grew blackfin support because
>> > messing
>>
>> i imagine it would be ... too bad qemu lacks real documentation
>
> Actually there was documentation on the old stuff if you knew where to dig it
> up (a good starting point was Fabrice's old usenix paper and then I had some
> links I'd collected from there)
yes, but spotiness doesnt help
> but they just ripped out the old code
> generator in favor of a new one
> (See http://lists.gnu.org/archive/html/qemu-devel/2008-02/msg00011.html ) so
> there's not much point in going there right now. (Hopefully they'll have a
> 1.0 release within our lifetimes.)
i'm aware the tcg stuff exists, but that's about it. only hope for
new ports atm is to contract a qemu guy. not that the qemu guys would
mind. thanks for the link.
-mike
^ permalink raw reply
* Re: [PATCH] console - Add configurable support for console charset translation
From: Rob Landley @ 2008-06-03 21:31 UTC (permalink / raw)
To: Matt Mackall; +Cc: H. Peter Anvin, Tim Bird, linux-tiny, linux-embedded
In-Reply-To: <1212507964.3953.67.camel@calx>
On Tuesday 03 June 2008 10:46:04 Matt Mackall wrote:
> On Mon, 2008-06-02 at 18:34 -0700, H. Peter Anvin wrote:
> > Tim Bird wrote:
> > > With CONSOLE_TRANSLATIONS turned off, this saves about 6K
> > > on my kernel configured for an ARM development board (OMAP
> > > 5912 OSK). In embedded products I'm familiar with,
> > > console translations are not needed.
> >
> > On most embedded products I'm familiar with, you wouldn't have virtual
> > consoles at all...?
>
> Actually, lots have frame buffers these days.
Cell phones, for instance.
My research is out of date now, but back in 2007 I tried to give a sense of
scale at my cross compiling tutorial at OLS. A cut and paste from my
(18 month old) notes:
--------------------------------------------------------------------------
According to http://eetimes.eu/showArticle.jhtml?articleID=199702110
Arm shipped the processors in 250 million "smart phones" in 2006 (83%
market share in that niche).
According to ARM Inc. quarterly results for Q1 2007:
http://media.corporate-ir.net/media_files/irol/19/197211/press/Q12007EarningsRelease.pdf
In the first quarter of 2007, the licensees of ARM Inc. shipped 724 million
ARM processors. In one quarter. (ARM Inc. collected $45 million in license
fees, which is an average of 6.2 cents/processor.)
In a May 23 2006 presentation to ARM investors, ARM Inc. estimated its 2006
market share at 80-90% of the cell phone market, but only 21% of the larger
market. (The embedded world is big, folks.)
Drew highlighted five key growth areas for ARM between now and 2010...
[The first four are] set-top-box, high-definition television and DVD systems,
solid-state and hard disk drive storage, automotive electronics and 32-bit
microcontrollers. In these areas in 2006, ARM had market shares of 14, 20,
5 and 13 percent respectively... They are also market sectors that will
represent a total available annual market of nearly a billion cores or more
each in 2010. The biggest annual market opportunities are likely to be
automotive and 32-bit microcontrollers at 2.0 billion and 1.9 billion cores
each.
ARM's fifth key target market is the smart phone, where it is already
enjoying success. In this area the company shipped 250 million units in 2006
and has a market share of 83 percent, according to its own estimates.
--------------------------------------------------------------------------
(And that's why Apple did the iPhone.)
Speaking of set-top boxes (another thing with a framebuffer), Netflix is
transitioning its business model to video-on-demand, with a $99 set top box
that runs Linux on Arm:
http://www.linuxdevices.com/news/NS8633598605.html
Here's the CEO of netflix saying the DVD-by-mail business will peak within
the next 5 years:
http://www.reuters.com/article/marketsNews/idINN2843042120080528?rpc=44
And the manufacturer almost immediately went to a 10 day wait on delivery of
new devices:
http://forums.roku.com/viewtopic.php?t=16722
(And that's why Apple did Apple TV.)
Rob
--
"One of my most productive days was throwing away 1000 lines of code."
- Ken Thompson.
^ permalink raw reply
page: next (older) | prev (newer) | latest
- recent:[subjects (threaded)|topics (new)|topics (active)]
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox