On 26/07/2022 21.21, Alex Bennée wrote:
Hangs have been observed in the tests and currently we don't timeout if a subprocess hangs. Rectify that.Signed-off-by: Alex Bennée <alex.ben...@linaro.org> --- v3 - expand timeout to 180 at Thomas' suggestion --- tests/qtest/qos-test.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/qtest/qos-test.c b/tests/qtest/qos-test.c index 7e1c8fc579..ac23def284 100644 --- a/tests/qtest/qos-test.c +++ b/tests/qtest/qos-test.c @@ -185,7 +185,7 @@ static void run_one_test(const void *arg) static void subprocess_run_one_test(const void *arg) { const gchar *path = arg; - g_test_trap_subprocess(path, 0, + g_test_trap_subprocess(path, 180 * G_USEC_PER_SEC, G_TEST_SUBPROCESS_INHERIT_STDOUT | G_TEST_SUBPROCESS_INHERIT_STDERR); g_test_trap_assert_passed(); }
Reviewed-by: Thomas Huth <th...@redhat.com>
