Re: [PATCH] arch: configuration, deleting 'CONFIG_BUG' since always need it.

Arnd Bergmann
2013-05-23T08:54:10Z

On Thursday 23 May 2013, Geert Uytterhoeven wrote:
> > -config BUG
> > - bool "BUG() support" if EXPERT
> > - default y
> > - help
> > - Disabling this option eliminates support for BUG and WARN, reducing
> > - the size of your kernel image and potentially quietly ignoring
> > - numerous fatal conditions. You should only consider disabling this
> > - option for embedded systems with no facilities for reporting errors.
> > - Just say Y.
>
> ... It's about reducing memory size on devices where you can't show bug or
> warning messages.
>
> > So need get rid of 'CONFIG_BUG', and let it always enabled everywhere.
>
> So please keep it.

Agreed. The one annoying property of disabling BUG() support is that it causes
a large number of build warnings since the compiler now has to assume that a lot
of code is reachable when it is normally annotate as unreachable.

When I do "randconfig" tests, I always turn on CONFIG_BUG because of this.

Arnd