From patchwork Thu Jun 14 13:03:18 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Szabolcs Nagy X-Patchwork-Id: 27847 Received: (qmail 118854 invoked by alias); 14 Jun 2018 13:03:31 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Delivered-To: mailing list libc-alpha@sourceware.org Received: (qmail 118683 invoked by uid 89); 14 Jun 2018 13:03:29 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-25.6 required=5.0 tests=AWL, BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, KAM_SHORT, RCVD_IN_DNSWL_NONE, SPF_HELO_PASS, SPF_PASS autolearn=ham version=3.3.2 spammy= X-HELO: EUR04-DB3-obe.outbound.protection.outlook.com Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Szabolcs.Nagy@arm.com; Cc: nd@arm.com Subject: [COMMITTED] aarch64: Use an ifunc/VDSO to implement gettimeofday in shared glibc To: sellcey@cavium.com, libc-alpha References: <1525975253.28825.227.camel@cavium.com> <0b2d054c-d016-d825-780a-27e2b7c75c6a@arm.com> <1526425669.8676.49.camel@cavium.com> <3e69a4f6-55a2-35b7-1047-0ff2662eb513@arm.com> <1527020758.1923.18.camel@cavium.com> From: Szabolcs Nagy Message-ID: <8dc9bbde-7644-f916-eb24-9d7dd5b3c07d@arm.com> Date: Thu, 14 Jun 2018 14:03:18 +0100 User-Agent: Mozilla/5.0 (X11; Linux aarch64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 MIME-Version: 1.0 In-Reply-To: <1527020758.1923.18.camel@cavium.com> X-ClientProxiedBy: LO2P265CA0154.GBRP265.PROD.OUTLOOK.COM (2603:10a6:600:9::22) To AM0PR08MB3283.eurprd08.prod.outlook.com (2603:10a6:208:5e::28) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-HT: Tenant X-MS-Office365-Filtering-Correlation-Id: 60e1d958-f29f-442e-c121-08d5d1f73639 X-MS-TrafficTypeDiagnostic: AM0PR08MB3283: NoDisclaimer: True X-Exchange-Antispam-Report-Test: UriScan:(180628864354917)(94056789713001); X-MS-Exchange-SenderADCheck: 1 X-Forefront-PRVS: 0703B549E4 Received-SPF: None (protection.outlook.com: arm.com does not designate permitted sender hosts) SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: arm.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 14 Jun 2018 13:03:23.2810 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 60e1d958-f29f-442e-c121-08d5d1f73639 X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: f34e5979-57d9-4aaa-ad4d-b122a662184d X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM0PR08MB3283 On 22/05/18 21:25, Steve Ellcey wrote: > Any chance you could add the comments that you think are needed and > check the patch in?  It seems like you and Adhemerval are both happy > with the code, it is just the comments (and the test if you want to use > the nanosleep test instead of adding the new ones) that need fixing. i removed the test as it cannot really test if vdso is in use or not, i might do something with ifunc + bindnow testing separately. committed as: This patch uses an ifunc to implement gettimeofday in the shared libc. This is faster compared to the vsyscall mechanism that has to check a global pointer, demangle it and call it indirectly when the VDSO is present. Resolving the gettimeofday symbol directly to the VDSO code is safe because there are no failures that the libc has to handle by setting errno like in a generic vsyscall (the only failure when the VDSO code falls back to a syscall is EFAULT, but passing an invalid pointer is undefined behaviour so returning -EFAULT is fine). If the kernel supports the VDSO interface we use it for extern calls, otherwise the old vsyscall method is used which falls back to a syscall. The static version of gettimeofday continues to use a syscall, libc.so internal calls use the old vsyscall method. 2018-06-14 Steve Ellcey Szabolcs Nagy * sysdeps/unix/sysv/linux/aarch64/gettimeofday.c: New file. diff --git a/sysdeps/unix/sysv/linux/aarch64/gettimeofday.c b/sysdeps/unix/sysv/linux/aarch64/gettimeofday.c new file mode 100644 index 0000000000..8ff83e4e7d --- /dev/null +++ b/sysdeps/unix/sysv/linux/aarch64/gettimeofday.c @@ -0,0 +1,71 @@ +/* Copyright (C) 2018 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 + . */ + +/* Get the current time of day and timezone information, + putting it into *tv and *tz. If tz is null, *tz is not filled. + Returns 0 on success, -1 on errors. */ + +#ifdef SHARED + +# define __gettimeofday __redirect___gettimeofday +# include +# undef __gettimeofday +# define HAVE_VSYSCALL +# include +# include + +/* Used as a fallback in the ifunc resolver if VDSO is not available + and for libc.so internal __gettimeofday calls. */ + +static int +__gettimeofday_vsyscall (struct timeval *tv, struct timezone *tz) +{ + return INLINE_VSYSCALL (gettimeofday, 2, tv, tz); +} + +/* PREPARE_VERSION will need an __LP64__ ifdef when ILP32 support + goes in. See _libc_vdso_platform_setup in + sysdeps/unix/sysv/linux/aarch64/init-first.c. */ + +# undef INIT_ARCH +# define INIT_ARCH() \ + PREPARE_VERSION (linux_version, "LINUX_2.6.39", 123718537); \ + void *vdso_gettimeofday = \ + _dl_vdso_vsym ("__kernel_gettimeofday", &linux_version); + +libc_ifunc_hidden (__redirect___gettimeofday, __gettimeofday, + vdso_gettimeofday ?: (void *) __gettimeofday_vsyscall) + +__hidden_ver1 (__gettimeofday_vsyscall, __GI___gettimeofday, + __gettimeofday_vsyscall); + +#else + +# include +# include +int +__gettimeofday (struct timeval *tv, struct timezone *tz) +{ + return INLINE_SYSCALL (gettimeofday, 2, tv, tz); +} +libc_hidden_def (__gettimeofday) + +#endif + +weak_alias (__gettimeofday, gettimeofday) +libc_hidden_weak (gettimeofday)