lkml.org 
[lkml]   [2008]   [Nov]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] MIPS: Make BUG() __noreturn.
Ingo Molnar wrote:
> * Andrew Morton <akpm@linux-foundation.org> wrote:
>
>
>>> +static inline void __noreturn BUG(void)
>>> +{
>>> + __asm__ __volatile__("break %0" : : "i" (BRK_BUG));
>>> + /* Fool GCC into thinking the function doesn't return. */
>>> + while (1)
>>> + ;
>>> +}
>>>
>> This kind of sucks, doesn't it? It adds instructions into the
>> kernel text, very frequently on fast paths. Those instructions are
>> never executed, and we're blowing away i-cache just to quash
>> compiler warnings.
>>
>> For example, this:
>>
>> --- a/arch/x86/include/asm/bug.h~a
>> +++ a/arch/x86/include/asm/bug.h
>> @@ -22,14 +22,12 @@ do { \
>> ".popsection" \
>> : : "i" (__FILE__), "i" (__LINE__), \
>> "i" (sizeof(struct bug_entry))); \
>> - for (;;) ; \
>> } while (0)
>>
>> #else
>> #define BUG() \
>> do { \
>> asm volatile("ud2"); \
>> - for (;;) ; \
>> } while (0)
>> #endif
>>
>> _
>>
>> reduces the size of i386 mm/vmalloc.o text by 56 bytes.
>>
>
> yes - the total image effect is significantly - recently looked at how
> much larger !CONFIG_BUG builds would get if we inserted an infinite
> loop into them - it was in the 50K text range (!).
>
> but in the x86 ud2 case we could guarantee that we wont ever return
> from that exception. Mind sending a patch with a signoff, a
> description and an infinite loop in the u2d handler?
>

There are two arguments against making BUG() a noreturn:

* if you compile without BUG enabled, then it won't be noreturn anyway
* making it noreturn kills the lifetime of any variables that would
otherwise be considered alive, making the DWARF debug info at that
point less reliable (which is a pain even for post-mortem debugging)

The counter-argument is that not making it noreturn will keep variables
alive that wouldn't otherwise be, causing greater register pressure,
spillage, etc.

If adding an infinite loop really adds 50k to the image, the extra size
must come from the changes to variable lifetime rather than the loop
instructions themselves (which are only 2 bytes per instance, and we
don't have 25,000 BUGs in the kernel, do we?).

J


\
 
 \ /
  Last update: 2008-11-25 01:19    [W:2.767 / U:0.008 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site