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 =============================
next prev parent 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: linkBe 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).