[PATCH] powerpc: Add support for POWER10

[Paul E Murphy]

On 6/24/20 7:10 PM, Tulio Magno Quites Machado Filho via Libc-alpha wrote:
> Add the directories to hold POWER10 files.
> 
> Print the correct values for AT_PLATFORM and AT_BASE_PLATFORM.
> ---
>   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          |  3 +++
>   sysdeps/powerpc/powerpc64/be/power10/fpu/Implies      |  2 ++
>   .../powerpc64/be/power10/fpu/multiarch/Implies        |  1 +
>   .../powerpc/powerpc64/be/power10/multiarch/Implies    |  1 +
>   sysdeps/powerpc/powerpc64/le/power10/Implies          |  1 +
>   sysdeps/powerpc/powerpc64/le/power10/fpu/Implies      |  1 +
>   .../powerpc64/le/power10/fpu/multiarch/Implies        |  1 +
>   .../powerpc/powerpc64/le/power10/multiarch/Implies    |  1 +

I think sysdeps/powerpc/preconfigure needs updated to detect power10 too.

>   12 files changed, 25 insertions(+), 1 deletion(-)
>   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;

Other two digit machines also increment the str pointer.  Should that be 
done here too?  Though, it seems unnecessary.  Otherwise, OK.

> +	    }
> +	  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

OK

> diff --git a/sysdeps/powerpc/powerpc32/power10/fpu/multiarch/Implies b/sysdeps/powerpc/powerpc32/power10/fpu/multiarch/Implies

OK

> diff --git a/sysdeps/powerpc/powerpc32/power10/multiarch/Implies b/sysdeps/powerpc/powerpc32/power10/multiarch/Implies

OK


> diff --git a/sysdeps/powerpc/powerpc64/be/power10/Implies b/sysdeps/powerpc/powerpc64/be/power10/Implies
> new file mode 100644
> index 0000000000..2a4fe87b9e
> --- /dev/null
> +++ b/sysdeps/powerpc/powerpc64/be/power10/Implies
> @@ -0,0 +1,3 @@
> +powerpc/powerpc64/power10
> +powerpc/powerpc64/be/power9/fpu
> +powerpc/powerpc64/be/power9

OK.  powerpc64/power* is a terminal directory in the Implies order, so 
it's ok it doesn't exist. (similarly for the fpu subdirectory if needed)

> diff --git a/sysdeps/powerpc/powerpc64/be/power10/fpu/Implies b/sysdeps/powerpc/powerpc64/be/power10/fpu/Implies

OK

> diff --git a/sysdeps/powerpc/powerpc64/be/power10/fpu/multiarch/Implies b/sysdeps/powerpc/powerpc64/be/power10/fpu/multiarch/Implies

OK

> diff --git a/sysdeps/powerpc/powerpc64/be/power10/multiarch/Implies b/sysdeps/powerpc/powerpc64/be/power10/multiarch/Implies


OK

> diff --git a/sysdeps/powerpc/powerpc64/le/power10/Implies b/sysdeps/powerpc/powerpc64/le/power10/Implies


OK

> diff --git a/sysdeps/powerpc/powerpc64/le/power10/fpu/Implies b/sysdeps/powerpc/powerpc64/le/power10/fpu/Implies

OK

> diff --git a/sysdeps/powerpc/powerpc64/le/power10/fpu/multiarch/Implies b/sysdeps/powerpc/powerpc64/le/power10/fpu/multiarch/Implies

OK

> diff --git a/sysdeps/powerpc/powerpc64/le/power10/multiarch/Implies b/sysdeps/powerpc/powerpc64/le/power10/multiarch/Implies

OK