[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH] tests: replace rem = sleep(time) with g_tim
From: |
Alex Bennée |
Subject: |
Re: [Qemu-devel] [RFC PATCH] tests: replace rem = sleep(time) with g_timer |
Date: |
Fri, 11 Jan 2019 19:25:07 +0000 |
User-agent: |
mu4e 1.1.0; emacs 26.1.91 |
Greg Kurz <address@hidden> writes:
> On Fri, 11 Jan 2019 16:41:41 +0100
> Paolo Bonzini <address@hidden> wrote:
>
>> On 11/01/19 16:28, Alex Bennée wrote:
>> >> Why not g_usleep? It already does a while loop around nanosleep (which
>> >> returns the remaining time in the wait, like select but unlike sleep and
>> >> poll).
>> > Yeah I'm testing that now. However I have managed to trigger:
>> >
>> > ERROR:tests/test-qht-par.c:20:test_qht: assertion failed (rc == 0):
>> > (35584 == 0)
>>
>> I think that's a good old SIGSEGV (0x8B00).
>>
>
> Hmmm... system() returns a "wait status" that can be examined using the
> macros described in waitpid(2), and we have:
>
> /* If WIFEXITED(STATUS), the low-order 8 bits of the status. */
> #define __WEXITSTATUS(status) (((status) & 0xff00) >> 8)
>
> So this rather looks like a 139 exit status to me... Not sure how
> this can happen though.
Yeah the child segfaulted in mcount while closing down. I've started a
new thread with the details of the remaining failure modes:
Subject: Remaining CI failures
Date: Fri, 11 Jan 2019 19:10:07 +0000
Message-ID: <address@hidden>
--
Alex Bennée