diff options
author | Matt Fleming <matt@console-pimps.org> | 2009-10-10 11:17:06 -0400 |
---|---|---|
committer | Matt Fleming <matt@console-pimps.org> | 2009-10-11 12:12:32 -0400 |
commit | c2d474d6f8b48b6698343cfc1a3630c4647aa7b2 (patch) | |
tree | 714791c763d1547dcb0faf2095876949abfcef03 /arch/sh/kernel/dwarf.c | |
parent | ed4fe7f488008f38d5f423f0bcc736b1779d6ddc (diff) |
sh: Remove any reference to recursive functions from comments
Originally, dwarf_unwind_stack() was a recursive function and it seems
that some of the old comments were never updated.
Signed-off-by: Matt Fleming <matt@console-pimps.org>
Diffstat (limited to 'arch/sh/kernel/dwarf.c')
-rw-r--r-- | arch/sh/kernel/dwarf.c | 22 |
1 files changed, 11 insertions, 11 deletions
diff --git a/arch/sh/kernel/dwarf.c b/arch/sh/kernel/dwarf.c index ce8bff45d72c..f242cd120cf1 100644 --- a/arch/sh/kernel/dwarf.c +++ b/arch/sh/kernel/dwarf.c | |||
@@ -540,7 +540,8 @@ void dwarf_free_frame(struct dwarf_frame *frame) | |||
540 | } | 540 | } |
541 | 541 | ||
542 | /** | 542 | /** |
543 | * dwarf_unwind_stack - recursively unwind the stack | 543 | * dwarf_unwind_stack - unwind the stack |
544 | * | ||
544 | * @pc: address of the function to unwind | 545 | * @pc: address of the function to unwind |
545 | * @prev: struct dwarf_frame of the previous stackframe on the callstack | 546 | * @prev: struct dwarf_frame of the previous stackframe on the callstack |
546 | * | 547 | * |
@@ -558,9 +559,9 @@ struct dwarf_frame * dwarf_unwind_stack(unsigned long pc, | |||
558 | unsigned long addr; | 559 | unsigned long addr; |
559 | 560 | ||
560 | /* | 561 | /* |
561 | * If this is the first invocation of this recursive function we | 562 | * If we're starting at the top of the stack we need get the |
562 | * need get the contents of a physical register to get the CFA | 563 | * contents of a physical register to get the CFA in order to |
563 | * in order to begin the virtual unwinding of the stack. | 564 | * begin the virtual unwinding of the stack. |
564 | * | 565 | * |
565 | * NOTE: the return address is guaranteed to be setup by the | 566 | * NOTE: the return address is guaranteed to be setup by the |
566 | * time this function makes its first function call. | 567 | * time this function makes its first function call. |
@@ -582,9 +583,8 @@ struct dwarf_frame * dwarf_unwind_stack(unsigned long pc, | |||
582 | fde = dwarf_lookup_fde(pc); | 583 | fde = dwarf_lookup_fde(pc); |
583 | if (!fde) { | 584 | if (!fde) { |
584 | /* | 585 | /* |
585 | * This is our normal exit path - the one that stops the | 586 | * This is our normal exit path. There are two reasons |
586 | * recursion. There's two reasons why we might exit | 587 | * why we might exit here, |
587 | * here, | ||
588 | * | 588 | * |
589 | * a) pc has no asscociated DWARF frame info and so | 589 | * a) pc has no asscociated DWARF frame info and so |
590 | * we don't know how to unwind this frame. This is | 590 | * we don't know how to unwind this frame. This is |
@@ -626,10 +626,10 @@ struct dwarf_frame * dwarf_unwind_stack(unsigned long pc, | |||
626 | 626 | ||
627 | } else { | 627 | } else { |
628 | /* | 628 | /* |
629 | * Again, this is the first invocation of this | 629 | * Again, we're starting from the top of the |
630 | * recurisve function. We need to physically | 630 | * stack. We need to physically read |
631 | * read the contents of a register in order to | 631 | * the contents of a register in order to get |
632 | * get the Canonical Frame Address for this | 632 | * the Canonical Frame Address for this |
633 | * function. | 633 | * function. |
634 | */ | 634 | */ |
635 | frame->cfa = dwarf_read_arch_reg(frame->cfa_register); | 635 | frame->cfa = dwarf_read_arch_reg(frame->cfa_register); |