From: "mame (Yusuke Endoh)" <noreply@...>
Date: 2022-04-15T09:08:50+00:00
Subject: [ruby-core:108254] [Ruby master Bug#18735] The crash reporter need to be updated for Apple silicon

Issue #18735 has been updated by mame (Yusuke Endoh).


I think your patch does not work well for me. Does it work well on your machine?

```
% sw_vers
ProductName:    macOS
ProductVersion: 11.6
BuildVersion:   20G165

% ./miniruby -e 'Process.kill "SEGV", $$'
-e:1: [BUG] Segmentation fault at 0x0000000102312568
ruby 3.2.0dev (2022-04-15T08:52:10Z master 7f81f33547) [arm64-darwin20]

*snip*

-- C level backtrace information -------------------------------------------
/Users/mame/work/ruby/./miniruby(rb_vm_bugreport+0x285e000000000960) [0x285e00010248299c]
/Users/mame/work/ruby/./miniruby(0x4d788001022bfcb8) [0x4d788001022bfcb8]
/Users/mame/work/ruby/./miniruby(0x42508001023e6e14) [0x42508001023e6e14]
/usr/lib/system/libsystem_platform.dylib(_sigtramp+0x2d78800000000038) [0x2d788001868dac44]
[0x38]

-- Other runtime information -----------------------------------------------

*snip*
```

The trace has no information of any meaningful call frames. It should include `rb_f_kill`, `vm_exec_core`, etc. In fact it does on Linux as follows.

```
$ ./miniruby -e 'Process.kill "SEGV", $$'
-e:1: [BUG] Segmentation fault at 0x000003e800093ad5
ruby 3.2.0dev (2022-04-04T07:06:16Z master 57377e5d53) [x86_64-linux]

*snip*

-- C level backtrace information -------------------------------------------
./miniruby(rb_vm_bugreport+0x5bb) [0x55f6825c129b]
./miniruby(rb_bug_for_fatal_signal+0xec) [0x55f6823a5f6c]
./miniruby(sigsegv+0x4d) [0x55f68250fc2d]
[0x7f2f7365b520]
[0x7f2f7365b75b]
./miniruby(rb_f_kill+0x239) [0x55f682510dd9]
./miniruby(vm_call_cfunc_with_frame+0x137) [0x55f6825a1207]
./miniruby(vm_exec_core+0x114) [0x55f6825b2954]
./miniruby(rb_vm_exec+0xd3) [0x55f6825a3b83]
./miniruby(rb_ec_exec_node+0xb1) [0x55f6823aaca1]
./miniruby(ruby_run_node+0x4d) [0x55f6823b014d]
./miniruby(rb_main+0x73) [0x55f682302c73]
[0x7f2f73642fd0]
[0x7f2f7364307d]
./miniruby(_start+0x25) [0x55f682302b25]

*snip*
```

----------------------------------------
Bug #18735: The crash reporter need to be updated for Apple silicon
https://bugs.ruby-lang.org/issues/18735#change-97271

* Author: byroot (Jean Boussier)
* Status: Open
* Priority: Normal
* Backport: 2.7: UNKNOWN, 3.0: UNKNOWN, 3.1: UNKNOWN
----------------------------------------
C backtrace dumping is only enabled for `x86_64` arch on macOS:  `if defined(__APPLE__) && defined(__x86_64__) && defined(HAVE_LIBUNWIND_H)`

I tested `libunwind` on M1 Mac, it seems to work fine. I think we should either remove this `x86_64` condition, or add an `arm64` condition as well. 

Proposed patch: https://github.com/ruby/ruby/pull/5770

The register dumping code would need to be implemented as well, but it's a bit beyond my capacities.

I also think we should backport the fix on older rubies so that in case of crash we get better crash reports.




-- 
https://bugs.ruby-lang.org/

Unsubscribe: <mailto:ruby-core-request@ruby-lang.org?subject=unsubscribe>
<http://lists.ruby-lang.org/cgi-bin/mailman/options/ruby-core>