diff options
| author | LemonBoy <thatlemon@gmail.com> | 2021-04-25 19:10:11 +0200 |
|---|---|---|
| committer | LemonBoy <thatlemon@gmail.com> | 2021-04-25 19:10:11 +0200 |
| commit | 54db36cd88835ed6b9d9584d9568e9aa72e41480 (patch) | |
| tree | fa69c82ecc88148f9a2126db0bc8c0239a8d0caf /lib/std | |
| parent | 37b05742ff1544bccf7c8ae9b12c6707a5a54df2 (diff) | |
| download | zig-54db36cd88835ed6b9d9584d9568e9aa72e41480.tar.gz zig-54db36cd88835ed6b9d9584d9568e9aa72e41480.zip | |
std: Fix backtraces on sparcv9
Flush all the register windows to stack before starting the stack walk,
we may otherwise try to read garbage and crash and burn.
Add a few comptime annotations to debloat some functions.
Diffstat (limited to 'lib/std')
| -rw-r--r-- | lib/std/debug.zig | 15 |
1 files changed, 11 insertions, 4 deletions
diff --git a/lib/std/debug.zig b/lib/std/debug.zig index c84a0e0f18..6cca67f302 100644 --- a/lib/std/debug.zig +++ b/lib/std/debug.zig @@ -336,6 +336,13 @@ pub const StackIterator = struct { fp: usize, pub fn init(first_address: ?usize, fp: ?usize) StackIterator { + if (builtin.arch == .sparcv9) { + // Flush all the register windows on stack. + asm volatile ( + \\ flushw + ::: "memory"); + } + return StackIterator{ .first_address = first_address, .fp = fp orelse @frameAddress(), @@ -343,18 +350,18 @@ pub const StackIterator = struct { } // Offset of the saved BP wrt the frame pointer. - const fp_offset = if (builtin.arch.isRISCV()) + const fp_offset = if (comptime builtin.arch.isRISCV()) // On RISC-V the frame pointer points to the top of the saved register // area, on pretty much every other architecture it points to the stack // slot where the previous frame pointer is saved. 2 * @sizeOf(usize) - else if (builtin.arch.isSPARC()) + else if (comptime builtin.arch.isSPARC()) // On SPARC the previous frame pointer is stored at 14 slots past %fp+BIAS. 14 * @sizeOf(usize) else 0; - const fp_bias = if (builtin.arch.isSPARC()) + const fp_bias = if (comptime builtin.arch.isSPARC()) // On SPARC frame pointers are biased by a constant. 2047 else @@ -380,7 +387,7 @@ pub const StackIterator = struct { } fn next_internal(self: *StackIterator) ?usize { - const fp = if (builtin.arch.isSPARC()) + const fp = if (comptime builtin.arch.isSPARC()) // On SPARC the offset is positive. (!) math.add(usize, self.fp, fp_offset) catch return null else |
