On 21.01.21 19:47, Richard Henderson wrote: > From: Roman Bolshakov <r.bolsha...@yadro.com> > > Pages can't be both write and executable at the same time on Apple > Silicon. macOS provides public API to switch write protection [1] for > JIT applications, like TCG. > > 1. > https://developer.apple.com/documentation/apple_silicon/porting_just-in-time_compilers_to_apple_silicon > > Signed-off-by: Roman Bolshakov <r.bolsha...@yadro.com> > Message-Id: <20210113032806.18220-1-r.bolsha...@yadro.com> > [rth: Inline the qemu_thread_jit_* functions; > drop the MAP_JIT change for a follow-on patch.] > Signed-off-by: Richard Henderson <richard.hender...@linaro.org> > --- > > Supercedes: <20210113032806.18220-1-r.bolsha...@yadro.com> > > This is the version of Roman's patch that I'm queuing to tcg-next. > What's missing from the full "Fix execution" patch is setting MAP_JIT > for !splitwx in alloc_code_gen_buffer().
This patch segfaults in tcg_out32() for me if I add the MAP_JIT flag manually to the mmap call: (lldb) bt * thread #3, stop reason = EXC_BAD_ACCESS (code=2, address=0x118000000) * frame #0: 0x0000000100297e8c qemu-system-x86_64`tcg_prologue_init [inlined] tcg_out32(s=0x0000000100bb64c0, v=2847570941) at tcg.c:250:24 [opt] frame #1: 0x0000000100297e7c qemu-system-x86_64`tcg_prologue_init [inlined] tcg_out_insn_3314(s=0x0000000100bb64c0, insn=2847539200, r1=TCG_REG_X29, r2=TCG_REG_X30, rn=TCG_REG_SP, ofs=-96, pre=true, w=true) at tcg-target.c.inc:666 [opt] frame #2: 0x0000000100297e7c qemu-system-x86_64`tcg_prologue_init [inlined] tcg_target_qemu_prologue(s=0x0000000100bb64c0) at tcg-target.c.inc:2858 [opt] frame #3: 0x0000000100297e7c qemu-system-x86_64`tcg_prologue_init(s=0x0000000100bb64c0) at tcg.c:1116 [opt] frame #4: 0x00000001002d7ab8 qemu-system-x86_64`tcg_exec_init(tb_size=<unavailable>, splitwx=<unavailable>) at translate-all.c:1349:5 [opt] frame #5: 0x000000010028d690 qemu-system-x86_64`tcg_init(ms=<unavailable>) at tcg-all.c:113:5 [opt] frame #6: 0x000000010007d540 qemu-system-x86_64`accel_init_machine(accel=0x00000001020c9ec0, ms=0x00000001020c6880) at accel.c:55:11 [opt] frame #7: 0x00000001002b90f0 qemu-system-x86_64`do_configure_accelerator(opaque=0x000000016ff12ea0, opts=0x00000001020c9e30, errp=0x0000000100bc18e0) at vl.c:2148:11 [opt] frame #8: 0x0000000100482c00 qemu-system-x86_64`qemu_opts_foreach(list=<unavailable>, func=(qemu-system-x86_64`do_configure_accelerator at vl.c:2125), opaque=0x000000016ff12ea0, errp=0x0000000100bc18e0) at qemu-option.c:1147:14 [opt] frame #9: 0x00000001002b6d48 qemu-system-x86_64`qemu_init [inlined] configure_accelerators(progname=<unavailable>) at vl.c:2216:10 [opt] frame #10: 0x00000001002b6bd8 qemu-system-x86_64`qemu_init(argc=<unavailable>, argv=<unavailable>, envp=<unavailable>) at vl.c:3484 [opt] frame #11: 0x0000000100007aac qemu-system-x86_64`qemu_main(argc=3, argv=0x000000016fdff848, envp=<unavailable>) at main.c:49:5 [opt] frame #12: 0x000000010001dd34 qemu-system-x86_64`call_qemu_main(opaque=0x0000000000000000) at cocoa.m:1714:14 [opt] frame #13: 0x0000000100477c1c qemu-system-x86_64`qemu_thread_start(args=<unavailable>) at qemu-thread-posix.c:521:9 [opt] frame #14: 0x000000019846106c libsystem_pthread.dylib`_pthread_start + 320 Alex