From 2290cf73cce1292d9345a8183fd29ae3994a9481 Mon Sep 17 00:00:00 2001 From: Adhemerval Zanella Netto Date: Thu, 12 Jan 2023 10:58:50 -0300 Subject: Linux: Do not align the stack for __clone3 All internal callers of __clone3 should provide an already aligned stack. Removing the stack alignment in __clone3 is a net gain: it simplifies the internal function contract (mask/unmask signals) along with the arch-specific code. Checked on x86_64-linux-gnu. Reviewed-by: Carlos O'Donell --- .../unix/sysv/linux/tst-misalign-clone-internal.c | 74 ---------------------- 1 file changed, 74 deletions(-) delete mode 100644 sysdeps/unix/sysv/linux/tst-misalign-clone-internal.c (limited to 'sysdeps/unix/sysv/linux/tst-misalign-clone-internal.c') diff --git a/sysdeps/unix/sysv/linux/tst-misalign-clone-internal.c b/sysdeps/unix/sysv/linux/tst-misalign-clone-internal.c deleted file mode 100644 index 8b94a74819..0000000000 --- a/sysdeps/unix/sysv/linux/tst-misalign-clone-internal.c +++ /dev/null @@ -1,74 +0,0 @@ -/* Verify that __clone_internal properly aligns the child stack. - Copyright (C) 2021-2023 Free Software Foundation, Inc. - This file is part of the GNU C Library. - - The GNU C Library is free software; you can redistribute it and/or - modify it under the terms of the GNU Lesser General Public - License as published by the Free Software Foundation; either - version 2.1 of the License, or (at your option) any later version. - - The GNU C Library is distributed in the hope that it will be useful, - but WITHOUT ANY WARRANTY; without even the implied warranty of - MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU - Lesser General Public License for more details. - - You should have received a copy of the GNU Lesser General Public - License along with the GNU C Library; if not, see - . */ - -#include -#include -#include -#include -#include -#include -#include -#include -#include -#include -#include -#include - -static int -check_stack_alignment (void *arg) -{ - puts ("in f"); - - return TEST_STACK_ALIGN () ? 1 : 0; -} - -static int -do_test (void) -{ - puts ("in do_test"); - - if (TEST_STACK_ALIGN ()) - FAIL_EXIT1 ("stack isn't aligned\n"); - -#ifdef __ia64__ -# define STACK_SIZE (256 * 1024) -#else -# define STACK_SIZE (128 * 1024) -#endif - char st[STACK_SIZE + 1]; - /* NB: Align child stack to 1 byte. */ - char *stack = PTR_ALIGN_UP (&st[0], 2) + 1; - struct clone_args clone_args = - { - .stack = (uintptr_t) stack, - .stack_size = STACK_SIZE, - }; - pid_t p = __clone_internal (&clone_args, check_stack_alignment, 0); - - /* Clone must not fail. */ - TEST_VERIFY_EXIT (p != -1); - - int e; - xwaitpid (p, &e, __WCLONE); - TEST_VERIFY (WIFEXITED (e)); - TEST_COMPARE (WEXITSTATUS (e), 0); - - return 0; -} - -#include -- cgit v1.2.3-70-g09d2