[PATCH] S390: Add support for HP_TIMING_NOW.

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

[PATCH] S390: Add support for HP_TIMING_NOW.

Stefan Liebler-2
Hi,

this patch adds support for HP_TIMING_NOW if we build at least
with -march=z10 -mzarch.  Otherwise we are still using the
generic hp-timing.h.

Bye,
Stefan

ChangeLog:

        * sysdeps/s390/hp-timing.h: New file.

20190905_hptiming.patch (2K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] S390: Add support for HP_TIMING_NOW.

Stefan Liebler-2
On 9/10/19 11:13 AM, Stefan Liebler wrote:

> Hi,
>
> this patch adds support for HP_TIMING_NOW if we build at least
> with -march=z10 -mzarch.  Otherwise we are still using the
> generic hp-timing.h.
>
> Bye,
> Stefan
>
> ChangeLog:
>
>      * sysdeps/s390/hp-timing.h: New file.

Committed

Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] S390: Add support for HP_TIMING_NOW.

Adhemerval Zanella-2
In reply to this post by Stefan Liebler-2


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 <hp-timing-common.h>

Wouldn't be simple to define _HP_TIMING_S390_H instead?
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] S390: Add support for HP_TIMING_NOW.

Stefan Liebler-2
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 <hp-timing-common.h>
>
> Wouldn't be simple to define _HP_TIMING_S390_H instead?
>
Sure. I've just committed the attached patch.

Thanks.
Stefan

20190919_hptiming_fix.patch (1K) Download Attachment