diff options
author | Austin Clements <austin@google.com> | 2017-10-27 15:20:21 -0400 |
---|---|---|
committer | Austin Clements <austin@google.com> | 2017-10-30 16:33:55 +0000 |
commit | 15d6ab69fbd8c84cde109def59c7e002296c19e8 (patch) | |
tree | 4e41f0e94965a21024d4c3bd13011edfea04922c /src/runtime/asm_mips64x.s | |
parent | 67a7d5d88503646a7e411cf5ae9c38e485b9be1b (diff) | |
download | go-15d6ab69fbd8c84cde109def59c7e002296c19e8.tar.gz go-15d6ab69fbd8c84cde109def59c7e002296c19e8.zip |
runtime: make systemstack tail call if already switched
Currently systemstack always calls its argument, even if we're already
on the system stack. Unfortunately, traceback with _TraceJump stops at
the first systemstack it sees, which often cuts off runtime stacks
early in profiles.
Fix this by performing a tail call if we're already on the system
stack. This eliminates it from the traceback entirely, so it won't
stop prematurely (or all get mushed into a single node in the profile
graph).
Change-Id: Ibc69e8765e899f8d3806078517b8c7314da196f4
Reviewed-on: https://go-review.googlesource.com/74050
Reviewed-by: Cherry Zhang <cherryyz@google.com>
Reviewed-by: Keith Randall <khr@golang.org>
Diffstat (limited to 'src/runtime/asm_mips64x.s')
-rw-r--r-- | src/runtime/asm_mips64x.s | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/src/runtime/asm_mips64x.s b/src/runtime/asm_mips64x.s index 3510853804..12cea00adc 100644 --- a/src/runtime/asm_mips64x.s +++ b/src/runtime/asm_mips64x.s @@ -214,9 +214,12 @@ switch: noswitch: // already on m stack, just call directly + // Using a tail call here cleans up tracebacks since we won't stop + // at an intermediate systemstack. MOVV 0(REGCTXT), R4 // code pointer - JAL (R4) - RET + MOVV 0(R29), R31 // restore LR + ADDV $8, R29 + JMP (R4) /* * support for morestack |