mbox series

[0/3] riscv: cover to ARCH_STACKWALK

Message ID 20201113064223.103530-1-wangkefeng.wang@huawei.com (mailing list archive)
Headers show
Series riscv: cover to ARCH_STACKWALK | expand

Message

Kefeng Wang Nov. 13, 2020, 6:42 a.m. UTC 
Use the newer common infrastructure and simpler arch_stack_walk() interface
for stack walk.

Kefeng Wang (3):
  riscv: Cleanup stacktrace
  riscv: Make stack walk callback consistent with generic code
  riscv: Enable ARCH_STACKWALK

 arch/riscv/Kconfig                  |  1 +
 arch/riscv/include/asm/stacktrace.h | 17 ++++++++
 arch/riscv/kernel/perf_callchain.c  | 10 +----
 arch/riscv/kernel/stacktrace.c      | 62 +++++++----------------------
 4 files changed, 34 insertions(+), 56 deletions(-)
 create mode 100644 arch/riscv/include/asm/stacktrace.h

Comments

Palmer Dabbelt Nov. 21, 2020, 3:01 a.m. UTC | #1 
On Thu, 12 Nov 2020 22:42:20 PST (-0800), wangkefeng.wang@huawei.com wrote:
> Use the newer common infrastructure and simpler arch_stack_walk() interface
> for stack walk.
>
> Kefeng Wang (3):
>   riscv: Cleanup stacktrace
>   riscv: Make stack walk callback consistent with generic code
>   riscv: Enable ARCH_STACKWALK
>
>  arch/riscv/Kconfig                  |  1 +
>  arch/riscv/include/asm/stacktrace.h | 17 ++++++++
>  arch/riscv/kernel/perf_callchain.c  | 10 +----
>  arch/riscv/kernel/stacktrace.c      | 62 +++++++----------------------
>  4 files changed, 34 insertions(+), 56 deletions(-)
>  create mode 100644 arch/riscv/include/asm/stacktrace.h

Thanks, these are on for-next.
Kefeng Wang Dec. 17, 2020, 2:34 a.m. UTC | #2 
On 2020/11/21 11:01, Palmer Dabbelt wrote:
> On Thu, 12 Nov 2020 22:42:20 PST (-0800), wangkefeng.wang@huawei.com 
> wrote:
>> Use the newer common infrastructure and simpler arch_stack_walk() 
>> interface
>> for stack walk.
>>
>> Kefeng Wang (3):
>>   riscv: Cleanup stacktrace
>>   riscv: Make stack walk callback consistent with generic code
>>   riscv: Enable ARCH_STACKWALK
>>
>>  arch/riscv/Kconfig                  |  1 +
>>  arch/riscv/include/asm/stacktrace.h | 17 ++++++++
>>  arch/riscv/kernel/perf_callchain.c  | 10 +----
>>  arch/riscv/kernel/stacktrace.c      | 62 +++++++----------------------
>>  4 files changed, 34 insertions(+), 56 deletions(-)
>>  create mode 100644 arch/riscv/include/asm/stacktrace.h
>
> Thanks, these are on for-next.


hi Palmer,  I found a extra code about deleting "__INIT",  is there a 
mistaken when merged into for- next?

see link

https://kernel.googlesource.com/pub/scm/linux/kernel/git/riscv/linux/+/5cb0080f1bfdccb56d5c225d1cd648cdb33fa9bb%5E%21/#F1