On Thu, Sep 06, 2007 at 10:43:49PM +0200, Oleg Verych wrote:
quoted text
> On Thu, Sep 06, 2007 at 02:21:43PM +0200, Adrian Bunk wrote:
> []
> > > You've did a tool. Documenting this tool to have it available for
> > > testers/janitors/maintainers is a better way, than to have all that
> > > opinions/problems with merging-to-mainline.
> > 
> > There is no problem with his patch.
> > 
> > His patch improves the build process.
> 
> I would like to know timing, btw. Size, especially shown 1%, doesn't
> matter if link time increased dramatically. `Allyes' config, when i
> had fast and rammish machine was terrible thing (last winter). If 32
> cores/cpus is will of author, then i'm even more suspicious.

For non-developers size and speed of the kernel matter much more than 
compile time.

When you go towards embedded systems with limited resources a 1% size 
decrease would often be worth it even if it would (hypothetically) 
increase the compile time by a factor of 10.

quoted text
>...
> > > > I don't understand why you are opposed to toolchain helping
> > > > humans to get optimized result. But it's fine with me.
> > > > I won't force anyone to select CONFIG_DISCARD_UNUSED_SECTIONS.
> > > 
> > > That's why. It's treating symptoms, isn't it?
> > 
> > There's nothing that requires treatment.
> 
> [Help for] The developers/contributors of those drivers, no?
>...

They did everything right.

You should better try to understand the problem first before behaving as 
if you knew everything better than everyone else...

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

-
unsubscribe notice
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/