[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4] tests/libqtest: Improve kill_qemu()
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v4] tests/libqtest: Improve kill_qemu() |
Date: |
Wed, 15 Aug 2018 15:35:18 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux) |
Eric Blake <address@hidden> writes:
> On 08/13/2018 02:17 AM, Markus Armbruster wrote:
>> Eric Blake <address@hidden> writes:
>>
>>> On 08/10/2018 08:28 AM, Eric Blake wrote:
>>>> Instead of using a raw assert, print the information in an
>>>> easier to understand way:
>>>>
>>>> /i386/ahci/sanity: tests/libqtest.c:119: kill_qemu() detected QEMU death
>>>> from signal 11 (Segmentation fault) (core dumped)
>>>
>>> Well, it would help if my commit message actually matched...
>>>
>>>
>>>> + if (wstatus) {
>>>> + if (WIFEXITED(wstatus)) {
>>>> + fprintf(stderr, "%s:%d: kill_qemu() tried to terminate
>>>> QEMU "
>>>> + "process but encountered exit status %d\n",
>>>> + __FILE__, __LINE__, WEXITSTATUS(wstatus));
>>>> + } else if (WIFSIGNALED(wstatus)) {
>>>> + int sig = WTERMSIG(wstatus);
>>>> + const char *signame = strsignal(sig) ?: "unknown ???";
>>>> + const char *dump = WCOREDUMP(wstatus) ? " (dumped core)"
>>>> : "";
>>>> +
>>>> + fprintf(stderr, "%s:%d: kill_qemu() detected QEMU death "
>>>> + "from signal %d (%s)%s\n",
>>>
>>> ...the code.
>>
>> I got libqtest patches in my queue, and I could stick this patch in.
>> Would you like me to touch up the commit message when I apply? Or
>> should I expect v5?
>>
>
> If you don't mind doing the touchup (s/core dumped/dumped core/), then
> I don't need to submit v5.
Since "core dumped" is how the shell traditionally reports this, I'd
prefer to adjust the code to match the commit message. Okay?
Re: [Qemu-devel] [PATCH v4] tests/libqtest: Improve kill_qemu(), Marc-André Lureau, 2018/08/16