On 07/07/20 11:03, Thomas Huth wrote: >> +++ b/tests/qtest/libqtest.c >> @@ -173,7 +173,12 @@ static void kill_qemu(QTestState *s) >> fprintf(stderr, "%s:%d: kill_qemu() detected QEMU death " >> "from signal %d (%s)%s\n", >> __FILE__, __LINE__, sig, signame, dump); >> - abort(); >> + if (WCOREDUMP(wstatus)) { >> + /* Preserve child coredump */ >> + exit(1); >> + } else { >> + abort(); >> + } >> } >> } > Would it maybe rather make sense to always use exit(1) unconditionally here?
But why is it a problem to overwrite the child core dump? Aren't both stashed away if you use the core.PID name as is common? Paolo