From 6cc5de60edabe0c05c8c2cb73e23a7634bdd1c8c Mon Sep 17 00:00:00 2001 From: Libo Chen Date: Mon, 29 Apr 2013 16:04:07 +0400 Subject: [PATCH] session03: change CLONE_STACK_SIZE from PAGE_SIZE to 4096 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Update to patch "check: Introduce --ms option" (commit: 379abca968850d0fb66f42115178b040b3a1f5bb) I find this error in my box: cc -g -O2 -Wall -Werror -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=0 -iquote ../../lib/arch/x86/include -I../../lib session03.c ../../lib/libzdtmtst.a -o session03 session03.c:112:7: error: variably modified ‘stack’ at file scope The reason is : "#define CLONE_STACK_SIZE PAGE_SIZE" is conflict with "#define PAGE_SIZE (sysconf(_SC_PAGESIZE))" defined in . I think CLONE_STACK_SIZE should be seted 4096 as session02.c. If this patch is appropriate, please let me know. Signed-off-by: Libo Chen Signed-off-by: Qiang Huang Acked-by: Andrew Vagin Signed-off-by: Pavel Emelyanov --- test/zdtm/live/static/session03.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/zdtm/live/static/session03.c b/test/zdtm/live/static/session03.c index 6cb8e4dcb..0c761b383 100644 --- a/test/zdtm/live/static/session03.c +++ b/test/zdtm/live/static/session03.c @@ -106,7 +106,7 @@ static void mainloop() handle_command(); } -#define CLONE_STACK_SIZE PAGE_SIZE +#define CLONE_STACK_SIZE 4096 /* All arguments should be above stack, because it grows down */ struct clone_args { char stack[CLONE_STACK_SIZE];