[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] tests: avoid coroutine pool test crash
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH] tests: avoid coroutine pool test crash |
Date: |
Tue, 24 May 2016 08:29:26 +0800 |
User-agent: |
Mutt/1.6.1 (2016-04-27) |
On Fri, 05/20 11:00, Stefan Hajnoczi wrote:
> Skip the test_co_queue test case if the coroutine pool is not enabled.
> The test case does not work without the pool because it touches memory
> belonging to a freed coroutine (on purpose).
>
> Reported-by: Eduardo Habkost <address@hidden>
> Signed-off-by: Stefan Hajnoczi <address@hidden>
> ---
> tests/test-coroutine.c | 10 +++++++++-
> 1 file changed, 9 insertions(+), 1 deletion(-)
>
> diff --git a/tests/test-coroutine.c b/tests/test-coroutine.c
> index dd4ced9..ea7f87f 100644
> --- a/tests/test-coroutine.c
> +++ b/tests/test-coroutine.c
> @@ -369,7 +369,15 @@ static void perf_cost(void)
> int main(int argc, char **argv)
> {
> g_test_init(&argc, &argv, NULL);
> - g_test_add_func("/basic/co_queue", test_co_queue);
> +
> + /* This test assumes there is a freelist and marks freed coroutine memory
> + * with a sentinel value. If there is no freelist this would
> legitimately
> + * crash, so skip it.
> + */
> + if (CONFIG_COROUTINE_POOL) {
> + g_test_add_func("/basic/co_queue", test_co_queue);
> + }
> +
> g_test_add_func("/basic/lifecycle", test_lifecycle);
> g_test_add_func("/basic/yield", test_yield);
> g_test_add_func("/basic/nesting", test_nesting);
> --
> 2.5.5
>
>
Reviewed-by: Fam Zheng <address@hidden>