From patchwork Fri Feb 8 18:39:10 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Adhemerval Zanella Netto X-Patchwork-Id: 157870 Delivered-To: patch@linaro.org Received: by 2002:a02:48:0:0:0:0:0 with SMTP id 69csp2284417jaa; Fri, 8 Feb 2019 10:40:13 -0800 (PST) X-Google-Smtp-Source: AHgI3IZ6P0LC1ey5fk5jkVdv13L8ypTyJ0wV8p9fCm/C/vd0KGaXeYpwGjfmeVMlBUp0D3HRGx4O X-Received: by 2002:a63:f811:: with SMTP id n17mr12149271pgh.23.1549651213390; Fri, 08 Feb 2019 10:40:13 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1549651213; cv=none; d=google.com; s=arc-20160816; b=sieuj52OJlgy13bT6RFlLDsEJMMskLcSOUlATPYuWg2xKEy4eWxmas976w9s5PmXAe 26YULchQwUFlRPRHqQMwlw8bvdhSBVlEWbiSvDU4fzXuk9uvJNsv2iLOVR96ie6DfGei KneMZPP7ZZsdyoxls/WxizfrW039bjs47Zga0iZxepa9SKU1Y5O9DZ1Rl7g1f90nq8aR 0FTuKCKIhgOdCJpqXMcTFwuH92kGKzLOXG5clIZSeZAHFa3FMrS8QO5Sc+T2mNiPP4M9 LvBLNJk7ia8LUIXGDutDAZLSk3xJayKd+smJzcJGDVAr5/j0uVIlLsf39RVxUjNv+OAy DZYA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature:delivered-to:sender:list-help:list-post:list-archive :list-subscribe:list-unsubscribe:list-id:precedence:mailing-list :dkim-signature:domainkey-signature; bh=xpzGUeUr4A5Zb759BBkshhgKpGaT6ZobE/CyjVTfibs=; b=A3yYFUfJtaBTZYmq4llywCKcduFA4cFUMjLuuVtRn5nyZNeEHNaVR4hrC6swZ0b5CZ jBB9lFmELtwOb0UavHtxddhEhNNoynjb5gyHKdw8j872FMpxXGQViuq6PiRejYdnJQh4 eClEDCO55emkYPC6co5x2nysg4TVRBROvL/N3rSfx9J9nTP6eg4Xyr02u4pRz21m/jP8 DqDsgNcOMzRetPWpt4A1UH608rwPdgIxr2l5rE433Vo/IX6sxxSm3JG+jRG/eiEz1m5q 13zippfIIDX6d6PxfT5ee2h+LL3aWheVxvcDMO8jgaN4LyoPAGJ2nJxvHcPgBYBt7hOe sBlw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@sourceware.org header.s=default header.b=EdnOhMV6; dkim=pass header.i=@linaro.org header.s=google header.b=y50uKpgA; spf=pass (google.com: domain of libc-alpha-return-99887-patch=linaro.org@sourceware.org designates 209.132.180.131 as permitted sender) smtp.mailfrom="libc-alpha-return-99887-patch=linaro.org@sourceware.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from sourceware.org (server1.sourceware.org. [209.132.180.131]) by mx.google.com with ESMTPS id e6si2633784pgd.428.2019.02.08.10.40.13 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 08 Feb 2019 10:40:13 -0800 (PST) Received-SPF: pass (google.com: domain of libc-alpha-return-99887-patch=linaro.org@sourceware.org designates 209.132.180.131 as permitted sender) client-ip=209.132.180.131; Authentication-Results: mx.google.com; dkim=pass header.i=@sourceware.org header.s=default header.b=EdnOhMV6; dkim=pass header.i=@linaro.org header.s=google header.b=y50uKpgA; spf=pass (google.com: domain of libc-alpha-return-99887-patch=linaro.org@sourceware.org designates 209.132.180.131 as permitted sender) smtp.mailfrom="libc-alpha-return-99887-patch=linaro.org@sourceware.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org DomainKey-Signature: a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id :list-unsubscribe:list-subscribe:list-archive:list-post :list-help:sender:from:to:cc:subject:date:message-id:in-reply-to :references; q=dns; s=default; b=Sbbv3GYb+iicx47XywZ1AbFtNMqNFYl zZZmb7tTzD91HUX7vD8kgGl6tIG+MQmw1sV2BLmBvG+MSR6d+1xB+8pzAbQ6U2u+ g1DC8ZwSgaMaauubqKIGaSM96vCvEY7buyfX3GxD9z9XVnJhTYm507CxblnLVko9 hlelTdvCebwA= DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=sourceware.org; h=list-id :list-unsubscribe:list-subscribe:list-archive:list-post :list-help:sender:from:to:cc:subject:date:message-id:in-reply-to :references; s=default; bh=ISlj1gN5gM75jGsqcH4MRf+ZLKM=; b=EdnOh MV6OuPIbbCcUp6yYEJKTmR/9WXOjJxisf/SlDQWEdCBXjE/Meb90+bj+8BuhWD+V USCa1g/ppipBBqCO3LGXTD7Q+SanOG26O17yWZhlxtujoxBkj+ibAftBr0F657FD reGxNiceXakuvmigdpgwyX+jdt3I1n4BZNHMDQ= Received: (qmail 77369 invoked by alias); 8 Feb 2019 18:39:26 -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 77262 invoked by uid 89); 8 Feb 2019 18:39:25 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-26.9 required=5.0 tests=BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, RCVD_IN_DNSWL_NONE, SPF_PASS autolearn=ham version=3.3.2 spammy=Sum, 2712, Len X-HELO: mail-qt1-f196.google.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=xpzGUeUr4A5Zb759BBkshhgKpGaT6ZobE/CyjVTfibs=; b=y50uKpgA38SB3KrHFW0BHMjSYJV8iY4RLp8m5oOX+R9XkyC7+EEe88OX+86G+7zF/2 OvJbNXPk2u8qbXTsWFLUBmGpC4boRCuO/A7IrkCYGpe3R43dNNu8lykyiBglKneIu4nG fAWmyDVFaJ1LIICjt8KEjfXe9brYcCSiBPim8agJiKthDSzuPjBiedHS7yhKoUnupxir 0MemBidNbBoQFr8FoQMrUVF624UxBtNXsU4h6dmV16MrkTy9s+8DhLPnxn7NiCC8SXYq 6Q+yTL0l4eU3t5xH88SPpYkWGbFZJp2W67fANz03Z7/EgNIB27hJjZdGxZTegk59VOeQ nqFg== Return-Path: From: Adhemerval Zanella To: libc-alpha@sourceware.org Cc: Wilco Dijkstra Subject: [PATCH 6/6] Add generic hp-timing support Date: Fri, 8 Feb 2019 16:39:10 -0200 Message-Id: <20190208183910.30799-6-adhemerval.zanella@linaro.org> In-Reply-To: <20190208183910.30799-1-adhemerval.zanella@linaro.org> References: <20190208183910.30799-1-adhemerval.zanella@linaro.org> From: Wilco Dijkstra Add missing generic hp_timing support. It uses clock_gettime (CLOCK_MONOTONIC) which has unspecified starting time, nano-second accuracy, and should faster on architectures that implementes the symbol as vDSO. Checked on aarch64-linux-gnu, x86_64-linux-gnu, and i686-linux-gnu. I also checked the builds for all afected ABIs. * benchtests/Makefile (USE_CLOCK_GETTIME) Remove. * benchtests/README: Update description. * benchtests/bench-timing.h: Default to hp-timing. * sysdeps/generic/hp-timing.h (HP_TIMING_DIFF, HP_TIMING_ACCUM_NT, HP_TIMING_PRINT): Remove. (HP_TIMING_NOW): Add generic implementation. (hp_timing_t): Change to uint64_t. --- benchtests/Makefile | 6 ----- benchtests/README | 7 +----- benchtests/bench-timing.h | 44 +++++++------------------------------ sysdeps/generic/hp-timing.h | 25 +++++++++++++-------- 4 files changed, 25 insertions(+), 57 deletions(-) -- 2.17.1 diff --git a/benchtests/Makefile b/benchtests/Makefile index e1907753b4..50cb7cd2e8 100644 --- a/benchtests/Makefile +++ b/benchtests/Makefile @@ -127,17 +127,11 @@ endif CPPFLAGS-nonlib += -DDURATION=$(BENCH_DURATION) -D_ISOMAC -# Use clock_gettime to measure performance of functions. The default is to use -# HP_TIMING if it is available. -ifdef USE_CLOCK_GETTIME -CPPFLAGS-nonlib += -DUSE_CLOCK_GETTIME -else # On x86 processors, use RDTSCP, instead of RDTSC, to measure performance # of functions. All x86 processors since 2010 support RDTSCP instruction. ifdef USE_RDTSCP CPPFLAGS-nonlib += -DUSE_RDTSCP endif -endif DETAILED_OPT := diff --git a/benchtests/README b/benchtests/README index aaf0b659e2..c4f03fd872 100644 --- a/benchtests/README +++ b/benchtests/README @@ -27,12 +27,7 @@ BENCH_DURATION. The benchmark suite does function call measurements using architecture-specific high precision timing instructions whenever available. When such support is -not available, it uses clock_gettime (CLOCK_PROCESS_CPUTIME_ID). One can force -the benchmark to use clock_gettime by invoking make as follows: - - $ make USE_CLOCK_GETTIME=1 bench - -Again, one must run `make bench-clean' before changing the measurement method. +not available, it uses clock_gettime (CLOCK_MONOTONIC). On x86 processors, RDTSCP instruction provides more precise timing data than RDTSC instruction. All x86 processors since 2010 support RDTSCP diff --git a/benchtests/bench-timing.h b/benchtests/bench-timing.h index 93fe379f99..e213dec3fd 100644 --- a/benchtests/bench-timing.h +++ b/benchtests/bench-timing.h @@ -18,49 +18,21 @@ #undef attribute_hidden #define attribute_hidden +#define __clock_gettime clock_gettime #include #include -#if HP_TIMING_INLINE && !defined USE_CLOCK_GETTIME -# define GL(x) _##x -# define GLRO(x) _##x +#define GL(x) _##x +#define GLRO(x) _##x typedef hp_timing_t timing_t; -# define TIMING_TYPE "hp_timing" +#define TIMING_TYPE "hp_timing" -# define TIMING_INIT(res) ({ (res) = 1; }) +#define TIMING_INIT(res) ({ (res) = 1; }) -# define TIMING_NOW(var) HP_TIMING_NOW (var) -# define TIMING_DIFF(diff, start, end) HP_TIMING_DIFF ((diff), (start), (end)) -# define TIMING_ACCUM(sum, diff) HP_TIMING_ACCUM_NT ((sum), (diff)) - -#else - -#include -typedef uint64_t timing_t; - -# define TIMING_TYPE "clock_gettime" - -/* Measure the resolution of the clock so we can scale the number of - benchmark iterations by this value. */ -# define TIMING_INIT(res) \ -({ \ - struct timespec start; \ - clock_getres (CLOCK_PROCESS_CPUTIME_ID, &start); \ - (res) = start.tv_nsec; \ -}) - -# define TIMING_NOW(var) \ -({ \ - struct timespec tv; \ - clock_gettime (CLOCK_PROCESS_CPUTIME_ID, &tv); \ - (var) = (uint64_t) (tv.tv_nsec + (uint64_t) 1000000000 * tv.tv_sec); \ -}) - -# define TIMING_DIFF(diff, start, end) (diff) = (end) - (start) -# define TIMING_ACCUM(sum, diff) (sum) += (diff) - -#endif +#define TIMING_NOW(var) HP_TIMING_NOW (var) +#define TIMING_DIFF(diff, start, end) HP_TIMING_DIFF ((diff), (start), (end)) +#define TIMING_ACCUM(sum, diff) HP_TIMING_ACCUM_NT ((sum), (diff)) #define TIMING_PRINT_MEAN(d_total_s, d_iters) \ printf ("\t%g", (d_total_s) / (d_iters)) diff --git a/sysdeps/generic/hp-timing.h b/sysdeps/generic/hp-timing.h index 278998d2c2..2528279558 100644 --- a/sysdeps/generic/hp-timing.h +++ b/sysdeps/generic/hp-timing.h @@ -20,16 +20,23 @@ #ifndef _HP_TIMING_H #define _HP_TIMING_H 1 -/* There are no generic definitions for the times. We could write something - using the `gettimeofday' system call where available but the overhead of - the system call might be too high. */ +#include +#include +#include -/* Provide dummy definitions. */ +/* It should not be used for ld.so. */ #define HP_TIMING_INLINE (0) -typedef int hp_timing_t; -#define HP_TIMING_NOW(var) -#define HP_TIMING_DIFF(Diff, Start, End) -#define HP_TIMING_ACCUM_NT(Sum, Diff) -#define HP_TIMING_PRINT(Buf, Len, Val) + +typedef uint64_t hp_timing_t; + +/* The clock_gettime (CLOCK_MONOTONIC) has unspecified starting time, + nano-second accuracy, and for some architectues is implemented as + vDSO symbol. */ +#define HP_TIMING_NOW(var) \ +({ \ + struct timespec tv; \ + __clock_gettime (CLOCK_MONOTONIC, &tv); \ + (var) = (tv.tv_nsec + UINT64_C(1000000000) * tv.tv_sec); \ +}) #endif /* hp-timing.h */