From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: [PATCH] arch: configuration, deleting 'CONFIG_BUG' since always need it. Date: Thu, 23 May 2013 10:54:10 +0200 Message-ID: <201305231054.10735.arnd@arndb.de> References: <519DCBEF.3090208@asianux.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: linuxppc-dev-bounces+glppe-linuxppc-embedded-2=m.gmane.org@lists.ozlabs.org Sender: "Linuxppc-dev" To: Geert Uytterhoeven Cc: Catalin Marinas , Linux-sh list , Chen Gang , Heiko Carstens , "paulus@samba.org" , "H. Peter Anvin" , Michel Lespinasse , Hans-Christian Egtvedt , Linux-Arch , linux-s390@vger.kernel.org, Russell King - ARM Linux , Yoshinori Sato , Richard Weinberger , Helge Deller , the arch/x86 maintainers , "James E.J. Bottomley" , "mingo@redhat.com" , Frederic Weisbecker , Paul McKenney , =?utf-8?q?H=C3=A5vard_Skinnemoen?= , Serge Hallyn , Mike List-Id: linux-arch.vger.kernel.org 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