Re: [GIT] kbuild core updates for 2.6.39-rc1

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Mar 19, 2011 at 1:58 AM, Michal Marek <mmarek@xxxxxxx> wrote:
>
> * Section mismatch analysis enabled by default
>  - there are only a few warnings for defconfig builds,
>    all{yes,mod}config still has a lot of them. Depending on the general
>    perception of this change, we might disable it later in the rc phase
>    again.

No.

We don't enable stuff like this by default. Not when it actually
changes some really fundamental compiler flags, and thus the whole "we
can disable it late in -rc again" is a totally broken option, since if
we have any compiler issues, we'd want to know it early.

And we _know_ we are going to disable it, since it enables the "don't
inline" flag to gcc for called-once functions. So there's no question
at all what the default should be in any release kernel.

Anyway - in general, enabling options like this had better have some
_seriously_ major reasons. IOW, major known issues that just have to
be sorted out and cannot be sorted out any other way. That doesn't
seem to be the case here.

WE DO NOT EVER ENABLE NEW THINGS BY DEFAULT. I know, I know - every
developer always thinks that _his_ code is special and magically so
important that it should be enabled by default, but no, that is simply
not the case. The only reason to enable something by default is that
it used to never be a config option in the first place, so if somebody
builds a new kernel, and he used to just silently get the feature,
then the new config option is still on by default.

Not pulling. This is a pet peeve of mine. We don't use "set to 'y' on
default" as a way to trick people into enabling code. No, no, no.

                             Linus
--
To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux&nblp;USB Development]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite Secrets]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux