Hi Ying,

On Tue, Mar 05, 2024 at 05:51:21PM +0800, Ying Huang wrote:
> From: Ying Huang <ying.hu...@oss.cipunited.com>
> 
> Add abi_cfi, set_initial_registers_tid, unwind on mips.

Sorry for reviewing out of order. But this one looked easy enough.
The new abi_cfi, unwind and set_initial_registers_tid implementations
looks correct (even though I don't really known MIPS).

Pushed with the following ChangeLog entry added to the commit message.

        * backends/Makefile.am (mips_SRCS): Add mips_initreg.c,
        mips_cfi.c and mips_unwind.c.
        * backends/mips_init.c (mips_init): HOOK abi_cfi, unwind and
        set_initial_registers_tid. Set frame_nregs to 71.
        * backends/mips_cfi.c: New file.
        * backends/mips_initreg.c: Likewise.
        * backends/mips_unwind.c: Likewise.

It would be helpful to see if you could come up with a testcase
similar to the ones in tests/run-backtrace-core-<arch>.sh with
tests/tests/backtrace.<arch>.{exec,core}.bz2 so it can be tested from
a non-MIPS setup (as opposed to the tests/run-backtrace-native.sh and
tests/run-backtrace-native-core.sh tests).

Thanks,

Mark

Reply via email to