lkml.org 
[lkml]   [2010]   [Apr]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/23] Make register values available to panic notifiers
David Howells wrote:
> David VomLehn <dvomlehn@cisco.com> wrote:
>
>
>> +NORET_TYPE void panic(const char *fmt, ...)
>> +{
>> + va_list args;
>> + const struct pt_regs *regs;
>> + int i;
>> +
>> + preempt_disable();
>> + bust_spinlocks(1);
>> + regs = save_ptregs(&__get_cpu_var(panic_panic_regs));
>> + va_start(args, fmt);
>> + vpanic_with_regs(regs, fmt, args);
>> + /* Since vpanic_with_regs doesn't return, we skip va_end() */
>> + /* Infinite loop so compiler doesn't complain about this returning */
>> + for (i = 0; ; )
>> + mdelay(1);
>> +}
>>
>
> Can the use of va_start() clobber lots of registers, thereby rendering the
> exercise pointless on some arches?
>
The implementations I'm familiar with only need one or two registers.
What it
*does* do is to force the contents of registers being used to pass
argument values
onto the stack. This is roughly what gcc does for asm() statements when you
tell it registers are clobbered.
> Also, can the save_ptregs() function be out of line asm? The FRV constructed
> inline statement is huge (and wrong).
>

With this implementation it has to be inline. One use of the saved
registers is to
backtrace the stack. If you call a function to save the registers, the
stack pointer
and program counter would be those of the called function, which will not be
valid after it returns. I expect that you could come up with an alternative
out-of-line function--on every processor I know, you could backtrace one
frame
to get reasonable values for those registers,. Unfortunately, you would
run the
risk of clobbering other registers by doing the function call. The more you
change register values from those in the function that calls panic(),
the less
useful this becomes. In this case, I think an inline function is worth the
effort to get working. (I'd be interested in know more details about how
tshings are broken in the FRV)
> David
>



\
 
 \ /
  Last update: 2010-04-14 22:43    [W:0.107 / U:0.032 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site