From: Matheus Castanho <msc@linux.ibm.com>
To: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>,
libc-alpha@sourceware.org
Subject: Re: [PATCHv2] powerpc: Add support for POWER10
Date: Fri, 26 Jun 2020 16:09:19 -0300 [thread overview]
Message-ID: <934f9570-768e-3dd4-2f53-fb09ba719152@linux.ibm.com> (raw)
In-Reply-To: <20200625230116.301613-1-tuliom@linux.ibm.com>
Hi Tulio,
On 6/25/20 8:01 PM, Tulio Magno Quites Machado Filho wrote:
> Changes since v1:
> - Remove 2 directories from Implies. Added another directory that was
> missing.
> - Listed power10 in preconfigure.ac
>
> ---8<---
>
> 1. Add the directories to hold POWER10 files.
>
> 2. Print the correct values for AT_PLATFORM and AT_BASE_PLATFORM.
>
> 3. Let submachine=power10 be set automatically.
> ---
> sysdeps/powerpc/dl-procinfo.h | 11 ++++++++++-
> sysdeps/powerpc/powerpc32/power10/Implies | 2 ++
> .../powerpc/powerpc32/power10/fpu/multiarch/Implies | 1 +
> sysdeps/powerpc/powerpc32/power10/multiarch/Implies | 1 +
> sysdeps/powerpc/powerpc64/be/power10/Implies | 2 ++
> sysdeps/powerpc/powerpc64/be/power10/fpu/Implies | 1 +
> .../powerpc64/be/power10/fpu/multiarch/Implies | 1 +
> .../powerpc/powerpc64/be/power10/multiarch/Implies | 1 +
> sysdeps/powerpc/powerpc64/le/power10/Implies | 2 ++
> sysdeps/powerpc/powerpc64/le/power10/fpu/Implies | 1 +
> .../powerpc64/le/power10/fpu/multiarch/Implies | 1 +
> .../powerpc/powerpc64/le/power10/multiarch/Implies | 1 +
> sysdeps/powerpc/preconfigure | 2 +-
> sysdeps/powerpc/preconfigure.ac | 2 +-
> 14 files changed, 26 insertions(+), 3 deletions(-)
> create mode 100644 sysdeps/powerpc/powerpc32/power10/Implies
> create mode 100644 sysdeps/powerpc/powerpc32/power10/fpu/multiarch/Implies
> create mode 100644 sysdeps/powerpc/powerpc32/power10/multiarch/Implies
> create mode 100644 sysdeps/powerpc/powerpc64/be/power10/Implies
> create mode 100644 sysdeps/powerpc/powerpc64/be/power10/fpu/Implies
> create mode 100644 sysdeps/powerpc/powerpc64/be/power10/fpu/multiarch/Implies
> create mode 100644 sysdeps/powerpc/powerpc64/be/power10/multiarch/Implies
> create mode 100644 sysdeps/powerpc/powerpc64/le/power10/Implies
> create mode 100644 sysdeps/powerpc/powerpc64/le/power10/fpu/Implies
> create mode 100644 sysdeps/powerpc/powerpc64/le/power10/fpu/multiarch/Implies
> create mode 100644 sysdeps/powerpc/powerpc64/le/power10/multiarch/Implies
>
> diff --git a/sysdeps/powerpc/dl-procinfo.h b/sysdeps/powerpc/dl-procinfo.h
> index 9ae85f46ec..497dc568e9 100644
> --- a/sysdeps/powerpc/dl-procinfo.h
> +++ b/sysdeps/powerpc/dl-procinfo.h
> @@ -37,7 +37,7 @@
> #define HWCAP_IMPORTANT (PPC_FEATURE_HAS_ALTIVEC \
> + PPC_FEATURE_HAS_DFP)
>
> -#define _DL_PLATFORMS_COUNT 15
> +#define _DL_PLATFORMS_COUNT 16
>
> #define _DL_FIRST_PLATFORM 32
> /* Mask to filter out platforms. */
> @@ -60,6 +60,7 @@
> #define PPC_PLATFORM_PPC476 12
> #define PPC_PLATFORM_POWER8 13
> #define PPC_PLATFORM_POWER9 14
> +#define PPC_PLATFORM_POWER10 15
>
> static inline const char *
> __attribute__ ((unused))
> @@ -91,6 +92,14 @@ _dl_string_platform (const char *str)
> str += 5;
> switch (*str)
> {
> + case '1':
> + if (str[1] == '0')
> + {
> + ret = _DL_FIRST_PLATFORM + PPC_PLATFORM_POWER10;
> + }
> + else
> + return -1;
> + break;
> case '4':
> ret = _DL_FIRST_PLATFORM + PPC_PLATFORM_POWER4;
> break;
> diff --git a/sysdeps/powerpc/powerpc32/power10/Implies b/sysdeps/powerpc/powerpc32/power10/Implies
> new file mode 100644
> index 0000000000..b298aca71f
> --- /dev/null
> +++ b/sysdeps/powerpc/powerpc32/power10/Implies
> @@ -0,0 +1,2 @@
> +powerpc/powerpc32/power9/fpu
> +powerpc/powerpc32/power9
> diff --git a/sysdeps/powerpc/powerpc32/power10/fpu/multiarch/Implies b/sysdeps/powerpc/powerpc32/power10/fpu/multiarch/Implies
> new file mode 100644
> index 0000000000..1ffddd1cda
> --- /dev/null
> +++ b/sysdeps/powerpc/powerpc32/power10/fpu/multiarch/Implies
> @@ -0,0 +1 @@
> +powerpc/powerpc32/power9/fpu/multiarch
> diff --git a/sysdeps/powerpc/powerpc32/power10/multiarch/Implies b/sysdeps/powerpc/powerpc32/power10/multiarch/Implies
> new file mode 100644
> index 0000000000..46363b5207
> --- /dev/null
> +++ b/sysdeps/powerpc/powerpc32/power10/multiarch/Implies
> @@ -0,0 +1 @@
> +powerpc/powerpc32/power9/multiarch
> diff --git a/sysdeps/powerpc/powerpc64/be/power10/Implies b/sysdeps/powerpc/powerpc64/be/power10/Implies
> new file mode 100644
> index 0000000000..5aea9bc144
> --- /dev/null
> +++ b/sysdeps/powerpc/powerpc64/be/power10/Implies
> @@ -0,0 +1,2 @@
> +powerpc/powerpc64/be/power9/fpu
> +powerpc/powerpc64/be/power9
> diff --git a/sysdeps/powerpc/powerpc64/be/power10/fpu/Implies b/sysdeps/powerpc/powerpc64/be/power10/fpu/Implies
> new file mode 100644
> index 0000000000..19b2a318e2
> --- /dev/null
> +++ b/sysdeps/powerpc/powerpc64/be/power10/fpu/Implies
> @@ -0,0 +1 @@
> +powerpc/powerpc64/be/power9/fpu
> diff --git a/sysdeps/powerpc/powerpc64/be/power10/fpu/multiarch/Implies b/sysdeps/powerpc/powerpc64/be/power10/fpu/multiarch/Implies
> new file mode 100644
> index 0000000000..a336feb55b
> --- /dev/null
> +++ b/sysdeps/powerpc/powerpc64/be/power10/fpu/multiarch/Implies
> @@ -0,0 +1 @@
> +powerpc/powerpc64/be/power9/fpu/multiarch
> diff --git a/sysdeps/powerpc/powerpc64/be/power10/multiarch/Implies b/sysdeps/powerpc/powerpc64/be/power10/multiarch/Implies
> new file mode 100644
> index 0000000000..b4c0f0a58a
> --- /dev/null
> +++ b/sysdeps/powerpc/powerpc64/be/power10/multiarch/Implies
> @@ -0,0 +1 @@
> +powerpc/powerpc64/be/power9/multiarch
> diff --git a/sysdeps/powerpc/powerpc64/le/power10/Implies b/sysdeps/powerpc/powerpc64/le/power10/Implies
> new file mode 100644
> index 0000000000..9e1b787348
> --- /dev/null
> +++ b/sysdeps/powerpc/powerpc64/le/power10/Implies
> @@ -0,0 +1,2 @@
> +powerpc/powerpc64/le/power9/fpu
> +powerpc/powerpc64/le/power9
> diff --git a/sysdeps/powerpc/powerpc64/le/power10/fpu/Implies b/sysdeps/powerpc/powerpc64/le/power10/fpu/Implies
> new file mode 100644
> index 0000000000..1b3525a0d7
> --- /dev/null
> +++ b/sysdeps/powerpc/powerpc64/le/power10/fpu/Implies
> @@ -0,0 +1 @@
> +powerpc/powerpc64/le/power9/fpu
> diff --git a/sysdeps/powerpc/powerpc64/le/power10/fpu/multiarch/Implies b/sysdeps/powerpc/powerpc64/le/power10/fpu/multiarch/Implies
> new file mode 100644
> index 0000000000..189beb801d
> --- /dev/null
> +++ b/sysdeps/powerpc/powerpc64/le/power10/fpu/multiarch/Implies
> @@ -0,0 +1 @@
> +powerpc/powerpc64/le/power9/fpu/multiarch
> diff --git a/sysdeps/powerpc/powerpc64/le/power10/multiarch/Implies b/sysdeps/powerpc/powerpc64/le/power10/multiarch/Implies
> new file mode 100644
> index 0000000000..e6e5fc887e
> --- /dev/null
> +++ b/sysdeps/powerpc/powerpc64/le/power10/multiarch/Implies
> @@ -0,0 +1 @@
> +powerpc/powerpc64/le/power9/multiarch
> diff --git a/sysdeps/powerpc/preconfigure b/sysdeps/powerpc/preconfigure
> index deac5a41e2..dfe8e20399 100644
> --- a/sysdeps/powerpc/preconfigure
> +++ b/sysdeps/powerpc/preconfigure
> @@ -56,7 +56,7 @@ fi
>
> ;;
>
> - a2|970|power[4-9]|power5x|power6+)
> + a2|970|power[4-9]|power5x|power6+|power10)
> submachine=${archcpu}
> if ${libc_cv_cc_submachine+:} false; then :
> $as_echo_n "(cached) " >&6
> diff --git a/sysdeps/powerpc/preconfigure.ac b/sysdeps/powerpc/preconfigure.ac
> index e6a5ae93a5..6c63bd8257 100644
> --- a/sysdeps/powerpc/preconfigure.ac
> +++ b/sysdeps/powerpc/preconfigure.ac
> @@ -46,7 +46,7 @@ case "${machine}:${submachine}" in
> AC_CACHE_VAL(libc_cv_cc_submachine,libc_cv_cc_submachine="")
> ;;
>
> - a2|970|power[[4-9]]|power5x|power6+)
> + a2|970|power[[4-9]]|power5x|power6+|power10)
> submachine=${archcpu}
> AC_CACHE_VAL(libc_cv_cc_submachine,libc_cv_cc_submachine="")
> ;;
>
This updated version looks good to me.
Thanks,
Matheus Castanho
next prev parent reply other threads:[~2020-06-26 19:09 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-06-25 0:10 [PATCH] " Tulio Magno Quites Machado Filho
2020-06-25 18:43 ` Paul E Murphy
2020-06-25 22:36 ` Tulio Magno Quites Machado Filho
2020-06-25 23:01 ` [PATCHv2] " Tulio Magno Quites Machado Filho
2020-06-26 19:09 ` Matheus Castanho [this message]
2020-06-26 20:48 ` Paul E Murphy
2020-06-29 13:20 ` Tulio Magno Quites Machado Filho
2020-06-25 18:47 ` [PATCH] " Matheus Castanho
2020-06-25 19:18 ` Tulio Magno Quites Machado Filho
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=934f9570-768e-3dd4-2f53-fb09ba719152@linux.ibm.com \
--to=msc@linux.ibm.com \
--cc=libc-alpha@sourceware.org \
--cc=tuliom@linux.ibm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).