From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: AS31976 209.132.180.0/23 X-Spam-Status: No, score=-3.9 required=3.0 tests=AWL,BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_EF,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,RCVD_IN_DNSWL_MED,SPF_HELO_PASS,SPF_PASS shortcircuit=no autolearn=ham autolearn_force=no version=3.4.2 Received: from sourceware.org (server1.sourceware.org [209.132.180.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dcvr.yhbt.net (Postfix) with ESMTPS id 8CD1A1F463 for ; Thu, 19 Sep 2019 10:28:31 +0000 (UTC) DomainKey-Signature: a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id :list-unsubscribe:list-subscribe:list-archive:list-post :list-help:sender:subject:to:references:from:date:mime-version :in-reply-to:content-type:message-id; q=dns; s=default; b=GsjrUq CB8RskIoGUfefxnVhA945Hl2bXhKd4O1DWKbgq4Tpn0JmDlBkmlOvSZj3WhsFiqZ MsFcFX3IM1XLtZuM/UYgrOTqj00XZumrBqNxvlPxJ6pByatky+HVSa7ifOuhy9Ex F3+tn6LSBwD1Md3EIuqyaE8emnjtuqjuXJZ1Y= 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:subject:to:references:from:date:mime-version :in-reply-to:content-type:message-id; s=default; bh=KsE0jb7lWIs5 GFuhNt3S5VsCC0w=; b=HQbOTGXwjR85SpeIgR+LxwjKlbZYeJVWPOY5h5cCexcu ce2sezB7LEeiw2C/BTanamuJgZo3BaY2ZG/goyP/Dm06PbLUR36TmVPsMOrNgdl9 gS1N3mUAFtC0rmj2qkqcr2V/WTaA1v0pkmum6TWpwfHif8otnKMMzEDja8lZJGI= Received: (qmail 35988 invoked by alias); 19 Sep 2019 10:28:27 -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 Received: (qmail 35950 invoked by uid 89); 19 Sep 2019 10:28:25 -0000 Authentication-Results: sourceware.org; auth=none X-HELO: mx0a-001b2d01.pphosted.com Subject: Re: [PATCH] S390: Add support for HP_TIMING_NOW. To: libc-alpha@sourceware.org References: <463e855f-b4bb-97ea-9f83-2fa64a9e7164@linux.ibm.com> From: Stefan Liebler Date: Thu, 19 Sep 2019 12:28:15 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.8.0 MIME-Version: 1.0 In-Reply-To: Content-Type: multipart/mixed; boundary="------------FFC8F4F1A2CB042E8EF91A22" x-cbid: 19091910-0016-0000-0000-000002AE0597 X-IBM-AV-DETECTION: SAVI=unused REMOTE=unused XFE=unused x-cbparentid: 19091910-0017-0000-0000-0000330EB45C Message-Id: <63b4b727-5b1d-f5e4-3552-be8c24352ae3@linux.ibm.com> This is a multi-part message in MIME format. --------------FFC8F4F1A2CB042E8EF91A22 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit On 9/18/19 4:24 PM, Adhemerval Zanella wrote: > > > On 10/09/2019 06:13, Stefan Liebler wrote: >> + >> +#ifndef _HP_TIMING_H >> + >> +/* The stckf instruction is available starting with z9-109 zarch CPUs. >> + As there is no extra configure check for z9-109, the z10 one is used. */ >> +#ifdef HAVE_S390_MIN_Z10_ZARCH_ASM_SUPPORT >> +# define _HP_TIMING_H 1 >> +# include > > Wouldn't be simple to define _HP_TIMING_S390_H instead? > Sure. I've just committed the attached patch. Thanks. Stefan --------------FFC8F4F1A2CB042E8EF91A22 Content-Type: text/x-patch; name="20190919_hptiming_fix.patch" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="20190919_hptiming_fix.patch" commit ce89dc9649b789e85948d5da11145e34cafa2bcf Author: Stefan Liebler Date: Thu Sep 19 11:17:20 2019 +0200 S390: Use _HP_TIMING_S390_H instead of _HP_TIMING_H. Use macro _HP_TIMING_S390_H instead of _HP_TIMING_H in s390 specific hp-timing.h ChangeLog: * sysdeps/s390/hp-timing.h (_HP_TIMING_H): Undefine. (_HP_TIMING_S390_H): Define. diff --git a/sysdeps/s390/hp-timing.h b/sysdeps/s390/hp-timing.h index 793fb66a4d..c5c605c937 100644 --- a/sysdeps/s390/hp-timing.h +++ b/sysdeps/s390/hp-timing.h @@ -16,12 +16,12 @@ License along with the GNU C Library. If not, see . */ -#ifndef _HP_TIMING_H +#ifndef _HP_TIMING_S390_H +#define _HP_TIMING_S390_H 1 /* The stckf instruction is available starting with z9-109 zarch CPUs. As there is no extra configure check for z9-109, the z10 one is used. */ #ifdef HAVE_S390_MIN_Z10_ZARCH_ASM_SUPPORT -# define _HP_TIMING_H 1 # include /* We use 64 bit values for the times. --------------FFC8F4F1A2CB042E8EF91A22--