| From 7beee7b39adeda657f45989b0635033dae25a1fd Mon Sep 17 00:00:00 2001 |
| From: Raphael M Zinsly <rzinsly@linux.ibm.com> |
| Date: Thu, 12 Nov 2020 13:12:24 -0300 |
| Subject: powerpc: Add optimized stpncpy for POWER9 |
| |
| Add stpncpy support into the POWER9 strncpy. |
| |
| Reviewed-by: Matheus Castanho <msc@linux.ibm.com> |
| Reviewed-by: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com> |
| |
| diff --git a/sysdeps/powerpc/powerpc64/le/power9/stpncpy.S b/sysdeps/powerpc/powerpc64/le/power9/stpncpy.S |
| new file mode 100644 |
| index 0000000000..81d9673d8b |
| |
| |
| @@ -0,0 +1,24 @@ |
| +/* Optimized stpncpy implementation for POWER9 LE. |
| + Copyright (C) 2020 Free Software Foundation, Inc. |
| + This file is part of the GNU C Library. |
| + |
| + The GNU C Library is free software; you can redistribute it and/or |
| + modify it under the terms of the GNU Lesser General Public |
| + License as published by the Free Software Foundation; either |
| + version 2.1 of the License, or (at your option) any later version. |
| + |
| + The GNU C Library is distributed in the hope that it will be useful, |
| + but WITHOUT ANY WARRANTY; without even the implied warranty of |
| + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU |
| + Lesser General Public License for more details. |
| + |
| + You should have received a copy of the GNU Lesser General Public |
| + License along with the GNU C Library; if not, see |
| + <https://www.gnu.org/licenses/>. */ |
| + |
| +#define USE_AS_STPNCPY |
| +#include <sysdeps/powerpc/powerpc64/le/power9/strncpy.S> |
| + |
| +weak_alias (__stpncpy, stpncpy) |
| +libc_hidden_def (__stpncpy) |
| +libc_hidden_builtin_def (stpncpy) |
| diff --git a/sysdeps/powerpc/powerpc64/le/power9/strncpy.S b/sysdeps/powerpc/powerpc64/le/power9/strncpy.S |
| index cbfc37bda3..b4ba428662 100644 |
| |
| |
| @@ -18,11 +18,19 @@ |
| |
| #include <sysdep.h> |
| |
| +#ifdef USE_AS_STPNCPY |
| +# ifndef STPNCPY |
| +# define FUNC_NAME __stpncpy |
| +# else |
| +# define FUNC_NAME STPNCPY |
| +# endif |
| +#else |
| # ifndef STRNCPY |
| # define FUNC_NAME strncpy |
| # else |
| # define FUNC_NAME STRNCPY |
| # endif |
| +#endif /* !USE_AS_STPNCPY */ |
| |
| #ifndef MEMSET |
| /* For builds without IFUNC support, local calls should be made to internal |
| @@ -41,6 +49,12 @@ |
| |
| char * [r3] strncpy (char *dest [r3], const char *src [r4], size_t n [r5]) |
| |
| + or |
| + |
| + char * [r3] stpncpy (char *dest [r3], const char *src [r4], size_t n [r5]) |
| + |
| + if USE_AS_STPNCPY is defined. |
| + |
| The implementation can load bytes past a null terminator, but only |
| up to the next 16-byte aligned address, so it never crosses a page. */ |
| |
| @@ -66,7 +80,15 @@ ENTRY (FUNC_NAME, 4) |
| |
| /* Empty/1-byte string optimization */ |
| cmpdi r5,0 |
| +#ifdef USE_AS_STPNCPY |
| + bgt L(cont) |
| + /* Compute pointer to last byte copied into dest. */ |
| + addi r3,r3,1 |
| + blr |
| +L(cont): |
| +#else |
| beqlr |
| +#endif |
| |
| addi r4,r4,1 |
| neg r7,r4 |
| @@ -96,12 +118,20 @@ ENTRY (FUNC_NAME, 4) |
| sldi r10,r5,56 /* stxvl wants size in top 8 bits */ |
| stxvl 32+v0,r11,r10 /* Partial store */ |
| |
| +#ifdef USE_AS_STPNCPY |
| + /* Compute pointer to last byte copied into dest. */ |
| + add r3,r11,r5 |
| +#endif |
| blr |
| |
| L(null): |
| sldi r10,r8,56 /* stxvl wants size in top 8 bits */ |
| stxvl 32+v0,r11,r10 /* Partial store */ |
| |
| +#ifdef USE_AS_STPNCPY |
| + /* Compute pointer to last byte copied into dest. */ |
| + add r3,r11,r7 |
| +#endif |
| add r11,r11,r8 |
| sub r5,r5,r8 |
| b L(zero_padding) |
| @@ -185,6 +215,10 @@ L(n_tail4): |
| sldi r10,r5,56 /* stxvl wants size in top 8 bits */ |
| addi r11,r11,48 /* Offset */ |
| stxvl 32+v3,r11,r10 /* Partial store */ |
| +#ifdef USE_AS_STPNCPY |
| + /* Compute pointer to last byte copied into dest. */ |
| + add r3,r11,r5 |
| +#endif |
| blr |
| |
| L(prep_n_tail1): |
| @@ -196,6 +230,10 @@ L(prep_n_tail1): |
| L(n_tail1): |
| sldi r10,r5,56 /* stxvl wants size in top 8 bits */ |
| stxvl 32+v0,r11,r10 /* Partial store */ |
| +#ifdef USE_AS_STPNCPY |
| + /* Compute pointer to last byte copied into dest. */ |
| + add r3,r11,r5 |
| +#endif |
| blr |
| |
| L(prep_n_tail2): |
| @@ -209,6 +247,10 @@ L(n_tail2): |
| sldi r10,r5,56 /* stxvl wants size in top 8 bits */ |
| addi r11,r11,16 /* offset */ |
| stxvl 32+v1,r11,r10 /* Partial store */ |
| +#ifdef USE_AS_STPNCPY |
| + /* Compute pointer to last byte copied into dest. */ |
| + add r3,r11,r5 |
| +#endif |
| blr |
| |
| L(prep_n_tail3): |
| @@ -223,6 +265,10 @@ L(n_tail3): |
| sldi r10,r5,56 /* stxvl wants size in top 8 bits */ |
| addi r11,r11,32 /* Offset */ |
| stxvl 32+v2,r11,r10 /* Partial store */ |
| +#ifdef USE_AS_STPNCPY |
| + /* Compute pointer to last byte copied into dest. */ |
| + add r3,r11,r5 |
| +#endif |
| blr |
| |
| L(prep_tail1): |
| @@ -232,6 +278,10 @@ L(tail1): |
| addi r9,r8,1 /* Add null terminator */ |
| sldi r10,r9,56 /* stxvl wants size in top 8 bits */ |
| stxvl 32+v0,r11,r10 /* Partial store */ |
| +#ifdef USE_AS_STPNCPY |
| + /* Compute pointer to last byte copied into dest. */ |
| + add r3,r11,r8 |
| +#endif |
| add r11,r11,r9 |
| sub r5,r5,r9 |
| b L(zero_padding) |
| @@ -246,6 +296,10 @@ L(tail2): |
| sldi r10,r9,56 /* stxvl wants size in top 8 bits */ |
| addi r11,r11,16 /* offset */ |
| stxvl 32+v1,r11,r10 /* Partial store */ |
| +#ifdef USE_AS_STPNCPY |
| + /* Compute pointer to last byte copied into dest. */ |
| + add r3,r11,r8 |
| +#endif |
| add r11,r11,r9 |
| sub r5,r5,r9 |
| b L(zero_padding) |
| @@ -261,6 +315,10 @@ L(tail3): |
| sldi r10,r9,56 /* stxvl wants size in top 8 bits */ |
| addi r11,r11,32 /* offset */ |
| stxvl 32+v2,r11,r10 /* Partial store */ |
| +#ifdef USE_AS_STPNCPY |
| + /* Compute pointer to last byte copied into dest. */ |
| + add r3,r11,r8 |
| +#endif |
| add r11,r11,r9 |
| sub r5,r5,r9 |
| b L(zero_padding) |
| @@ -276,6 +334,10 @@ L(tail4): |
| sldi r10,r9,56 /* stxvl wants size in top 8 bits */ |
| addi r11,r11,48 /* offset */ |
| stxvl 32+v3,r11,r10 /* Partial store */ |
| +#ifdef USE_AS_STPNCPY |
| + /* Compute pointer to last byte copied into dest. */ |
| + add r3,r11,r8 |
| +#endif |
| add r11,r11,r9 |
| sub r5,r5,r9 |
| |
| @@ -331,7 +393,8 @@ L(zero_padding_memset): |
| ld r0,FRAMESIZE+16(r1) |
| |
| mr r3,r30 /* Restore the return value of strncpy, i.e.: |
| - dest. */ |
| + dest. For stpncpy, the return value is the |
| + same as return value of memset. */ |
| ld r30,FRAMESIZE-8(r1) /* Restore r30. */ |
| /* Restore the stack frame. */ |
| addi r1,r1,FRAMESIZE |
| @@ -342,3 +405,6 @@ L(zero_padding_memset): |
| blr |
| |
| END (FUNC_NAME) |
| +#ifndef USE_AS_STPNCPY |
| +libc_hidden_builtin_def (strncpy) |
| +#endif |
| diff --git a/sysdeps/powerpc/powerpc64/multiarch/Makefile b/sysdeps/powerpc/powerpc64/multiarch/Makefile |
| index cd2b47b403..f46bf50732 100644 |
| |
| |
| @@ -33,7 +33,7 @@ sysdep_routines += memcpy-power8-cached memcpy-power7 memcpy-a2 memcpy-power6 \ |
| |
| ifneq (,$(filter %le,$(config-machine))) |
| sysdep_routines += strcmp-power9 strncmp-power9 strcpy-power9 stpcpy-power9 \ |
| - rawmemchr-power9 strlen-power9 strncpy-power9 |
| + rawmemchr-power9 strlen-power9 strncpy-power9 stpncpy-power9 |
| endif |
| CFLAGS-strncase-power7.c += -mcpu=power7 -funroll-loops |
| CFLAGS-strncase_l-power7.c += -mcpu=power7 -funroll-loops |
| diff --git a/sysdeps/powerpc/powerpc64/multiarch/ifunc-impl-list.c b/sysdeps/powerpc/powerpc64/multiarch/ifunc-impl-list.c |
| index 135326c97a..8e19ebbf09 100644 |
| |
| |
| @@ -318,6 +318,12 @@ __libc_ifunc_impl_list (const char *name, struct libc_ifunc_impl *array, |
| |
| /* Support sysdeps/powerpc/powerpc64/multiarch/stpncpy.c. */ |
| IFUNC_IMPL (i, name, stpncpy, |
| +#ifdef __LITTLE_ENDIAN__ |
| + IFUNC_IMPL_ADD (array, i, stpncpy, |
| + (hwcap2 & PPC_FEATURE2_ARCH_3_00) |
| + && (hwcap & PPC_FEATURE_HAS_VSX), |
| + __stpncpy_power9) |
| +#endif |
| IFUNC_IMPL_ADD (array, i, stpncpy, |
| hwcap2 & PPC_FEATURE2_ARCH_2_07, |
| __stpncpy_power8) |
| diff --git a/sysdeps/powerpc/powerpc64/multiarch/stpncpy-power9.S b/sysdeps/powerpc/powerpc64/multiarch/stpncpy-power9.S |
| new file mode 100644 |
| index 0000000000..1188bd0894 |
| |
| |
| @@ -0,0 +1,29 @@ |
| +/* Optimized stpncpy implementation for POWER9 LE. |
| + Copyright (C) 2020 Free Software Foundation, Inc. |
| + This file is part of the GNU C Library. |
| + |
| + The GNU C Library is free software; you can redistribute it and/or |
| + modify it under the terms of the GNU Lesser General Public |
| + License as published by the Free Software Foundation; either |
| + version 2.1 of the License, or (at your option) any later version. |
| + |
| + The GNU C Library is distributed in the hope that it will be useful, |
| + but WITHOUT ANY WARRANTY; without even the implied warranty of |
| + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU |
| + Lesser General Public License for more details. |
| + |
| + You should have received a copy of the GNU Lesser General Public |
| + License along with the GNU C Library; if not, see |
| + <https://www.gnu.org/licenses/>. */ |
| + |
| +#define STPNCPY __stpncpy_power9 |
| + |
| +#undef libc_hidden_builtin_def |
| +#define libc_hidden_builtin_def(name) |
| + |
| +#define MEMSET __memset_power8 |
| +#ifdef SHARED |
| +# define MEMSET_is_local |
| +#endif |
| + |
| +#include <sysdeps/powerpc/powerpc64/le/power9/stpncpy.S> |
| diff --git a/sysdeps/powerpc/powerpc64/multiarch/stpncpy.c b/sysdeps/powerpc/powerpc64/multiarch/stpncpy.c |
| index 17df886431..3758f29ad1 100644 |
| |
| |
| @@ -26,10 +26,18 @@ |
| extern __typeof (__stpncpy) __stpncpy_ppc attribute_hidden; |
| extern __typeof (__stpncpy) __stpncpy_power7 attribute_hidden; |
| extern __typeof (__stpncpy) __stpncpy_power8 attribute_hidden; |
| +# ifdef __LITTLE_ENDIAN__ |
| +extern __typeof (__stpncpy) __stpncpy_power9 attribute_hidden; |
| +# endif |
| # undef stpncpy |
| # undef __stpncpy |
| |
| libc_ifunc_redirected (__redirect___stpncpy, __stpncpy, |
| +# ifdef __LITTLE_ENDIAN__ |
| + (hwcap2 & PPC_FEATURE2_ARCH_3_00) && |
| + (hwcap & PPC_FEATURE_HAS_VSX) |
| + ? __stpncpy_power9 : |
| +# endif |
| (hwcap2 & PPC_FEATURE2_ARCH_2_07) |
| ? __stpncpy_power8 |
| : (hwcap & PPC_FEATURE_HAS_VSX) |