Il 05/09/2013 14:19, Michal Novotny ha scritto: > This is the patch to introduce SIGILL handler to be able to trigger > SIGSEGV signal in qemu. This has been written to help debugging > state when qemu crashes by SIGSEGV as a simple reproducer to > emulate such situation in case of need.
What's wrong with "kill -11" or, within gdb, "j *0x1234"? Why do you need a SIGILL handler for this? In fact, SIGILL is a pretty bad choice: QEMU includes a JIT compiler, so a SIGILL is a relatively common thing to happen while debugging it. Also: (1) there is a known bug in qemu-thread-posix.c, which should not block SIGILL, SIGBUS, SIGSEGV, SIGFPE and SIGSYS. Without fixing that, this trick will only work for the iothread and not for the VCPU threads. If you can produce a patch for this, it would be very nice. > > + int *p = NULL; > + > + *p = 0xDEADBEEF; (2) This is undefined behavior. You probably want something like "volatile int *p = (volatile int *)(intptr_t)4;" instead. Paolo
