LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Tim Bird <tim.bird@am.sony.com>
To: Adrian Bunk <bunk@fs.tum.de>
Cc: Christoph Hellwig <hch@infradead.org>,
	linux kernel <linux-kernel@vger.kernel.org>
Subject: Re: ANNOUNCE: CE Linux Forum - Specification V1.0 draft
Date: Wed, 19 May 2004 09:59:08 -0700	[thread overview]
Message-ID: <40AB925C.50001@am.sony.com> (raw)
In-Reply-To: <20040519152706.GD22742@fs.tum.de>

Adrian Bunk wrote:
> On Mon, May 17, 2004 at 02:22:29PM -0700, Tim Bird wrote:
>>Christoph Hellwig wrote:
>>>If you want my 2Cent:
>>>
>>>- stop these rather useless specifications and provide patchkits instead
>>>- try to actually submit the patches upstream to get a feeling which
>>>  of your 'features' are compltely hopeless, which are okay and which
>>>  can better be solved in different ways.
>>
>>I should point out that some of the features specified have already been
>>submitted as patchsets. 
 >
> A good example that this is true is section 7.9.2 of your 
> "specification".
> 
> It lists under "Work in Progress":
>   Kernel SHALL be configuralble with compiler size options, such as -Os.
> 
> Besides the text in the "Rationale" being obviously wrong, this is 
> already implemented in kernel 2.6. But the people writing this
> "specification" didn't send a patch - the trivial patch was sent by 
> someone who is in no way related to your "Forum".

First, I'll point out that this spec, as you noted, is still
a work in progress.

Yes, the rationale is wrong.  Thanks for pointing that out.
I'll get it fixed before we release a spec on this.  We have
a separate agenda item in our size working group to look at
inline expansions (See section 7.9.3 where it lists candidate
projects that are not started yet.) There is already valuable
work going on in the area of inline reduction, but
unfortunately, we don't have anything to contribute to that
discussion yet.

As for the patch, you are correct that the kernel makefile system
supports compilation with -Os, and someone besides us submitted
the patch for that.  However, there is more work needed to
validate that the option doesn't break things, on many different
architectures.

I have reports from the uClinux crowd that use of
the -Os option is fairly typical for users of uClinux, and they
have no reports of breakage.  However, we want to take a methodical
approach to validating that use of this option is fully supported
by the Linux kernel.  Also, we want to test and report the size
and performance effects of the use of the flag.  This work is
not done yet, so the spec. is still under construction.

Just jamming in the compiler option is not really what we intend here.
I'll try to make sure this spec., when released, is worded to
express our requirement that the option be meaningful and safe,
rather than just supported by the build system.

If you see patches from us (or one of our members) related to this spec.,
they will be to fix issues where use of -Os breaks something in the kernel.

Thanks for the feedback.  We'll keep working on this one.

P.S. Why is "Forum" in quotes?

=============================
Tim Bird
Architecture Group Co-Chair
CE Linux Forum
Senior Staff Engineer
Sony Electronics
E-mail: Tim.Bird@am.sony.com
=============================


  reply	other threads:[~2004-05-19 16:59 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-05-17 19:05 ANNOUNCE: CE Linux Forum - Specification V1.0 draft Tim Bird
2004-05-17 19:19 ` Christoph Hellwig
2004-05-17 19:59   ` Tim Bird
2004-05-17 20:42   ` Mark Gross
2004-05-17 20:48     ` Arjan van de Ven
     [not found]     ` <20040518074854.A7348@infradead.org>
2004-05-18 19:32       ` Mark Gross
2004-05-18 19:56         ` Russell King
2004-05-18 20:45           ` Bartlomiej Zolnierkiewicz
2004-05-18 20:00         ` viro
2004-05-19 19:30       ` Tim Bird
2004-05-19 21:57         ` Russell King
2004-05-19 22:02           ` Jeff Garzik
2004-05-19 23:46           ` Tim Bird
2004-05-19 22:08         ` Theodore Ts'o
2004-05-19 23:37           ` Tim Bird
2004-05-17 21:22   ` Tim Bird
2004-05-19 15:27     ` Adrian Bunk
2004-05-19 16:59       ` Tim Bird [this message]
2004-05-19 20:16         ` Adrian Bunk
2004-05-19 20:38           ` Tim Bird
2004-05-19 20:48             ` Nicolas Pitre
2004-05-19 22:00         ` Russell King

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=40AB925C.50001@am.sony.com \
    --to=tim.bird@am.sony.com \
    --cc=bunk@fs.tum.de \
    --cc=hch@infradead.org \
    --cc=linux-kernel@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).