4KSTACKS + DEBUG_STACKOVERFLOW harmful

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Linux Kernel Mailing List <linux-kernel@...>
Cc: Jesper Juhl <jesper.juhl@...>
Date: Wednesday, August 29, 2007 - 6:34 pm

Noticed today that the combination of 4KSTACKS and DEBUG_STACKOVERFLOW
config options is a bit deadly.

DEBUG_STACKOVERFLOW warns in do_IRQ if we're within THREAD_SIZE/8 of the
end of useable stack space, or 512 bytes on a 4k stack.

If we are, then it goes down the dump_stack path, which uses most, if
not all, of the remaining stack, thereby turning a well-intentioned
warning into a full-blown catastrophe.

The callchain from the warning looks something like this, with stack
usage shown as found on my x86 box:

4 dump_stack
4   show_trace
8     show_trace_log_lvl
4       dump_trace
          print_context_stack
12          print_trace_address
              print_symbol
232             __print_symbol
164               sprint_symbol
20                  printk
___
448

448 bytes to tell us that we're within 512 bytes (or less) of certain
doom... and I think there's call overhead on top of that?

The large stack usage in those 2 functions is due to big char arrays, of
size KSYM_NAME_LEN (128 bytes) and KSYM_SYMBOL_LEN (223 bytes).

IOW, the stack warning effectively reduces useful stack left in our itty
bitty 4k stacks by over 10%.

Any suggestions for ways around this?  The warning is somewhat helpful,
and I guess the obvious option is to lighten up the dump_stack path, but
it's still effectively reducing precious available stack space by some
amount.

With CONFIG_DEBUG_STACK_USAGE, we could print at oops time: "oh, and by
the way, you blew your stack" if there is no zeroed stack space left, as
a post-mortem.  Even without that option, I think we could still check
whether the *current* %esp at oops time has gone too far?  But if we
blew the stack, returned, and *then* oops, I think it'd be hard to know
without the DEBUG_STACK_USAGE option that we ran out of room.

-Eric
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
4KSTACKS + DEBUG_STACKOVERFLOW harmful, Eric Sandeen, (Wed Aug 29, 6:34 pm)
Re: 4KSTACKS + DEBUG_STACKOVERFLOW harmful, Mike Snitzer, (Wed May 28, 10:36 am)
Re: 4KSTACKS + DEBUG_STACKOVERFLOW harmful, Eric Sandeen, (Wed May 28, 11:13 am)
Re: 4KSTACKS + DEBUG_STACKOVERFLOW harmful, Denys Vlasenko, (Fri Aug 31, 7:11 am)
Re: 4KSTACKS + DEBUG_STACKOVERFLOW harmful, Jörn, (Fri Aug 31, 10:35 am)
Re: 4KSTACKS + DEBUG_STACKOVERFLOW harmful, Denys Vlasenko, (Fri Aug 31, 1:16 pm)
Re: 4KSTACKS + DEBUG_STACKOVERFLOW harmful, Jesper Juhl, (Wed Aug 29, 6:53 pm)
Re: 4KSTACKS + DEBUG_STACKOVERFLOW harmful, Eric Sandeen, (Wed Aug 29, 7:01 pm)
Re: 4KSTACKS + DEBUG_STACKOVERFLOW harmful, Kyle Moffett, (Wed Aug 29, 7:55 pm)