Re: [Qemu-devel] [PATCH v4 2/3] libqtest: fail if child coredumps

2018-05-25 Thread Thomas Huth
On 25.05.2018 14:15, Michael S. Tsirkin wrote: > On Fri, May 25, 2018 at 08:10:48AM +0200, Thomas Huth wrote: >> On 24.05.2018 20:25, Michael S. Tsirkin wrote: >>> Right now tests report OK status if QEMU crashes during cleanup. >>> Let's catch that case and fail the test. >>> >>> Signed-off-by: Mi

Re: [Qemu-devel] [PATCH v4 2/3] libqtest: fail if child coredumps

2018-05-25 Thread Michael S. Tsirkin
On Fri, May 25, 2018 at 08:10:48AM +0200, Thomas Huth wrote: > On 24.05.2018 20:25, Michael S. Tsirkin wrote: > > Right now tests report OK status if QEMU crashes during cleanup. > > Let's catch that case and fail the test. > > > > Signed-off-by: Michael S. Tsirkin > > --- > > tests/libqtest.c |

Re: [Qemu-devel] [PATCH v4 2/3] libqtest: fail if child coredumps

2018-05-25 Thread Thomas Huth
On 25.05.2018 08:10, Thomas Huth wrote: > On 24.05.2018 20:25, Michael S. Tsirkin wrote: >> Right now tests report OK status if QEMU crashes during cleanup. >> Let's catch that case and fail the test. >> >> Signed-off-by: Michael S. Tsirkin >> --- >> tests/libqtest.c | 9 - >> 1 file chan

Re: [Qemu-devel] [PATCH v4 2/3] libqtest: fail if child coredumps

2018-05-24 Thread Thomas Huth
On 24.05.2018 20:25, Michael S. Tsirkin wrote: > Right now tests report OK status if QEMU crashes during cleanup. > Let's catch that case and fail the test. > > Signed-off-by: Michael S. Tsirkin > --- > tests/libqtest.c | 9 - > 1 file changed, 8 insertions(+), 1 deletion(-) > > diff --

[Qemu-devel] [PATCH v4 2/3] libqtest: fail if child coredumps

2018-05-24 Thread Michael S. Tsirkin
Right now tests report OK status if QEMU crashes during cleanup. Let's catch that case and fail the test. Signed-off-by: Michael S. Tsirkin --- tests/libqtest.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/tests/libqtest.c b/tests/libqtest.c index 43fb97e..f869854