Re: PATCH: Add --size-check=[error|warning]

[Ingo Molnar <mingo@elte.hu>]

* Jan Beulich <JBeulich@novell.com> wrote:

> >>> On 14.03.11 at 12:02, Sedat Dilek <sedat.dilek@googlemail.com> wrote:
> > [1] Jan, what do you mean by "side-effects". Can you explain that a
> > bit more precisely?
> 
> The -B compiler option controls more than just finding "helper" binaries.
> 
> > [2] Where can someone set a "global behaviour" (hardcoded options) for
> > his/her assembler in the kernel's build-system (speaking of
> > "--size-check=[error|warning]")?
> 
> Nowhere, selecting behavior is possible only via the command line.
> 
> > [3] Can the kernel-buildsystem check for system's binutils/as version
> > and/or its features/options? If yes, where would that be and can you
> > offer a snippet for a solution?
> 
> Making the kernel build system check for certain newly introduced
> gas options would again require changes to the kernel sources,
> which is precisely what is impossible to do for past kernel releases
> (and bisection in particular).

Yes, and all the counter-arguments here continue to miss that very simple point. 
That point was made in the first post about this topic and it's still not 
acknowledged - let alone addressed.

This breakage is unnecessary and retroactively goes back 130,000 commits. A warning 
combined with not issuing the debug symbol would be just as fine and would still 
result in valid output.

Thanks,

	Ingo